D22617: Add view-pages-* icons, as needed in Okular for page layout selection
Noah Davis
noreply at phabricator.kde.org
Fri Jul 26 14:48:31 BST 2019
ndavis accepted this revision.
ndavis added a comment.
This revision is now accepted and ready to land.
Looks ready to land!
In D22617#502629 <https://phabricator.kde.org/D22617#502629>, @davidhurka wrote:
> - Add missing .ColorScheme-Text { to 24px versions of snap-page.svg
>
> And then the class name was missing, together with the opening brace: `.ColorScheme-Text {`. This is not a comfortable workflow...
It could be a lot better, I've just gotten used to it and the confusion from learning it for the first time is gone. Ideally, we'd have a CSS file that all the SVGs link to instead of embedding stylesheets, or maybe we'd do something else that I don't know about. I tried doing the former and couldn't figure out how to make it work.
> Would it make sense to parse the SVG files and collect the path elements in QPainterPaths, to put them in a new SVG file with the correct color scheme CSS stuff?
TBH, I don't know what you're talking about. I don't actually know Qt that well.
REPOSITORY
R266 Breeze Icons
BRANCH
add-icons-for-pagelayouts
REVISION DETAIL
https://phabricator.kde.org/D22617
To: davidhurka, #vdg, ndavis, ngraham
Cc: #okular, kde-frameworks-devel, ngraham, LeGast00n, sbergeron, michaelh, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20190726/fa48e7f5/attachment.html>
More information about the Kde-frameworks-devel
mailing list