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

Dennis Nienhüser earthwings at gentoo.org
Sun Mar 1 19:30:26 UTC 2015


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


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).

- Dennis Nienhüser


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/20150301/5ae76c78/attachment.html>


More information about the Marble-devel mailing list