----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/114153/#review44612 -----------------------------------------------------------
Rendering differences (that I judged ugly) were the reason why I chose to go the dashed outline route. - Fabio D'Urso On Nov. 27, 2013, 3:22 p.m., Jon Mease wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/114153/ > ----------------------------------------------------------- > > (Updated Nov. 27, 2013, 3:22 p.m.) > > > Review request for Okular. > > > Repository: okular > > > Description > ------- > > Unlike in other document formats, the annotations on PDF documents are > rendered by the Poppler back-end along with the document itself. Because > this document rendering step is expensive we don't render annotations on PDF > documents while the annotations are being moved (With Ctrl+Left click drag). > Instead of rendering the annotation itself during the move, we render a > dashed outline of the annotation. For non-PDF document types the annotations > are rendered by Okular on top of the document, and so there is no large > performance penalty in rendering the annotation smoothly as it is moved. I > find the aesthetic experience of moving annotations on non-PDF to be much > more pleasing. > > In this small patch updates the paintCroppedPageOnPainter() function draw > external annotations using the internal annotation drawing logic while the > annotation is being moved. It also removes the dashed annotation outline > during the move. With this small change the experience of moving an > annotation on a PDF now matches that of moving an annotation on the other > document formats. > > Two small oddities: The rendering of the popup note icon differs between the > Poppler back-end and Okular's internal rendering so the icon changes form > while being moved and then changes back after being dropped. The rendering > of fonts on inline notes between the Poppler back-end and Okular's internal > rendering seems to differ in some cases so as you move an inline note the > font changes. > > Thoughts? > > > Diffs > ----- > > ui/pagepainter.cpp d5d9c3e > > Diff: http://git.reviewboard.kde.org/r/114153/diff/ > > > Testing > ------- > > Tested drawing and moving each of the annotation types on a PDF document and > on a DVI document. The behavior on the DVI document is unchanged. I find the > behavior on the PDF document to be more natural. > > > Thanks, > > Jon Mease > >
_______________________________________________ Okular-devel mailing list Okular-devel@kde.org https://mail.kde.org/mailman/listinfo/okular-devel
