Re: Review Request 129893: Implement continuous search

2017-09-26 Thread Elvis Angelaccio
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/129893/ --- (Updated Sept. 26, 2017, 9:28 p.m.) Status -- This change has been m

Re: Review Request 129893: Implement continuous search

2017-09-26 Thread Elvis Angelaccio
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/129893/ --- (Updated Sept. 26, 2017, 8:42 p.m.) Review request for Okular, KDE Usabil

Re: Review Request 129893: Implement continuous search

2017-09-18 Thread Albert Astals Cid
> On set. 17, 2017, 10:20 a.m., Elvis Angelaccio wrote: > > Ping? Or should I re-upload the patch on phabricator for better visibility? > > Albert Astals Cid wrote: > Code is still wrong, what more visibility do you want? If you're not > going to allow for yes/no, why do you need a roundtri

Re: Review Request 129893: Implement continuous search

2017-09-18 Thread Anthony Fieroni
> On Септ. 17, 2017, 1:20 след обяд, Elvis Angelaccio wrote: > > Ping? Or should I re-upload the patch on phabricator for better visibility? > > Albert Astals Cid wrote: > Code is still wrong, what more visibility do you want? If you're not > going to allow for yes/no, why do you need a rou

Re: Review Request 129893: Implement continuous search

2017-09-17 Thread Albert Astals Cid
> On set. 17, 2017, 10:20 a.m., Elvis Angelaccio wrote: > > Ping? Or should I re-upload the patch on phabricator for better visibility? Code is still wrong, what more visibility do you want? If you're not going to allow for yes/no, why do you need a roundtrip to the ui to decide to continue th

Re: Review Request 129893: Implement continuous search

2017-09-17 Thread Elvis Angelaccio
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/129893/#review103715 --- Ping? Or should I re-upload the patch on phabricator for bet

Re: Review Request 129893: Implement continuous search

2017-08-30 Thread 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 Usabili

Re: Review Request 129893: Implement continuous search

2017-08-29 Thread Albert Astals Cid
> On feb. 2, 2017, 11:38 p.m., Albert Astals Cid wrote: > > i'm not sure i agree with this change, but this doesn't seem the proper way > > to do it, if we're going to continue searching anyways, don't send a > > Okular::Document::EndOfDocumentReached, just continue searching. > > Elvis Angela

Re: Review Request 129893: Implement continuous search

2017-08-25 Thread Elvis Angelaccio
> On Feb. 2, 2017, 11:38 p.m., Albert Astals Cid wrote: > > i'm not sure i agree with this change, but this doesn't seem the proper way > > to do it, if we're going to continue searching anyways, don't send a > > Okular::Document::EndOfDocumentReached, just continue searching. > > Elvis Angela

Re: Review Request 129893: Implement continuous search

2017-04-07 Thread Elvis Angelaccio
> On March 19, 2017, 10:30 p.m., Thomas Pfeiffer wrote: > > Great idea! > > The only feedback I have is that currently the message is visually detached > > from the search bar (which the user is likely to focus on at this point > > because that's where they've just clicked a button). > > Could

Re: Review Request 129893: Implement continuous search

2017-03-19 Thread Thomas Pfeiffer
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/129893/#review102908 --- Great idea! The only feedback I have is that currently the m

Re: Review Request 129893: Implement continuous search

2017-03-11 Thread 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 Usabil

Re: Review Request 129893: Implement continuous search

2017-03-11 Thread Albert Astals Cid
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/129893/#review102793 --- Someone from accessibility: ping? - Albert Astals Cid On

Re: Review Request 129893: Implement continuous search

2017-02-09 Thread Elvis Angelaccio
> On Feb. 2, 2017, 11:38 p.m., Albert Astals Cid wrote: > > i'm not sure i agree with this change, but this doesn't seem the proper way > > to do it, if we're going to continue searching anyways, don't send a > > Okular::Document::EndOfDocumentReached, just continue searching. > > Elvis Angela

Re: Review Request 129893: Implement continuous search

2017-02-09 Thread Elvis Angelaccio
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/129893/ --- (Updated Feb. 9, 2017, 10:08 p.m.) Review request for Okular, KDE Usabili

Re: Review Request 129893: Implement continuous search

2017-02-03 Thread Albert Astals Cid
> On Feb. 2, 2017, 11:38 p.m., Albert Astals Cid wrote: > > i'm not sure i agree with this change, but this doesn't seem the proper way > > to do it, if we're going to continue searching anyways, don't send a > > Okular::Document::EndOfDocumentReached, just continue searching. > > Elvis Angela

Re: Review Request 129893: Implement continuous search

2017-02-03 Thread Elvis Angelaccio
> On Feb. 2, 2017, 11:38 p.m., Albert Astals Cid wrote: > > i'm not sure i agree with this change, but this doesn't seem the proper way > > to do it, if we're going to continue searching anyways, don't send a > > Okular::Document::EndOfDocumentReached, just continue searching. Should we ask in

Re: Review Request 129893: Implement continuous search

2017-02-02 Thread Albert Astals Cid
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/129893/#review102350 --- i'm not sure i agree with this change, but this doesn't seem

Re: Review Request 129893: Implement continuous search

2017-02-01 Thread Elvis Angelaccio
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/129893/ --- (Updated Feb. 1, 2017, 11:20 p.m.) Review request for Okular and Albert A