Bug 414821

Summary: "Missing icon" icon is dark for Breeze dark
Product: [Frameworks and Libraries] frameworks-attica Reporter: postix <postix>
Component: generalAssignee: Dan Leinir Turthra Jensen <leinir>
Status: ASSIGNED ---    
Severity: normal CC: admin, aleixpol, nate, postix
Priority: NOR Keywords: usability
Version: 5.64.0   
Target Milestone: ---   
Platform: Other   
OS: Linux   
Latest Commit: Version Fixed In:
Sentry Crash Report:
Attachments: Screenshot. The icon is only slightly vissible due to the selection.
Screenshot of the dropshadow.

Description postix 2019-12-04 12:13:59 UTC
Created attachment 124306 [details]
Screenshot. The icon is only slightly vissible due to the selection.

SUMMARY

Under Breeze dark discovers displays a dark version for a missing icon, which is not visible.

SOFTWARE/OS VERSIONS
Operating System: Manjaro Linux 
KDE Plasma Version: 5.17.4
KDE Frameworks Version: 5.65.0
Qt Version: 5.13.2
Comment 1 postix 2019-12-04 12:17:44 UTC
Created attachment 124307 [details]
Screenshot of the dropshadow.

If there are no screenshots, there is also the dark version of the "image missing" icon, which has a weird drop shadow.
Comment 2 Nate Graham 2019-12-04 17:11:55 UTC
Looks like attica is giving us a pixmap as the placeholder icon that doesn't look good with dark themes.
Comment 3 Dan Leinir Turthra Jensen 2019-12-04 17:54:41 UTC
Well damn... Right, that is the image returned by the KDE Store, and it's what we ended up with last year about this time, because the previous one was just a 100x100 100% transparent png, which made everything look super broken. So... now i guess we need to find something to swap this one with, which works on every possible background and colour combination that people might pick. This sounds like a job for someone who does visual work, but also it's a KDE Store issue rather than Attica (which just forwards what it's given by the server).

So... VDG task, i guess, for picking the new image, but not sure of the product to aim it at. The store team will go with whatever they're given here (but of course they'd rather not have to do it too often).
Comment 4 Nate Graham 2019-12-04 19:25:21 UTC
You could just add a white border around this image. That would fix it for everything.
Comment 5 Dan Leinir Turthra Jensen 2019-12-05 09:54:38 UTC
If you're sure that'd work, then yup, that certainly would be a nice, easy fix :)
Comment 6 Nate Graham 2019-12-05 14:42:37 UTC
Definitely better than the status quo IMO, and it will take the pressure off so we can take our sweet time implementing a different solution if we want.
Comment 7 Bug Janitor Service 2025-01-17 03:18:14 UTC
A possibly relevant merge request was started @ https://invent.kde.org/frameworks/breeze-icons/-/merge_requests/441