Review Request: Make KHTMLFindBar look like Okular's findbar
Sascha Peilicke
sasch.pe at gmx.de
Tue Apr 13 17:20:42 BST 2010
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/3590/
-----------------------------------------------------------
(Updated 2010-04-13 16:20:42.270303)
Review request for kdelibs and Bernhard Beschow.
Changes
-------
Added screeny
Summary
-------
Currently Konsole, Okular and Konqi have differently looking findbars. This patch aligns KHTMLFindBar to look like Okular's findbar.
This means the search input lineedit uses the maximum size and all toolbuttons are replaced with pushbuttons (they show icon and text anyway). This emphasises the search lineedit and makes the buttons look more visually pleasant IMHO. Furthermore the status label was removed, it displayed only a text which said that the search text wasn't found anymore. This info is already pleasant in the lineedit, it becomes red if nothing is found and green if there are more results available. This should be sufficient.
Diffs
-----
trunk/KDE/kdelibs/khtml/ui/findbar/khtmlfindbar.cpp 1114392
trunk/KDE/kdelibs/khtml/ui/findbar/khtmlfindbar_base.ui 1114392
Diff: http://reviewboard.kde.org/r/3590/diff
Testing
-------
tested on r1113765, works.
Screenshots
-----------
http://reviewboard.kde.org/r/3590/s/356/
Thanks,
Sascha
More information about the kde-core-devel
mailing list