> On 2009-11-17 09:01:45, Marco Martin wrote: > > ths is absolutely needed, to not face a dig regression in 4.4... can't > > really try the patch right now, can you provide a screenshot of it?
Sorry about the dalay =/The screenshots show the button and the menu added. - Anselmo ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/2191/#review3141 ----------------------------------------------------------- On 2009-11-17 05:41:34, Anselmo Melo wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviewboard.kde.org/r/2191/ > ----------------------------------------------------------- > > (Updated 2009-11-17 05:41:34) > > > 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 > > _______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
