[Okular-devel] Review Request 114060: Proposed viewport transition refinements for Find and Undo/Redo actions

[Jon Mease]

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

(Updated Nov. 28, 2013, 12:47 a.m.)


Review request for Okular.


Changes
-------

Updates based on Albert's feedback


Repository: okular


Description
-------

This patch introduces viewport transitions for undo/redo actions on annotations and forms.  When an annotation/form action is undone/redone but the associated annotation/form is not currently visible, the viewport is updated to center on the undo/redo action. If the annotation/form is visible, the viewport is not updated.

The viewport transitions for the Find action have also been updated to this same algorithm.  Previously the viewport was moved to center on each matching search term even if the search term was already visible in the viewport. This lead to unnecessary viewport transitions if the search term matched several items in a single paragraph for example.

These proposed changes to the viewport transition behavior are consistent with the find and undo behavior of many existing applications including Kate, Open Office, and Foxit PDF Reader.


Diffs (updated)
-----

  core/document.h fe296e0 
  core/document.cpp 265ee09 
  core/document_p.h 3a257de 
  core/documentcommands.cpp 7799bb0 
  core/documentcommands_p.h fe1c577 
  core/page.cpp 0bafa99 
  core/utils.cpp 5dd8448 
  core/utils_p.h df82fe1 

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


Testing
-------

Manual testing of the viewport behavior for find and undo/redo actions on several documents.  I also tested that the desired behavior is maintained when documents are rotated.


Thanks,

Jon Mease

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