----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/2467/ -----------------------------------------------------------
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/ Thanks, Shantanu _______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel