[okular] [Bug 381629] add arrow's option to annotation tools

2019-06-01 Thread Tobias Deiminger
https://bugs.kde.org/show_bug.cgi?id=381629

Tobias Deiminger  changed:

   What|Removed |Added

 Resolution|--- |FIXED
 Status|CONFIRMED   |RESOLVED

--- Comment #6 from Tobias Deiminger  ---
Resolved by adding arrow option to straight line tool. Will be released with
KDE Applications 19.08.0.

-- 
You are receiving this mail because:
You are watching all bug changes.

[okular] [Bug 381629] add arrow's option to annotation tools

2019-06-01 Thread Tobias Deiminger
https://bugs.kde.org/show_bug.cgi?id=381629

--- Comment #5 from Tobias Deiminger  ---
Git commit d0e3a77de2c3b219c94b37a0a07953635840cd7f by Tobias Deiminger.
Committed on 01/06/2019 at 15:28.
Pushed by tobiasdeiminger into branch 'master'.

Add line annotation ending arrows for non PDF documents

Summary:
This implements drawing the various line ending styles for 2 point lines in
non-PDF documents.

Looks like this:
{F6833437}

Test Plan:
- open a *.txt document
- draw line annotations with different arrow styles: Square, Diamond,
OpenArrow, ClosedArrow, ROpenArrow, RClosedArrow, Butt, Slash, Circle
- ... as start and as end style (start needs D21238 to be configurable in GUI)
- ... filled or not (fill color can't be configured in GUI yet)
- ... with different leader line settings
- ... at various angles
- rotate and scale page
- leader line with setting 100 gives 100 pixel line at 100% zoom (use kruler to
verify)

Reviewers: #okular, aacid, sander

Reviewed By: sander

Subscribers: ngraham, sander, knambiar, okular-devel

Tags: #okular

Differential Revision: https://phabricator.kde.org/D21248

M  +0-4ui/annotationwidgets.cpp
M  +295  -70   ui/pagepainter.cpp
M  +51   -0ui/pagepainter.h

https://commits.kde.org/okular/d0e3a77de2c3b219c94b37a0a07953635840cd7f

-- 
You are receiving this mail because:
You are watching all bug changes.

[okular] [Bug 381629] add arrow's option to annotation tools

2019-05-26 Thread Tobias Deiminger
https://bugs.kde.org/show_bug.cgi?id=381629

--- Comment #4 from Tobias Deiminger  ---
Git commit b1c30cd0edd3540065b010730eccbb91a75faff4 by Tobias Deiminger, on
behalf of Rajeesh K Nambiar.
Committed on 26/05/2019 at 08:15.
Pushed by tobiasdeiminger into branch 'master'.

Okular Annotation: add line start/end style config only for Straight Line tool

Summary:
“Inner Color” configuration of Polygon tool was overlapping with the line
start/end styles intended for only Straight Line tool. Fix it.

Test Plan:
1. Configure annotations
2. Create/Edit Polygon tool
3. Observe that no Line Start/End styles are visible
4. Create/Edit Straight Line tool
5. Observe that line start/end styles can be configured

Reviewers: #okular, tobiasdeiminger

Reviewed By: tobiasdeiminger

Subscribers: tobiasdeiminger, okular-devel

Tags: #okular

Differential Revision: https://phabricator.kde.org/D21332

M  +45   -43   ui/annotationwidgets.cpp
M  +7-7ui/annotationwidgets.h

https://commits.kde.org/okular/b1c30cd0edd3540065b010730eccbb91a75faff4

-- 
You are receiving this mail because:
You are watching all bug changes.

[okular] [Bug 381629] add arrow's option to annotation tools

2018-11-06 Thread Tobias Deiminger
https://bugs.kde.org/show_bug.cgi?id=381629

--- Comment #3 from Tobias Deiminger  ---
FYI: https://gitlab.freedesktop.org/poppler/poppler/merge_requests/106

-- 
You are receiving this mail because:
You are watching all bug changes.

[okular] [Bug 381629] add arrow's option to annotation tools

2018-11-06 Thread Simone Gaiarin
https://bugs.kde.org/show_bug.cgi?id=381629

Simone Gaiarin  changed:

   What|Removed |Added

 CC||simg...@gmail.com
 Ever confirmed|0   |1
 Status|REPORTED|CONFIRMED

-- 
You are receiving this mail because:
You are watching all bug changes.

[okular] [Bug 381629] add arrow's option to annotation tools

2018-11-01 Thread Tobias Deiminger
https://bugs.kde.org/show_bug.cgi?id=381629

Tobias Deiminger  changed:

   What|Removed |Added

 CC||haxti...@posteo.de

--- Comment #2 from Tobias Deiminger  ---
Certainly a useful feature. Needs person power, and might be a student project
(?).

Entry point would be to read the PDF reference, which defines "OpenArrow" and
"ClosedArrow" for Line and Polyline Annotations (annotations in Okular closely
follow the PDF standard, even for non-PDF documents). 

For non-PDF docs, drawing those line endings would have to be implemented in
ui/pagepainter.cpp, PagePainter::paintCroppedPageOnPainter, "if ( type ==
Okular::Annotation::ALine )".

For PDF documents, drawing operations had to go into the poppler library,
poppler/Annot.cc, AnnotLine::generateLineAppearance and AnnotPolygon::draw.
Popplers core and Qt5 API is already prepared to take line ending settings. But
eventually there's no life but a "// TODO: Line endings" in the code.

Of course we would also have to implement all the surrounding UI dialogs,
config settings and document archive entries.

-- 
You are receiving this mail because:
You are watching all bug changes.

[okular] [Bug 381629] add arrow's option to annotation tools

2018-10-31 Thread azzamsa
https://bugs.kde.org/show_bug.cgi?id=381629

azzamsa  changed:

   What|Removed |Added

 CC||oktinast...@gmail.com

--- Comment #1 from azzamsa  ---
Come here for requesting this feature too. Hope it will be implemented soon.

-- 
You are receiving this mail because:
You are watching all bug changes.