[Marble-devel] Review Request 124663: gsoc: Fixes style referencing for new polygons and polylines in the annotation plugin

Marius Stanciu stanciumarius94 at gmail.com
Sun Aug 9 10:06:11 UTC 2015


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

(Updated Aug. 9, 2015, 10:06 a.m.)


Status
------

This change has been marked as submitted.


Review request for Marble.


Repository: marble


Description
-------

Added more complete default styles for polylines and polygons in the annotationDocuments. These weren't completely set before( only some fields for #polygon, and not at all for #polyline ).
So loading the file in another program( not marble )  might result in different rendering. ( i have searched if the kml standard offers defaults that shouldn't be mentioned; haven't found any ).

In the annotation plugin, when a new polygon(polyline) is added, setStyleUrl( "#polygon" ) is called.
Yet, in the updatePolygon() slot, that style is unnecesarely overwritten on every update( even those not related with style changes ), without changing the styleUrl; 
( when a custom style is set, the #polygon url should not be a valid reference anymore, cause the new style differs from it )


Diffs
-----

  src/lib/marble/geodata/data/GeoDataFeature.cpp a86cb66 
  src/plugins/render/annotate/AnnotatePlugin.cpp b5a816b 
  src/plugins/render/annotate/EditPolygonDialog.h 0dbb089 
  src/plugins/render/annotate/EditPolygonDialog.cpp 6f250b0 
  src/plugins/render/annotate/EditPolylineDialog.h 4d383b8 
  src/plugins/render/annotate/EditPolylineDialog.cpp fe7abc9 

Diff: https://git.reviewboard.kde.org/r/124663/diff/


Testing
-------

creating polyline/polygons---->setting styles--->exporting--->importing: results are as expected


Thanks,

Marius Stanciu

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/marble-devel/attachments/20150809/b5a8194d/attachment.html>


More information about the Marble-devel mailing list