Created attachment 115522 [details] Screenshot of the Dictionary icon SUMMARY STEPS TO REPRODUCE Open the plasmoids list and look for the dictionary icon. OBSERVED RESULT Is the same icon used by the Oxygen theme but with some graphic problems. It's evident that the pages of the book are out of place. EXPECTED RESULT Add an icon using the Breeze aesthetics. SOFTWARE VERSIONS KDE Plasma Version: 5.13.5 KDE Frameworks Version: 5.50.5 Qt Version: 5.11.2 ADDITIONAL INFORMATION I can confirm that the problem is present in Manjaro and Neon.
Confirmed on openSUSE Tumbleweed. Looks like the .svg icon file got messed up somehow. I don't have the requisite Inkscape skills to fix it, unfortunately.
This is actually part of the kdeplasma-addons product, not Breeze icons. Moved to the correct category
This icon clearly does not match the Breeze icon style, so perhaps the dictionary should be given an entirely new icon?
Agreed.
Whats the icon name?
The plasmoid ships its own "accessories-dictionary" svg file but Oxygen has one already. If we added a similar icon in Breeze, we could stop if from shipping its own icon and just use the one on the icon theme.
This bug is fixed on my system. Operating System: Arch Linux KDE Plasma Version: 5.16.90 KDE Frameworks Version: 5.62.0 Qt Version: 5.13.1
Mine still looks like the initial screenshot
No new icon has been made, so it should still be broken.
Created attachment 122762 [details] screenshot taken on Plasma 5.17 beta Weird. I'm attaching a screenshot. And yes, I use breeze icon theme.
Still broken for me on master, though.
(In reply to Patrick Silva from comment #10) > Created attachment 122762 [details] > screenshot taken on Plasma 5.17 beta > > Weird. I'm attaching a screenshot. And yes, I use breeze icon theme. This may be because you have the Oxygen icon theme installed and Breeze inherits from Oxygen on some distros. I can see in your screenshot that the blur effect is being used, even though the Qt SVG renderer does not support blur. This means the SVG has been converted to a PNG.
This is an Oxygen icon and Oxygen icons are meant to be converted to PNGs. Perhaps this icon should be converted to a PNG when the applet is built? That, or the icon should be completely replaced with a Breeze icon. The latter should probably happen eventually, but the former would be enough to close this bug.
Created attachment 149186 [details] broken icon in krunner As seen in the screenshot attache to this comment, Dictionary icon is broken in krunner too. Operating System: Arch Linux KDE Plasma Version: 5.24.90 KDE Frameworks Version: 5.94.0 Qt Version: 5.15.4 Graphics Platform: Wayland
It's broken in the icon itself. Manuel, would you be interested in making a nice new Breeze-style dictionary icon?
Git commit 0ca1edead879cfa29c72124015a3f08bc9d751d3 by ivan tkachenko. Committed on 15/07/2022 at 21:03. Pushed by ratijas into branch 'master'. Remove clipPath portion of SVG Not only Qt SVG does not support clipping, but it would also simply draw a clipping mask as is it were a regular shape! M +- -- applets/dict/sc-apps-accessories-dictionary.svgz https://invent.kde.org/plasma/kdeplasma-addons/commit/0ca1edead879cfa29c72124015a3f08bc9d751d3
Git commit 3854385a81922824bf7ac3ccf4a544fb943a2b5c by Nate Graham, on behalf of ivan tkachenko. Committed on 15/07/2022 at 21:36. Pushed by ngraham into branch 'Plasma/5.25'. Remove clipPath portion of SVG Not only Qt SVG does not support clipping, but it would also simply draw a clipping mask as is it were a regular shape! (cherry picked from commit 0ca1edead879cfa29c72124015a3f08bc9d751d3) M +- -- applets/dict/sc-apps-accessories-dictionary.svgz https://invent.kde.org/plasma/kdeplasma-addons/commit/3854385a81922824bf7ac3ccf4a544fb943a2b5c
Git commit 4fae3fa7c9a21d2fc6224ea48e10525330ae46ce by Nate Graham, on behalf of ivan tkachenko. Committed on 15/07/2022 at 21:37. Pushed by ngraham into branch 'Plasma/5.24'. Remove clipPath portion of SVG Not only Qt SVG does not support clipping, but it would also simply draw a clipping mask as is it were a regular shape! (cherry picked from commit 0ca1edead879cfa29c72124015a3f08bc9d751d3) M +- -- applets/dict/sc-apps-accessories-dictionary.svgz https://invent.kde.org/plasma/kdeplasma-addons/commit/4fae3fa7c9a21d2fc6224ea48e10525330ae46ce