davidhurka updated this revision to Diff 62603. davidhurka added a comment.
- 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... 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? REPOSITORY R266 Breeze Icons CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D22617?vs=62602&id=62603 BRANCH add-icons-for-pagelayouts REVISION DETAIL https://phabricator.kde.org/D22617 AFFECTED FILES icons-dark/actions/16/snap-page.svg icons-dark/actions/16/view-pages-continuous.svg icons-dark/actions/16/view-pages-facing-first-centered.svg icons-dark/actions/16/view-pages-facing.svg icons-dark/actions/16/view-pages-overview.svg icons-dark/actions/16/view-pages-single.svg icons-dark/actions/22/snap-page.svg icons-dark/actions/22/view-pages-continuous.svg icons-dark/actions/22/view-pages-facing-first-centered.svg icons-dark/actions/22/view-pages-facing.svg icons-dark/actions/22/view-pages-overview.svg icons-dark/actions/22/view-pages-single.svg icons-dark/actions/24/snap-page.svg icons/actions/16/snap-page.svg icons/actions/16/view-pages-continuous.svg icons/actions/16/view-pages-facing-first-centered.svg icons/actions/16/view-pages-facing.svg icons/actions/16/view-pages-overview.svg icons/actions/16/view-pages-single.svg icons/actions/22/snap-page.svg icons/actions/22/view-pages-continuous.svg icons/actions/22/view-pages-facing-first-centered.svg icons/actions/22/view-pages-facing.svg icons/actions/22/view-pages-overview.svg icons/actions/22/view-pages-single.svg icons/actions/24/snap-page.svg To: davidhurka, #vdg, ndavis, ngraham Cc: #okular, kde-frameworks-devel, ngraham, LeGast00n, sbergeron, michaelh, bruns