[Spectacle] [Bug 476084] Missing icons don't fallback gracefully

2023-10-27 Thread Kristen McWilliam
https://bugs.kde.org/show_bug.cgi?id=476084

--- Comment #6 from Kristen McWilliam  ---
Thank you~

I read through the spec..

"In some cases you don't always want to fall back to an icon in an inherited
theme. For instance, sometimes you look for a set of icons, prefering any of
them before using an icon from an inherited theme. To support such operations
implementations can contain a function that finds the first of a list of icon
names in the inheritance hierarchy . . . This can be very useful for example
when handling mimetype icons, where there are more and less "specific" versions
of icons."

This sounds to me like it is intended to be optional, and only for specific
circumstances. I'm not actually clear on why this behaviour would be desired
instead of a fallback theme really.

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

[Spectacle] [Bug 476084] Missing icons don't fallback gracefully

2023-10-26 Thread Nate Graham
https://bugs.kde.org/show_bug.cgi?id=476084

--- Comment #5 from Nate Graham  ---
Check out the part that starts with "In some cases you don't always want to
fall back to an icon in an inherited theme."

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

[Spectacle] [Bug 476084] Missing icons don't fallback gracefully

2023-10-25 Thread Kristen McWilliam
https://bugs.kde.org/show_bug.cgi?id=476084

--- Comment #4 from Kristen McWilliam  ---
Do you have a reference for this behaviour? Accoring to the Freedesktop Icon
Theme Spec
(https://specifications.freedesktop.org/icon-theme-spec/icon-theme-spec-latest.html):

"Inherits: The name of the theme that this theme inherits from. If an icon name
is not found in the current theme, it is searched for in the inherited theme
(and recursively in all the inherited themes)."

I checked Tela, and it does specify Inherits:
"Inherits=hicolor,Adwaita,breeze". I haven't seen anything about how it
searches for the first part of the name instead so far.

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

[Spectacle] [Bug 476084] Missing icons don't fallback gracefully

2023-10-25 Thread Nate Graham
https://bugs.kde.org/show_bug.cgi?id=476084

--- Comment #3 from Nate Graham  ---
Indeed. Ordinarily I might recommend that you bring this up upstream, but TBH I
doubt it would get anywhere. The FreeDesktop icon spec is probably one of the
most problematic ones out there, with major glaring flaws and unclear paths to
compliance by icon themes. It's for this reason that GNOME has started
abandoning it. We like to play nice and adhere to common standards in KDE, but
sometimes this bites us in cases where the common standards themselves aren't
ideal.

We could probably change KIconLoader to fall back to an icon in the fallback
theme for every single icon that's missing from the active theme, but this
would cause the iconography of apps' to switch between two styles much more
commonly. And again, it would break the spec. But maybe that's the lesser of
two evils?

Worth bringing up in a Gitlab Issue at
https://invent.kde.org/groups/teams/vdg/-/issues, IMO.

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

[Spectacle] [Bug 476084] Missing icons don't fallback gracefully

2023-10-25 Thread Kristen McWilliam
https://bugs.kde.org/show_bug.cgi?id=476084

--- Comment #2 from Kristen McWilliam  ---
Agreed, the behaviour seems slightly problematic - as evidenced by every
missing icon here falling back to "draw" instead of "draw-highlight",
"draw-arrow", etc.

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

[Spectacle] [Bug 476084] Missing icons don't fallback gracefully

2023-10-25 Thread Nate Graham
https://bugs.kde.org/show_bug.cgi?id=476084

Nate Graham  changed:

   What|Removed |Added

 CC||n...@kde.org
 Status|REPORTED|RESOLVED
 Resolution|--- |UPSTREAM

--- Comment #1 from Nate Graham  ---
I'm afraid this isn't the way Freedesktop icon themes work. When the requested
icon in the current theme is missing, the fallback icon is generated by
chopping off the last word of the missing icon and then seeing if the icon
theme has an icon by that name. In this case, it does.

Whether this behavior is appropriate or not could be debated. I can definitely
see the argument that is causes icons to become needlessly generic and maybe
even meaningless. However we oughtn't break the spec just because of this; we
should change it instead.

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