[Marble-devel] Re: Review Request: "Info Boxes" submenu for the context menu

[Thibaut Gridel]

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

Ship it!


It's the only viable way to let plugin users configure the float items.
I see the View->Infoboxes menu in the menu bar, any chance to have those factored?

- Thibaut


On Feb. 20, 2011, 2:58 p.m., Dennis Nienhüser wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://svn.reviewboard.kde.org/r/6528/
> -----------------------------------------------------------
> 
> (Updated Feb. 20, 2011, 2:58 p.m.)
> 
> 
> Review request for marble.
> 
> 
> Summary
> -------
> 
> Marble 1.1 adds the ability to configure, lock and hide info boxes via their context menu. When users hide an info box this way in applications embedding Marble (e.g. digikam) or on the N900, there is currently no way to get the info box back because the main menu with the needed actions does not exist. The patch adds a submenu to the context menu (RMB) of MarbleWidget that can be used to toggle info boxes and therefore also show previously hidden ones.
> 
> 
> Diffs
> -----
> 
>   trunk/KDE/kdeedu/marble/src/lib/MarbleWidgetPopupMenu.h 1221765 
>   trunk/KDE/kdeedu/marble/src/lib/MarbleWidgetPopupMenu.cpp 1221765 
> 
> Diff: http://svn.reviewboard.kde.org/r/6528/diff
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Dennis
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/marble-devel/attachments/20110312/ced729d4/attachment.htm