[Marble-devel] Review Request: Measure Tool / MarbleWidgetPopupMenu Refactoring

[Dennis Nienhüser]

> On 2010-04-28 01:09:15, jmho wrote:
> > After a little bit more testing, it looks that "Remove Measure Points"
> > does not seem to work any longer. It only removes the last
> > measure point.

I fixed that locally, but apparently not in the branch I generated the patch from. The signal is connected to the wrong slot.


- Dennis


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


On 2010-04-25 08:49:09, Dennis Nienhüser wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/3806/
> -----------------------------------------------------------
> 
> (Updated 2010-04-25 08:49:09)
> 
> 
> Review request for marble.
> 
> 
> Summary
> -------
> 
> I'd like to re-use MarbleWidgetPopupMenu for routing to have a consistent base context menu. Currently the measure tool is coupled tightly to the standard context menu (MarbleWidgetPopupMenu). This is inconvenient as the measure actions are disturbing for a routing context menu.
> 
> The patch moves the measure tool related stuff to MarbleWidgetDefaultInputHandler and provides a little hook in MarbleWidgetPopupMenu to add custom actions. We probably need to extend that in the future, but adding it now shouldn't hurt as MarbleWidgetPopupMenu is not part of the public library.
> 
> In the long term we should make the measure tool a plugin and give plugins the ability to extend the menus. This requires changes to the public API which I think is a bit late for 4.5. With this patch, however, a routing context menu can already be integrated neatly.
> 
> 
> Diffs
> -----
> 
>   /trunk/KDE/kdeedu/marble/src/lib/MarbleWidgetInputHandler.h 1118077 
>   /trunk/KDE/kdeedu/marble/src/lib/MarbleWidgetInputHandler.cpp 1118077 
>   /trunk/KDE/kdeedu/marble/src/lib/MarbleWidgetPopupMenu.h 1118077 
>   /trunk/KDE/kdeedu/marble/src/lib/MarbleWidgetPopupMenu.cpp 1118077 
> 
> Diff: http://reviewboard.kde.org/r/3806/diff
> 
> 
> Testing
> -------
> 
> Measure tool behaves the same, routing context menu using MarbleWidgetPopupMenu works without any measure actions in it (latter is not part of this patch).
> 
> 
> Thanks,
> 
> Dennis
> 
>