[Konversation-devel] Review Request 128321: searchbar: Use QIcon::fromTheme to get next/previous icons

Heiko Becker heirecka at exherbo.org
Thu Jun 30 11:48:18 UTC 2016


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

(Updated June 30, 2016, 1:48 p.m.)


Status
------

This change has been marked as submitted.


Review request for Konversation.


Changes
-------

Submitted with commit 257902f50773041d5c3d232063acf7306aa6b9c7 by Heiko Becker to branch 1.6.


Bugs: 361833
    https://bugs.kde.org/show_bug.cgi?id=361833


Repository: konversation


Description
-------

The icons on the next and previous buttons in the searchbar were
missing because QIcon(QString) expects a path containing a file
name, but was given an icon name.

BUG: 361833


Diffs
-----

  src/viewer/searchbar.cpp a5f0f69 

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


Testing
-------

Icons appear on the buttons.


Thanks,

Heiko Becker

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/konversation-devel/attachments/20160630/97e32ed6/attachment-0001.html>


More information about the Konversation-devel mailing list