Review Request: Fix clipping of cosmetic pen table-borders drawn for user convenience when on screen (accuBlankBorders)

[Sebastian Sauer]

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

Review request for Calligra and Casper Boemann.


Summary
-------

The problem is that if a table-cell does not define a table-border then we are still drawing such a table border as visual helper for the user. Those visual helper borders can be clipped off in some cases (I was so far only able to reproduce the problem with one document at zoom-level 63%) cause we don't take the width of that visual helper border into account on clipping.

The attached patch solves that the easy way by just adjusting the painters clipping in the textshape to be sure we don't cutoff cosmetic pens which are exactly at the border of the shape ( x-pos=0, y-pos=0, width=outlineRect().width(), height=outlineRect.height() ) and are therefore partly (as in 1/2 of them) are drawn outside of the outlineRect. Since the cosmetic pen has always a size of 1 pixel butting the half of could mean that the pixel isn't drawn any longer (not sure there why it only happens with some zoom-levels while it works fine with others).

Now the question is if there is a better way to prevent to have that pixel cutoff. Maybe even down the road somewhere in the textlayout-library?


Diffs
-----

  plugins/textshape/TextShape.cpp 6fddca0 

Diff: http://git.reviewboard.kde.org/r/102107/diff


Testing
-------


Thanks,

Sebastian

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/calligra-devel/attachments/20110728/2261900d/attachment.htm>