> On 2010-01-01 00:15:02, Aaron Seigo wrote: > >
Tested and found it works now. Committed revision 1068368. Lets Add Widgets :) - Shantanu ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/2467/#review3545 ----------------------------------------------------------- On 2009-12-31 04:28:54, Shantanu Tushar Jha wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviewboard.kde.org/r/2467/ > ----------------------------------------------------------- > > (Updated 2009-12-31 04:28:54) > > > Review request for Plasma. > > > Summary > ------- > > Problem- As of now, if the Add Widgets option in the Panel context menu is > clicked, nothing happens. > Reason - The signal is connected to PanelController, which is only shown when > the panel toolbox is open. > This solution - Make the PanelView class handle the signal if the toolbox is > not open. > > A small problem - As I've used the method which is called for the Add Widgets > action in DesktopView, the problem with this is that the Widgets explorer is > shown at the bottom regardless of which panel the context menu belongs to > (see the screenshot). It is workable, but obviously doesn't feel good. > One solution is to make PanelView show the PanelController (toolbox) if the > Add Widgets menu item was selected and then call a public function in > PanelController to show the Widgets Explorer. Is it acceptable? If not, any > suggestions? > > > This addresses bug 217034. > https://bugs.kde.org/show_bug.cgi?id=217034 > > > Diffs > ----- > > trunk/KDE/kdebase/workspace/plasma/desktop/shell/panelview.h 1067646 > trunk/KDE/kdebase/workspace/plasma/desktop/shell/panelview.cpp 1067646 > > Diff: http://reviewboard.kde.org/r/2467/diff > > > Testing > ------- > > Works on trunk build, except the minor issue as described above. > > > Screenshots > ----------- > > Add Widget clicked on upper panel > http://reviewboard.kde.org/r/2467/s/284/ > After updating fromSVN > http://reviewboard.kde.org/r/2467/s/285/ > > > Thanks, > > Shantanu > > _______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
