D22297: Add id="current-color-scheme" to collapse-all icons

2019-07-06 Thread N. Higa
nhiga created this revision. nhiga added a reviewer: VDG. nhiga added a project: Frameworks. nhiga requested review of this revision. REVISION SUMMARY BUG: 409546 REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D22297 AFFECTED FILES icons-dark/actions/16/collaps

D22297: Add id="current-color-scheme" to collapse-all icons

2019-07-06 Thread N. Higa
nhiga added a comment. Summary will be filled in later. Related: D19575 , Bug 409546 REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D22297 To: nhiga, #vdg Cc: kde-frameworks-

D22297: Add id="current-color-scheme" to collapse-all icons

2019-07-06 Thread N. Higa
nhiga added a reviewer: davidhurka. nhiga added a subscriber: davidhurka. nhiga added a comment. @davidhurka Would you please try this diff and see if it resolves the bug? REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D22297 To: nhiga, #vdg, davidhurka Cc:

D22297: Add id="current-color-scheme" to collapse-all icons

2019-07-06 Thread David Hurka
davidhurka accepted this revision. davidhurka added a comment. This revision is now accepted and ready to land. Yes, problem solved. The icon is now visible with a dark color theme. Did not check anything else, someone with more knowledge about icons should do that. REPOSITORY R266 Bre

D22297: Add id="current-color-scheme" to collapse-all icons

2019-07-06 Thread N. Higa
nhiga edited the summary of this revision. REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D22297 To: nhiga, #vdg, davidhurka Cc: davidhurka, kde-frameworks-devel, LeGast00n, michaelh, ngraham, bruns

D22297: Add id="current-color-scheme" to collapse-all icons

2019-07-06 Thread David Hurka
davidhurka added a comment. (Added screenshots) REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D22297 To: nhiga, #vdg, davidhurka Cc: davidhurka, kde-frameworks-devel, LeGast00n, michaelh, ngraham, bruns

D22297: Add id="current-color-scheme" to collapse-all icons

2019-07-06 Thread David Hurka
davidhurka edited the summary of this revision. REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D22297 To: nhiga, #vdg, davidhurka Cc: davidhurka, kde-frameworks-devel, LeGast00n, michaelh, ngraham, bruns

D22297: Add id="current-color-scheme" to collapse-all icons

2019-07-06 Thread Noah Davis
ndavis accepted this revision. ndavis added a comment. LGTM REPOSITORY R266 Breeze Icons REVISION DETAIL https://phabricator.kde.org/D22297 To: nhiga, #vdg, davidhurka, ndavis Cc: ndavis, davidhurka, kde-frameworks-devel, LeGast00n, michaelh, ngraham, bruns

D22297: Add id="current-color-scheme" to collapse-all icons

2019-07-07 Thread Nathaniel Graham
This revision was automatically updated to reflect the committed changes. Closed by commit R266:8ec6b1ba5670: Add id="current-color-scheme" to collapse-all icons (authored by Natsumi Higa , committed by ngraham). REPOSITORY R266 Breeze Icons CHANGES SINCE LAST