[Okular-devel] Review Request 109632: Annotation eraser

[Peter Grasch]

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

Review request for Okular.


Description
-------

This introduces a new annotation tool: "Eraser".

The eraser primarily erases other annotations that it comes into contact with (shapes, lines, highlights, etc.).

However, ink annotations are treated more like a real eraser: Existing paths are split and unaffected parts are preserved.
This is what it looks like: http://bedahr.org/eraser.ogv


Example tool configuration for your tools.xml (not included in patch):

<tool id="15" name="Eraser" pixmap="tool-eraser-okular">
   <tooltip>Eraser</tooltip>
   <engine type="Eraser" />
   <shortcut>7</shortcut>
</tool>


Diffs
-----

  ui/pageviewannotator.cpp 7bd7496 

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


Testing
-------

While it worked fine for the few PDFs I threw at it on this relatively powerful machine, it was pointed out to me in #okular, that calls to modifyPageAnnotation are very expensive as poppler has to re-draw the pdf (with the annotations) for every change.
I hope we can resume the discussion about possible improvements here.


Thanks,

Peter Grasch

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/okular-devel/attachments/20130321/e617ae13/attachment.html>