Review Request: Draft: Widgets Explorer "Add Widgets"

[Anselmo Melo]

> On 2009-11-20 19:46:47, Marco Martin wrote:
> > i still think the button should be between the input field and the tabbar, to not take even more vertical space
> 
> Anselmo Melo wrote:
>     Nica. I'll change that and update the screenshot.

Hm... well, the input field and the tabbar are part of the same widget (filteringwidget). Maybe it's better place it on the left of input field to not mix "filters" with "new widget" stuff. I'm going to the airport right now, I'll try to work on it during the flight =)


- Anselmo


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/2191/#review3208
-----------------------------------------------------------


On 2009-11-20 19:41:08, Anselmo Melo wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/2191/
> -----------------------------------------------------------
> 
> (Updated 2009-11-20 19:41:08)
> 
> 
> Review request for Plasma.
> 
> 
> Summary
> -------
> 
> "Add widgets" button in the new widgets explorer. There are points that need to be defined, such as the button position / strings in the interface. In this draft, a toolbutton and a kmenu are used like in the old explorer (Variable names changed because IMHO the old ones weren't clear).
> A known issue in the interface is the menu position: Although corona->popupPosition returns a reasonable point, the menu appears on the top of the screen - probably, my fault.
> 
> Suggestions and comments are welcome =)
> 
> 
> This addresses bug 211576.
>     https://bugs.kde.org/show_bug.cgi?id=211576
> 
> 
> Diffs
> -----
> 
>   /trunk/KDE/kdebase/workspace/libs/plasmagenericshell/widgetsExplorer/widgetexplorer.cpp 1049196 
> 
> Diff: http://reviewboard.kde.org/r/2191/diff
> 
> 
> Testing
> -------
> 
> Adding widgets from GHNS works, Google Gadgets are installed but the widget list isn't updated correctly. "From file" needs more tests.
> 
> 
> Screenshots
> -----------
> 
> Ger New Widgets button
>   http://reviewboard.kde.org/r/2191/s/267/
> Menu
>   http://reviewboard.kde.org/r/2191/s/268/
> 
> 
> Thanks,
> 
> Anselmo
> 
>