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 Aug. 30, 2017, 2:47 p.m.)


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


Changes
-------

Simpler diff. It still uses EndOfDocumentReached but at least it doesn't add new values to SearchType.


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 (updated)
-----

  core/document.cpp b725ecfc7 
  ui/searchlineedit.cpp 74e6ef60c 

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
----------------

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/20170830/c41e94da/attachment.html>