Review Request 129893: Implement continuous search

[Elvis Angelaccio]

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

(Updated March 11, 2017, 5:41 p.m.)


Review request for Okular, KDE Usability and Albert Astals Cid.


Changes
-------

Screenshots before/after to help the usability people.


Repository: okular


Description
-------

Show non-intrusive info messages whenever the search start over from the beginning or the bottom of the document, instead of asking the user if s/he wants to continue the search. This is consistent with search in KWrite/Kate and with web browsers.


Diffs
-----

  core/document.h 1fd86262 
  core/document.cpp 41b9ddfe 
  ui/searchlineedit.cpp baac8be0 

Diff: https://git.reviewboard.kde.org/r/129893/diff/


Testing
-------

Search for something in a pdf, click Next until reaching the end of document. Click again Next and the search starts over from the beginning of the document, without the "Continue from the beginning?" dialog.


File Attachments (updated)
----------------

Before: dialog asks if the search should continue from the beginning.
  https://git.reviewboard.kde.org/media/uploaded/files/2017/03/11/16eca574-0572-455d-babe-54f1087a403f__before.png
After: always continue search from beginning, with a non-intrusive notification.
  https://git.reviewboard.kde.org/media/uploaded/files/2017/03/11/23b69d9a-6fbf-45a5-9595-d355dac26042__after.png


Thanks,

Elvis Angelaccio

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