[rekonq] Review Request 110130: Do not show search provider shortcuts in url list.

Maarten De Meyer de.meyer.maarten at gmail.com
Tue Apr 23 21:00:20 UTC 2013


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

(Updated April 23, 2013, 9 p.m.)


Review request for rekonq.


Changes
-------

Check for search engine first. Otherwise we might delete words that are not web shortcuts. 'testing: 123'
Use remove instead of section, the query might contain multiple delimiters.


Description
-------

This fixes a small visual regression.
Now you no longer see 'xx:' in 'searching Google for: gg: query'

I can upload a screenshot if needed.


Diffs (updated)
-----

  src/urlbar/listitem.cpp cc0bf25 

Diff: http://git.reviewboard.kde.org/r/110130/diff/


Testing
-------

compiled and run.


Thanks,

Maarten De Meyer

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/rekonq/attachments/20130423/ddecfcfa/attachment.html>


More information about the rekonq mailing list