----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122725/#review78430 -----------------------------------------------------------
Ship it! Ship It! - Torsten Rahn On Feb. 25, 2015, 8:51 nachm., 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 nachm.) > > > 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
