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

Nate Graham <pointedst...@zoho.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|REOPENED                    |RESOLVED
         Resolution|---                         |FIXED
      Latest Commit|                            |https://commits.kde.org/gwe
                   |                            |nview/b09b216fe7b404dfc12f7
                   |                            |8834c38f4a0793a5d4d

--- Comment #6 from Nate Graham <pointedst...@zoho.com> ---
Git commit b09b216fe7b404dfc12f78834c38f4a0793a5d4d by Nathaniel Graham.
Committed on 27/09/2017 at 18:38.
Pushed by ngraham into branch 'Applications/17.08'.

Use standard QToolButtons so that their icons use the right colors

Summary:

For the inline context buttons, use the standard QToolButton widget which
respects theming so that icons are always visible

Test Plan:
Tested in KDE Neon with Breeze light and dark, Oxygen, and all default color
schemes. Big improvement.

Current with Breeze:
{F4047019}

Patched version with Breeze:
{F4103061}

Patched version with Breeze Dark:
{F4103078}

Reviewers: #kde_applications, vtasoulas, broulik, rkflx

Reviewed By: rkflx

Subscribers: mart, rkflx

Differential Revision: https://phabricator.kde.org/D7988

M  +0    -2    app/org.kde.gwenview.appdata.xml
M  +11   -0    lib/thumbnailview/contextbarbutton.cpp
M  +11   -0    lib/thumbnailview/contextbarbutton.h
M  +11   -6    lib/thumbnailview/previewitemdelegate.cpp
M  +5    -4    lib/thumbnailview/thumbnailbarview.cpp

https://commits.kde.org/gwenview/b09b216fe7b404dfc12f78834c38f4a0793a5d4d

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to