[Marble-devel] Review Request 122849: Implemented the scale feature (for icons and labels) when adding placemarks via the Edit Mode

[Marius Stanciu]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/122849/
-----------------------------------------------------------

(Updated April 11, 2015, 12:37 p.m.)


Status
------

This change has been marked as submitted.


Review request for Marble.


Changes
-------

Submitted with commit 82bc44cc536080d218e3ac40da0354efd8aac2fd by Torsten Rahn on behalf of Marius Stanciu to branch master.


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
-----

  src/lib/marble/EditPlacemarkDialog.ui a1ba027 
  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/20150411/1b74d50d/attachment.html>