D15204: Add typewriter annotation tool
Tobias Deiminger
noreply at phabricator.kde.org
Mon Sep 24 09:26:31 BST 2018
tobiasdeiminger marked 8 inline comments as done.
tobiasdeiminger added a comment.
This revision should be quite complete now.
AnnotWindow::reloadInfo `if ( m_annot->subType() == Okular::Annotation::AText )` seems like a wrong place to do special typewriter handling (make popup dialog pastel yellow), but I had no better idea. Guess it's not worth the while to invent some derived TypewriterAnnotWindow, because typewriter should get a complete different UI as midterm goal anyway. Any suggestion?
And I've got an open question: What is the original intention of the `engine [color]` attribute from tools.xml? Enrico Ros introduced it 13 years ago, but I see no reason why we couldn't get along with solely the `annotation [color]` attributes. As far as I can tell, engine color is always in sync with annotation [color], therefore it's redundant, and it's not clear when to use what. I'm ignoring it for typewriter for now.
REPOSITORY
R223 Okular
REVISION DETAIL
https://phabricator.kde.org/D15204
To: tobiasdeiminger
Cc: sander, okular-devel, ngraham, aacid
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/okular-devel/attachments/20180924/9cd5dae0/attachment.html>
More information about the Okular-devel
mailing list