Review Request 111233: Improve filterbar lock button

Commit Hook null at kde.org
Wed Jun 26 10:27:43 BST 2013


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

(Updated June 26, 2013, 9:27 a.m.)


Status
------

This change has been marked as submitted.


Review request for Dolphin, Aurélien Gâteau and Stuart Citrin.


Description
-------

Use the object-(un)locked icon, instead of the Lock Session icon which is really wrong for this purpose. Also make the button flat so it doesn't stick out so much. Makes it consistent with Gwenview's "Lock zoom" button.

(Wanted to create a screenshot but will not launch Dev Dolphin inside my session again as this always deletes all my Places entries)


Diffs
-----

  dolphin/src/filterbar/filterbar.cpp 3fa9cc1 

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


Testing
-------

Tested, works.


Thanks,

Kai Uwe Broulik

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20130626/791f0d3d/attachment.htm>


More information about the kfm-devel mailing list