----------------------------------------------------------- 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 > >
_______________________________________________ Marble-devel mailing list [email protected] https://mail.kde.org/mailman/listinfo/marble-devel
