Re: [Okular-devel] Review Request 109632: Annotation eraser

2014-04-23 Thread Peter Grasch
can resume the discussion about possible improvements here. Thanks, Peter Grasch ___ Okular-devel mailing list Okular-devel@kde.org https://mail.kde.org/mailman/listinfo/okular-devel

Re: [Okular-devel] Review Request 109633: Use crosshair cursor while annotating

2013-06-17 Thread Peter Grasch
://git.reviewboard.kde.org/r/109633/diff/ Testing --- Tested successfully: Tool selection, deselection, continuous mode. Thanks, Peter Grasch ___ Okular-devel mailing list Okular-devel@kde.org https://mail.kde.org/mailman/listinfo/okular-devel

Re: [Okular-devel] Review Request 109627: Outline based selection for annotation elements

2013-06-17 Thread Peter Grasch
tested the annotation objects above and a couple of special cases mentioned in the IRC. Thanks, Peter Grasch ___ Okular-devel mailing list Okular-devel@kde.org https://mail.kde.org/mailman/listinfo/okular-devel

Re: [Okular-devel] Review Request 109627: Outline based selection for annotation elements

2013-06-17 Thread Peter Grasch
the annotation objects above and a couple of special cases mentioned in the IRC. Thanks, Peter Grasch ___ Okular-devel mailing list Okular-devel@kde.org https://mail.kde.org/mailman/listinfo/okular-devel

Re: [Okular-devel] Review Request 109627: Outline based selection for annotation elements

2013-06-17 Thread Peter Grasch
On March 21, 2013, 11:22 p.m., Albert Astals Cid wrote: core/area.h, line 28 http://git.reviewboard.kde.org/r/109627/diff/1/?file=120762#file120762line28 Don't think this belongs to area.h to be honest Peter Grasch wrote: Would you prefer something like this: bool

Re: [Okular-devel] Review Request 109627: Outline based selection for annotation elements

2013-04-04 Thread Peter Grasch
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/109627/#review29666 --- On March 23, 2013, 7:52 p.m., Peter Grasch wrote

Re: [Okular-devel] Review Request 109632: Annotation eraser

2013-03-24 Thread Peter Grasch
--- On March 21, 2013, 1:26 a.m., Peter Grasch wrote: --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/109632

Re: [Okular-devel] Review Request 109625: Do not re-show tool name in continuous annotation mode

2013-03-23 Thread Peter Grasch
- ui/pageviewannotator.cpp 7bd7496 Diff: http://git.reviewboard.kde.org/r/109625/diff/ Testing --- Local tests show expected behavior. Thanks, Peter Grasch ___ Okular-devel mailing list Okular-devel@kde.org https://mail.kde.org/mailman

Re: [Okular-devel] Review Request 109633: Use crosshair cursor while annotating

2013-03-23 Thread Peter Grasch
ui/pageviewannotator.cpp 4615d1c Diff: http://git.reviewboard.kde.org/r/109633/diff/ Testing --- Tested successfully: Tool selection, deselection, continuous mode. Thanks, Peter Grasch ___ Okular-devel mailing list Okular-devel@kde.org

Re: [Okular-devel] Review Request 109633: Use crosshair cursor while annotating

2013-03-23 Thread Peter Grasch
, deselection, continuous mode. Thanks, Peter Grasch ___ Okular-devel mailing list Okular-devel@kde.org https://mail.kde.org/mailman/listinfo/okular-devel

Re: [Okular-devel] Review Request 109632: Annotation eraser

2013-03-23 Thread Peter Grasch
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/109632/#review29738 --- On March 21, 2013, 1:26 a.m., Peter Grasch wrote

Re: [Okular-devel] Review Request 109632: Annotation eraser

2013-03-22 Thread Peter Grasch
--- On March 21, 2013, 1:26 a.m., Peter Grasch wrote: --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/109632

[Okular-devel] Review Request 109627: Outline based selection for annotation elements

2013-03-21 Thread Peter Grasch
1db2763 Diff: http://git.reviewboard.kde.org/r/109627/diff/ Testing --- I tested the annotation objects above and a couple of special cases mentioned in the IRC. Thanks, Peter Grasch ___ Okular-devel mailing list Okular-devel@kde.org https

[Okular-devel] Review Request 109632: Annotation eraser

2013-03-21 Thread Peter Grasch
. I hope we can resume the discussion about possible improvements here. Thanks, Peter Grasch ___ Okular-devel mailing list Okular-devel@kde.org https://mail.kde.org/mailman/listinfo/okular-devel

[Okular-devel] Review Request 109633: Use crosshair cursor while annotating

2013-03-21 Thread Peter Grasch
this given Okulars architecture. Diffs - ui/pageviewannotator.cpp 7bd7496 ui/pageviewannotator.h 850d887 ui/pageview.cpp e8d481d Diff: http://git.reviewboard.kde.org/r/109633/diff/ Testing --- Tested successfully: Tool selection, deselection, continuous mode. Thanks, Peter

Re: [Okular-devel] Review Request 109632: Annotation eraser

2013-03-20 Thread Peter Grasch
. Thanks, Peter Grasch ___ Okular-devel mailing list Okular-devel@kde.org https://mail.kde.org/mailman/listinfo/okular-devel

[Okular-devel] [Bug 201108] thumbnail sidebar always gets shown; no way to disable

2012-05-22 Thread Peter Grasch
https://bugs.kde.org/show_bug.cgi?id=201108 Peter Grasch gra...@simon-listens.org changed: What|Removed |Added CC||gra...@simon-listens.org