----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119783/ -----------------------------------------------------------
(Updated Aug. 17, 2014, 3:32 p.m.) Review request for Marble. Changes ------- Fix name. Ooops, sorry. Repository: marble Description (updated) ------- This patch introduces the concept of Focus Item to Annotate Plugin which means that in order to execute some actions (or to simply modify an item) the user has to first mark the item as 'selected'. Torsten mentioned in Randa this feature and I thought it would be really nice to have something like this. I also reordered the actions within the View menu (or within the Toolbar in marble-qt) to the same order they will be when we will move them to a separate dockwidget or something similar. So there are first the buttons for saving/loading annotation files. Then there are the actions which, when triggered, lead to adding the specified graphic item (notice that I modified a little bit the way polygons are being added, so that it is consistent with paths and placemarks - now the dialog pops-up immediately when pressing 'Add Polygon'). Then, there are the actions which get activated when an item has the focus, depending on the item's type (for example, when a polygon has the focus, all four - Add Polygon Hole, Merge Nodes, Add Nodes, Remove Item get enabled while when a placemark has the focus, only Remove Item is enabled). Notice that I changed the significance of 'Remove Item' action: it used to enter a state on which every clicked item was being removed; now, it has the signifi cance of removing the focus item, which I find more intuitive and useful. Also, when the edit dialog of a graphic item is being shown, all actions are disabled to avoid opening multiple dialogs and things like that. Also, in the same situation, right clicking an item won't work - otherwise, one could remove the item while the dialog is being shown and the dialog won't get destroyed which would lead to crashes. One problem (in fact it is not a problem, but a feature which is missing) is that currently we don't have some visual effect when a placemark (text annotation) has the focus (you can only see that the 'Remove Item' within the toolbar gets enabled at the moment); for polygons/polylines - we show the nodes when the item has the focus. So I'm waiting suggestions on what should happen when a placemark has the focus. Next I will work on optimizations, solving some issues (like the icons for placemarks, saving osm files, deep-copying geometries, etc) and writing documentation. Later Edit: Because of this (http://commits.kde.org/marble/ccc56121d9de511f2c3a906e73eb20c6439abaf9) commit made by Bernhard after I made this review request, this patch will not apply anymore. At the same time, this patch includes the changes made by his commit, so if you don't mind, please run a `git checkout 775ac32e` before applying the patch. Diffs ----- src/plugins/render/annotate/SceneGraphicsItem.h 9e5b827 src/plugins/render/annotate/SceneGraphicsItem.cpp 6a7dec5 src/plugins/render/annotate/PolylineAnnotation.h f80c11d src/plugins/render/annotate/PolylineAnnotation.cpp 9b4db5a src/plugins/render/annotate/EditPolylineDialog.h 1e894a7 src/plugins/render/annotate/EditPolylineDialog.cpp 7a78e57 src/plugins/render/annotate/EditTextAnnotationDialog.cpp 02c8014 src/plugins/render/annotate/PlacemarkTextAnnotation.cpp 3c6a183 src/plugins/render/annotate/AnnotatePlugin.h 771f8aa src/plugins/render/annotate/AnnotatePlugin.cpp 2bfbc8d src/plugins/render/annotate/AreaAnnotation.h 8a33881 src/plugins/render/annotate/AreaAnnotation.cpp a50b546 src/plugins/render/annotate/EditPolygonDialog.h 0df843a src/plugins/render/annotate/EditPolygonDialog.cpp 522a71c src/plugins/render/annotate/EditPolygonDialog.ui 3cd9f3d Diff: https://git.reviewboard.kde.org/r/119783/diff/ Testing ------- Thanks, Cruceru Calin-Cristian
_______________________________________________ Marble-devel mailing list [email protected] https://mail.kde.org/mailman/listinfo/marble-devel
