[Okular-devel] Review Request 110589: Undo support for PDF forms

[Jon Mease]

> On June 3, 2013, 8:50 p.m., Albert Astals Cid wrote:
> > So it has been commited :-) Awesome :-)
> > 
> > Jon it'd be great if you could find some time to do some tests similar to the ones you did for undo/redo of annotations.

Thanks a lot Albert! :-)
Yes, I was planning on creating tests for this functionality fairly soon. I'm assuming there's no problem adding tests during the 4.11 feature freeze. Is that true?


- Jon


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


On June 3, 2013, 8:47 p.m., Jon Mease wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/110589/
> -----------------------------------------------------------
> 
> (Updated June 3, 2013, 8:47 p.m.)
> 
> 
> Review request for Okular.
> 
> 
> Description
> -------
> 
> Add undo / redo support for forms. Along with the previous annotation undo support I believe this completes the implementation of undo/redo for all document editing actions in Okular (Bug 177501). This review request corresponds to the "Undo/Redo support in PDF forms" feature in the 4.11 feature plan (http://techbase.kde.org/Schedules/KDE4/4.11_Feature_Plan)
> 
> Potential issue: If the last form or annotation that was modified is outside of the document viewport we are not currently moving the viewport to the form or annotation and so it can be unclear what has been undone.  I would appreciate suggestions on whether we should add this viewport shifting and, if so, how best to go about implementing it. 
> 
> 
> This addresses bug 177501.
>     http://bugs.kde.org/show_bug.cgi?id=177501
> 
> 
> Diffs
> -----
> 
>   core/document.h d443917 
>   core/document.cpp 2732441 
>   core/documentcommands.cpp 5fcc195 
>   core/documentcommands_p.h a9775a6 
>   ui/formwidgets.h 24108b8 
>   ui/formwidgets.cpp 57ecceb 
>   ui/pageview.h 5e839f2 
>   ui/pageview.cpp 6e093ef 
> 
> Diff: http://git.reviewboard.kde.org/r/110589/diff/
> 
> 
> Testing
> -------
> 
> Manual testing on a variety of PDFs including forms. I've attached three such documents below.
> 
> 
> File Attachments
> ----------------
> 
> Mixed forms 1
>   http://git.reviewboard.kde.org/media/uploaded/files/2013/05/22/forms-scribus.pdf
> 
>   http://git.reviewboard.kde.org/media/uploaded/files/2013/05/22/formSamples.pdf
> Exclusive checkboxes
>   http://git.reviewboard.kde.org/media/uploaded/files/2013/05/22/stripped-doc.pdf
> 
> 
> Thanks,
> 
> Jon Mease
> 
>

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