[KDE Usability] Review Request 125094: Adding icon to show hidden files

[Emmanuel Pescosta]

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

Ship it!


Looks good to me :)
Thanks!


src/views/dolphinviewactionhandler.cpp (lines 484 - 489)
<https://git.reviewboard.kde.org/r/125094/#comment61689>

    Please make use of the ?: operator
    
    shown ? "visibility" : "hint"


- Emmanuel Pescosta


On Dec. 22, 2015, 4:38 p.m., Roman Gilg wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/125094/
> -----------------------------------------------------------
> 
> (Updated Dec. 22, 2015, 4:38 p.m.)
> 
> 
> Review request for Dolphin and KDE Usability.
> 
> 
> Repository: dolphin
> 
> 
> Description
> -------
> 
> Icon based on 'edit-copy.svg'. Sorry, didn't know how to add the additionally breeze repository for the svg-files.
> First review request. Be nice please. :)
> 
> 
> Diffs
> -----
> 
>   src/views/dolphinviewactionhandler.cpp ac538a6 
> 
> Diff: https://git.reviewboard.kde.org/r/125094/diff/
> 
> 
> Testing
> -------
> 
> Tested on Arch, Plasma 5.4.
> 
> 
> File Attachments
> ----------------
> 
> hidden-files.svg (16, breeze)
>   https://git.reviewboard.kde.org/media/uploaded/files/2015/09/07/b35b6f89-4d89-4434-add7-b38ed2ae0aa1__hidden-files.svg
> hidden-files.svg (22, breeze)
>   https://git.reviewboard.kde.org/media/uploaded/files/2015/09/07/a15289c8-9721-40ce-b5ce-0047d5877e08__hidden-files.svg
> hidden-files.svg (16, breeze-dark)
>   https://git.reviewboard.kde.org/media/uploaded/files/2015/09/07/8d4c2940-1b3f-43b1-893e-09f49cb6f4aa__hidden-files.svg
> hidden-files.svg (22, breeze-dark)
>   https://git.reviewboard.kde.org/media/uploaded/files/2015/09/07/ce296c59-141a-4b50-90bf-d2a1c6e215d0__hidden-files.svg
> Demonstration
>   https://git.reviewboard.kde.org/media/uploaded/files/2015/09/07/c731d844-9525-415d-9226-743fea6f5727__change.png
> Revision 3
>   https://git.reviewboard.kde.org/media/uploaded/files/2015/12/22/e1b5f750-842f-4aef-b4be-5e96ad3c629f__change2.png
> 
> 
> Thanks,
> 
> Roman Gilg
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20151223/e94cfdcb/attachment.htm>
-------------- next part --------------
_______________________________________________
kde-usability mailing list
kde-usability at kde.org
https://mail.kde.org/mailman/listinfo/kde-usability