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

[Albert Astals Cid]

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



core/document.h
<http://git.reviewboard.kde.org/r/114060/#comment31861>

    const & for NormalizedRect?
    
    Also since this is just used by core/* stufff do we really to make it public? Can it go to some _p.h?



core/document.cpp
<http://git.reviewboard.kde.org/r/114060/#comment31862>

    const



core/document.cpp
<http://git.reviewboard.kde.org/r/114060/#comment31863>

    const



core/documentcommands.cpp
<http://git.reviewboard.kde.org/r/114060/#comment31865>

    const &



core/documentcommands.cpp
<http://git.reviewboard.kde.org/r/114060/#comment31866>

    const &



core/utils.h
<http://git.reviewboard.kde.org/r/114060/#comment31867>

    Same question as before, since this method seems to be used only in core/* can you see if you can put it in some _p.h so we don't expose it to the rest of the world?


- Albert Astals Cid


On Nov. 24, 2013, 2:23 a.m., Jon Mease wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/114060/
> -----------------------------------------------------------
> 
> (Updated Nov. 24, 2013, 2:23 a.m.)
> 
> 
> Review request for Okular.
> 
> 
> 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
> -----
> 
>   core/document.h fe296e0 
>   core/document.cpp 265ee09 
>   core/documentcommands.cpp 7799bb0 
>   core/documentcommands_p.h fe1c577 
>   core/page.cpp 0bafa99 
>   core/utils.h 8d5d5fc 
>   core/utils.cpp 5dd8448 
> 
> 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/20131127/f6dab6bf/attachment-0001.html>