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

[Torsten Rahn]

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

Ship it!


It's a bit unfortunate that this makes the RMB menu even longer (I think it's at the maximum already). I wonder whether we should trade this entry for the "Refresh Map" entry (which is already accessable via F5 / the menu and which should be quite rarely used in practice).

- Torsten


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/20110320/23d183f4/attachment.htm