From kde-panel-devel Mon Jul 26 18:23:08 2010 From: "Aaron Seigo" Date: Mon, 26 Jul 2010 18:23:08 +0000 To: kde-panel-devel Subject: Re: Review Request: Manual panel hiding [WIP] Message-Id: <20100726182308.14746.94345 () localhost> X-MARC-Message: https://marc.info/?l=kde-panel-devel&m=128016861230913 MIME-Version: 1 Content-Type: multipart/mixed; boundary="--===============1903611933==" --===============1903611933== Content-Type: multipart/alternative; boundary="===============7628145021907040558==" --===============7628145021907040558== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable > On 2010-05-04 03:49:54, Aaron Seigo wrote: > > any updates on this? > = > Will Stephenson wrote: > Ditto - our users are still asking for it - as they do on every relea= se. *gasp* all of them? ;) more seriously, "our users" as a generic term is not a very interesting met= ric, since in reality it could be 10 loud users out of 1000 or it could be = an actually significant %. :) that said ... looks like this patch is abandoned. would be nice if someone = picked it up and finished it out. - Aaron ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/3121/#review5390 ----------------------------------------------------------- On 2010-03-04 19:19:53, Andrzej JR Hunt wrote: > = > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviewboard.kde.org/r/3121/ > ----------------------------------------------------------- > = > (Updated 2010-03-04 19:19:53) > = > = > Review request for Plasma. > = > = > Summary > ------- > = > Manual panel hiding patch (still in progress). Currently only horizontal = panels are properly implemented, there is still a display bug whereby the p= lasma toolbox (cashew) ignores the contentsMargins set on the containment a= nd thus gets covered by the hiding buttons. > = > = > This addresses bug 158556. > https://bugs.kde.org/show_bug.cgi?id=3D158556 > = > = > Diffs > ----- > = > /trunk/KDE/kdebase/workspace/plasma/desktop/shell/panelcontroller.h 109= 7398 = > /trunk/KDE/kdebase/workspace/plasma/desktop/shell/panelcontroller.cpp 1= 097398 = > /trunk/KDE/kdebase/workspace/plasma/desktop/shell/panelview.h 1097398 = > /trunk/KDE/kdebase/workspace/plasma/desktop/shell/panelview.cpp 1097398 = > /trunk/KDE/kdebase/workspace/plasma/desktop/shell/scripting/panel.cpp 1= 097398 = > = > Diff: http://reviewboard.kde.org/r/3121/diff > = > = > Testing > ------- > = > Hiding the panels, changing states, changing positions and sizes. > = > = > Thanks, > = > Andrzej JR > = > --===============7628145021907040558== Content-Type: text/html; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable
This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde= .org/r/3121/

On May 4th, 2010, 3:49 a.m., Aaron Seigo wrote:

any updates on this?

On July 26th, 2010, 1:03 p.m., Will Stephenson wrote:

Ditto - our users are still asking for it - as they do on every rele=
ase.
*gasp* all of them? ;)

more seriously, "our users" as a generic term is not a very inter=
esting metric, since in reality it could be 10 loud users out of 1000 or it=
 could be an actually significant %. :)

that said ... looks like this patch is abandoned. would be nice if someone =
picked it up and finished it out.

- Aaron


On March 4th, 2010, 7:19 p.m., Andrzej JR Hunt wrote:

Review request for Plasma.
By Andrzej JR Hunt.

Updated 2010-03-04 19:19:53

Descripti= on 

Manual panel hiding patch (still i=
n progress). Currently only horizontal panels are properly implemented, the=
re is still a display bug whereby the plasma toolbox (cashew) ignores the c=
ontentsMargins set on the containment and thus gets covered by the hiding b=
uttons.

Testing <= /h1> 
Hiding the panels, changing states=
, changing positions and sizes.
Bugs: 158556

Diffs=

  • /trunk/KDE/kdebase/workspace/plasma/desktop/shell/panelcontroller.h (1097398)
  • /trunk/KDE/kdebase/workspace/plasma/desktop/shell/panelcontroller.cpp = (1097398)
  • /trunk/KDE/kdebase/workspace/plasma/desktop/shell/panelview.h (1097398)
  • /trunk/KDE/kdebase/workspace/plasma/desktop/shell/panelview.cpp (1097398)
  • /trunk/KDE/kdebase/workspace/plasma/desktop/shell/scripting/panel.cpp = (1097398)

View Diff
--===============7628145021907040558==-- --===============1903611933== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel --===============1903611933==--