[Marble-devel] Review Request 122725: Adds the possibility to choose between multiple formats for exporting coordinates via clipboard.

[Marius Stanciu]

> On March 1, 2015, 7:30 p.m., Dennis Nienhüser wrote:
> > One thing to keep in mind is that marble can be used as a library, where typically MarbleWidget is used to display some kind of map. In this case the context menu (including 'copy coordinates') is used as well, but the settings of the Marble application are not used/available. This should be reflected in the user interface (e.g. do not hint at Marble settings then).

Haven't thought of that. In this case, I have some more issues I can't figure out by myself and i would like your opinion:

If I don't have access to the config dialog, how would I save the copied coordinates format(and the bool option to not show again)? does marble widget library-only have any settings model or am I to create one? Or should i not allow the user to change the settings from the widget rmb menu.


- Marius


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/122725/#review76831
-----------------------------------------------------------


On Feb. 25, 2015, 8:51 p.m., Marius Stanciu wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/122725/
> -----------------------------------------------------------
> 
> (Updated Feb. 25, 2015, 8:51 p.m.)
> 
> 
> Review request for Marble.
> 
> 
> Bugs: 344237
>     http://bugs.kde.org/show_bug.cgi?id=344237
> 
> 
> Repository: marble
> 
> 
> Description
> -------
> 
> The original code on copying coordinates function didn't respect the DRY principle as it was implemented DIFFERENTLY, in two separate locations: the Edit>Copy Coordinates action (QtMainWindow.cpp), RMB->Copy Coordinates ( MarbleWidgetPopupMenu.cpp )
> 
> To solve this, I created a new specialized class CopyCoordinatesDialog.{cpp,h,ui} which handled all the formating for the coordinates.
> This class allows the user to PREVIEW and SELECT a format for exporting coordinates, all in a small window. The window pops up the first time a user tries to copy coordinates, and offers the option to not show again in the future.
> 
> It is also possible to modify the format option in the Settings dialog(as the window suggests).
> 
> The following formats have been added as an option:
>      -Degree (DMS)  (human readable)
>      -Degree (Decimal)   (human readable)
>      -Universal Transverse Mercator (UTM) (human readable)
>      -Longitude,Latitude (machine readable)
>      -Latitude,Longitude (machine readable)
>      -Geo: URI (machine readable)
>      -Geo: URI with altitude (machine readable)
>      -
> The exported object is a MimeData object composed by the following data:
>      - text ( the formated coordinates )
>      - KML representation
>      - GPX representation
> 
> 
> Diffs
> -----
> 
>   src/apps/marble-qt/QtMainWindow.h 595ffd2 
>   src/apps/marble-qt/QtMainWindow.cpp c4280c6 
>   src/lib/marble/CMakeLists.txt 6c65015 
>   src/lib/marble/CopyCoordinatesDialog.h PRE-CREATION 
>   src/lib/marble/CopyCoordinatesDialog.cpp PRE-CREATION 
>   src/lib/marble/CopyCoordinatesDialog.ui PRE-CREATION 
>   src/lib/marble/MarbleViewSettingsWidget.ui 43e153e 
>   src/lib/marble/MarbleWidget.h cd3b9bf 
>   src/lib/marble/MarbleWidget.cpp f83fcb8 
>   src/lib/marble/MarbleWidgetPopupMenu.cpp 6756f49 
>   src/lib/marble/QtMarbleConfigDialog.h 1fbee43 
>   src/lib/marble/QtMarbleConfigDialog.cpp ed5b62c 
> 
> Diff: https://git.reviewboard.kde.org/r/122725/diff/
> 
> 
> Testing
> -------
> 
> was tested on master branch @ 20 feb
> 
> 
> Thanks,
> 
> Marius Stanciu
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/marble-devel/attachments/20150302/a886d621/attachment.html>