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.