kossebau created this revision.
kossebau added reviewers: Marble, nienhueser, rahn, shentey.

REVISION SUMMARY
  GeoPainter API had a mixture of int and qreal-based methods, and internally
  was inconsistently relaying to int or qreal-based QPainter methods.
  
  Regions used for painting rects and ellipses depend on whether anti-aliasing
  is enabled or not, and can result in offsets of at least 1 pixel.
  So calculations now tries to respect that.
  
  GeoPainter::regionFromRect() was used for getting the region of pixmaps
  being rendered. Especially in non-anti-aliased mode though this does not
  match, also is there no stroke-width on painting pixmaps.
  A special GeoPainter::regionFromPixmapRect() should serve that purpose now.

REPOSITORY
  rMARBLE Marble

BRANCH
  improveGeoPainterAPI

REVISION DETAIL
  https://phabricator.kde.org/D2279

AFFECTED FILES
  src/bindings/python/sip/GeoPainter.sip
  src/lib/marble/GeoPainter.cpp
  src/lib/marble/GeoPainter.h
  src/lib/marble/routing/RoutingLayer.cpp

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: kossebau, #marble, nienhueser, rahn, shentey
Cc: marble-devel
_______________________________________________
Marble-devel mailing list
[email protected]
https://mail.kde.org/mailman/listinfo/marble-devel

Reply via email to