[prev in list] [next in list] [prev in thread] [next in thread]
List: kde-panel-devel
Subject: Re: Review Request 117450: Add Milou to the default panel layout
From: "Mark Gaiser" <markg85 () gmail ! com>
Date: 2014-04-12 18:43:50
Message-ID: 20140412184350.22682.50425 () probe ! kde ! org
[Download RAW message or body]
[Attachment #2 (multipart/alternative)]
> On April 12, 2014, 9:29 a.m., Giorgos Tsiapaliokas wrote:
> > I would prefer if we didn't add milou to the panel by default, because
> > a. IMO the panel will become over populated.
> > b. IMO the first thing the user will notice after an update in KDE is the panel.
> > The user will see that there is a new plasmoid which didn't exist back in
> > KDE4 by default. IIRC we had said that plasma next won't have any new big ui \
> > changes and this one seems to me a big one. Considering that users will update \
> > from KDE4 to the new major version, a changed panel may make plasma look "evil" \
> > again.
>
> Vishesh Handa wrote:
> Right no big UI changes - apart from KRunner, Kickoff and the System Tray.
-- meant as reply to Giorgos --
What kind of bogus argument is "a"? Sure, adding stuff to the tray should be done as \
little as possible to prevent "tray bloat" but now that we have a rocking search \
plasmoid it would certainly be worth getting that in the tray. I would consider \
searching - specially if the results are good - a vital part of the desktop. Not \
adding this would imho be a big mistake in the direction of plasma. This is what KDE \
needs in the search area which we where missing quite badly.
Your point b really just strikes me as a pointless argument.
"a changed panel may make plasma look "evil" again" ... Really? So a good search \
plasmoid is going to be considered evil? If we're going that route.. I would consider \
the notification icon (in kde 4, i haven't tried plasma next enough to say something \
about it there) to be very evil. "I" hardly ever find any of the notifications in \
there close to useful. With the exception of copying files.
I hope the plasma folks could reconsider this. IMHO it would be a missed opportunity \
if it wasn't included.
- Mark
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/117450/#review55488
-----------------------------------------------------------
On April 9, 2014, 3:22 p.m., Vishesh Handa wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/117450/
> -----------------------------------------------------------
>
> (Updated April 9, 2014, 3:22 p.m.)
>
>
> Review request for Plasma.
>
>
> Repository: plasma-desktop
>
>
> Description
> -------
>
> KRunner is hidden away as it is only accessible via a shortcut. It will be nice to \
> have the plasmoid visible by default. It will not be an extra dependency as \
> plasma-workspace already depends on Milou because of KRunner.
>
> Diffs
> -----
>
> desktoppackage/contents/layout.js 303e7af
>
> Diff: https://git.reviewboard.kde.org/r/117450/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Vishesh Handa
>
>
[Attachment #5 (text/html)]
<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 \
solid;"> <tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="https://git.reviewboard.kde.org/r/117450/">https://git.reviewboard.kde.org/r/117450/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: \
10px;"> <p style="margin-top: 0;">On April 12th, 2014, 9:29 a.m. UTC, <b>Giorgos \
Tsiapaliokas</b> wrote:</p> <blockquote style="margin-left: 1em; border-left: 2px \
solid #d0d0d0; padding-left: 10px;"> <pre style="white-space: pre-wrap; white-space: \
-moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: \
break-word;">I would prefer if we didn't add milou to the panel by default, \
because a. IMO the panel will become over populated.
b. IMO the first thing the user will notice after an update in KDE is the panel.
The user will see that there is a new plasmoid which didn't exist back in
KDE4 by default. IIRC we had said that plasma next won't have any new big ui \
changes and this one seems to me a big one. Considering that users will update from \
KDE4 to the new major version, a changed panel may make plasma look "evil" \
again.</pre> </blockquote>
<p>On April 12th, 2014, 5:50 p.m. UTC, <b>Vishesh Handa</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: \
10px;"> <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: \
-pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Right no big UI changes \
- apart from KRunner, Kickoff and the System Tray.</pre> </blockquote>
</blockquote>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: \
-pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">-- meant as reply to \
Giorgos -- What kind of bogus argument is "a"? Sure, adding stuff to the \
tray should be done as little as possible to prevent "tray bloat" but now \
that we have a rocking search plasmoid it would certainly be worth getting that in \
the tray. I would consider searching - specially if the results are good - a vital \
part of the desktop. Not adding this would imho be a big mistake in the direction of \
plasma. This is what KDE needs in the search area which we where missing quite badly.
Your point b really just strikes me as a pointless argument.
"a changed panel may make plasma look "evil" again" ... Really? \
So a good search plasmoid is going to be considered evil? If we're going that \
route.. I would consider the notification icon (in kde 4, i haven't tried plasma \
next enough to say something about it there) to be very evil. "I" hardly \
ever find any of the notifications in there close to useful. With the exception of \
copying files.
I hope the plasma folks could reconsider this. IMHO it would be a missed opportunity \
if it wasn't included.</pre> <br />
<p>- Mark</p>
<br />
<p>On April 9th, 2014, 3:22 p.m. UTC, Vishesh Handa wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" \
style="background-image: \
url('https://git.reviewboard.kde.org/static/rb/images/review_request_box_top_bg.ab6f3b1072c9.png'); \
background-position: left top; background-repeat: repeat-x; border: 1px black \
solid;"> <tr>
<td>
<div>Review request for Plasma.</div>
<div>By Vishesh Handa.</div>
<p style="color: grey;"><i>Updated April 9, 2014, 3:22 p.m.</i></p>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
plasma-desktop
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" \
style="border: 1px solid #b8b5a0"> <tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: \
-moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: \
break-word;">KRunner is hidden away as it is only accessible via a shortcut. It will \
be nice to have the plasmoid visible by default. It will not be an extra dependency \
as plasma-workspace already depends on Milou because of KRunner.</pre> </td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>desktoppackage/contents/layout.js <span style="color: \
grey">(303e7af)</span></li>
</ul>
<p><a href="https://git.reviewboard.kde.org/r/117450/diff/" style="margin-left: \
3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>
_______________________________________________
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic