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

[Marius Stanciu]

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

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/20150225/6e0b35b7/attachment-0001.html>