[Marble-devel] Review Request 124511: Merged the decoration(124154) and the label placement(124498) review requests.

Dávid Kolozsvári freedawson at gmail.com
Thu Aug 13 00:13:27 UTC 2015


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

(Updated Aug. 13, 2015, 12:13 a.m.)


Review request for Marble.


Changes
-------

Updated with Dennis Nienhüser's more accurate offset calculation for the faked 3D of buildings. This is used only when the maximum offset of the "shadow"/"walls" is more than 5 pixels.
Decorations are now visible in LowMapQuality - this is the base quality of animations - so they are seen while panning, zooming.


Repository: marble


Description
-------

I changed a little bit the decoration creating method, it now uses a QList to store the decorations, so multiple decorations can be added this way. It was an idea for the street labeling, but it makes sense without that too.


Diffs (updated)
-----

  src/lib/marble/ClipPainter.h 42b73c0 
  src/lib/marble/GeoPainter.h 7a757b9 
  src/lib/marble/GeoPainter.cpp d04138c 
  src/lib/marble/GeoPainter_p.h f0c4f9b 
  src/lib/marble/MarbleGlobal.h cf2768f 
  src/lib/marble/geodata/data/GeoDataFeature.h ea23cd8 
  src/lib/marble/geodata/data/GeoDataFeature.cpp 6f330fb 
  src/lib/marble/geodata/data/GeoDataFeature_p.h 496c356 
  src/lib/marble/geodata/graphicsitem/GeoLineStringGraphicsItem.h 4842809 
  src/lib/marble/geodata/graphicsitem/GeoLineStringGraphicsItem.cpp 4320c07 
  src/lib/marble/geodata/graphicsitem/GeoPolygonGraphicsItem.h f469dfb 
  src/lib/marble/geodata/graphicsitem/GeoPolygonGraphicsItem.cpp 81cfe9a 
  src/lib/marble/graphicsview/GeoGraphicsItem.h 4ca4727 
  src/lib/marble/graphicsview/GeoGraphicsItem.cpp b8fa693 
  src/lib/marble/graphicsview/GeoGraphicsItem_p.h 01becfc 
  src/lib/marble/layers/GeometryLayer.cpp 9eb3f50 

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


Testing
-------

It works on a freshly pulled version of Marble.


Thanks,

Dávid Kolozsvári

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


More information about the Marble-devel mailing list