[Marble-devel] Review Request 122849: Implemented the scale feature (for icons and labels) when adding placemarks via the Edit Mode
Marius Stanciu
stanciumarius94 at gmail.com
Wed Mar 25 21:42:17 UTC 2015
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/122849/
-----------------------------------------------------------
(Updated March 25, 2015, 9:42 p.m.)
Review request for Marble.
Changes
-------
Refactored the code to a much better location and fixed the centering issues
Repository: marble
Description
-------
The feature was disabled(greyed out) and it didn't work. Now drawing the placemark keeps account of the scale member.
I have one issue though:
I didn't really know where to place a constant for a maximum icon size( limiting the user ) so I could use your opinion.
Diffs (updated)
-----
src/lib/marble/EditPlacemarkDialog.ui e17bf14
src/lib/marble/PlacemarkLayout.cpp 6c0af30
src/lib/marble/VisiblePlacemark.cpp d57dcb0
src/lib/marble/geodata/data/GeoDataIconStyle.h 4afb115
src/lib/marble/geodata/data/GeoDataIconStyle.cpp 5385cd1
src/lib/marble/geodata/data/GeoDataLabelStyle.h 046f7bc
src/lib/marble/geodata/data/GeoDataLabelStyle.cpp 1ec0c78
Diff: https://git.reviewboard.kde.org/r/122849/diff/
Testing
-------
works for me.
Thanks,
Marius Stanciu
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/marble-devel/attachments/20150325/0cc70944/attachment.html>
More information about the Marble-devel
mailing list