Review Request: Make the Add Widgets context menu option work for Panels.

[Shantanu Tushar Jha]

-----------------------------------------------------------
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