https://bugs.kde.org/show_bug.cgi?id=471128

--- Comment #10 from Manuel de la Fuente <m...@nueljl.in> ---
> They are missing.
Yeah, I linked to the (then unmerged) merge request. Now that it's merged, it's
there. I probably should've sent the other comment after merging.
Also, as far as I'm aware, the symbolic folder just acts as a folder of icon
symlinks from non standard GNOME icons to the freedesktop ones, so adding the
16px icon there makes no sense unless it's a GTK based podcast app that's
missing the icon, which if it is, would have a different name.

> I've also noted that the 16 non-symbolic has an RSS icon instead of podcast
All the `folder-podcast` icons use the RSS icon

> plus the icon has an artifact, top-left of that circle is distorted.
The 16px non-symbolic variant looks pretty normal here. Adding it to the
Dolphin `places` pane on the sidebar renders it fine

> The folder is also colored, for some reason
> It's the wrong icon for a podcast folder and it shouldn't be colored (blue 
> default).
I think that's a different issue. Here's my thoughts on this:

Some of the newer `folder-${TYPE}` icons have color, which for the non
`folder-${COLOR}` icons is, in my honest opinion, not a good idea, since it's
inconsistent with the rest for no real reason. Why would an user want the RPM
folder to be red, but the Snap and Flatpak folder to be blue?

I also agree that the RSS icon isn't the right metaphor, as that is just one of
the channels where the user might've gotten the podcasts from, and doesn't
represent the concept of an actual podcast.

But still, this issue is about the missing size variants, so I stuck to adding
the missing variants for now.
I can do a MR to turn those folders monochrome/blue like the rest, and another
for changing the RSS logo to something like a microphone or another more
fitting icon, but after I'm done with the other ~35 icons for the 48px size.
It's more important to have no missing icons first.

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to