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