Coder Social home page Coder Social logo

Comments (2)

palob avatar palob commented on August 23, 2024

Hi, thanks for pointing this out.

There's some quirk with Numix you don't see with other themes: We've got separate themes for app icons (Circle, Square) which are designed in a different visual style and are provided in only one size (as opposed to Numix base=numix-icon-theme).
There is some overlap between the app icon themes and Numix base (and Adwaitaor breeze) in the sense you'll find icons of the same name. This is because in some desktop environments we cater for some of these duplicates are supposed to be app icons (in the Numix design language) while in other DEs they aren't.
Normally the app icon theme will take precedence over Numix base (or Adwaita or breeze), the inherits make sure icons which are a bitter fit design-wise and size-wise in the respective context are shown.
Note that we mostly don't provide *-symbolic icons or 16px icons which are designed in a GNOME-style symbolic way simply because there's "too less material to carve out a distinct Numixy design".

I can confirm that without the Adwaita inheritance I see out-of-place icons in GNOME apps. I'm a bit out of the KDE side of things loop. I recall some blank spots in KDE apps for icons we don't provide without the breeze inheritance in the past.

Now this inheritance line dates back quite a while and it's a hacky solution. Best practice would be to test whether the duplicative icons names are still needed in our app icons themes (they were at some point for DEs like Xfce, Cinnamon, LXDE/Qt) or remove them regardless.

So yes, technically this is a dependency on Adwaita and breeze. However since you only need those inheritances for GNOME or KDE apps (which depend on Adwaita/breeze in all distros I know of) it effectively isn't.
But I can add a section to the README.

A profound fix for this isn't likely in the near future given I'm the only person working on Numix these days at capacity keeping the app icon themes up to date.

gnome can be dropped right away, it has long been deprecated I think.

from numix-icon-theme.

alexxcons avatar alexxcons commented on August 23, 2024

Thank you for the details!

If an App requires additional icons which are not part of the standard, according to the spec,that App should either install these additional icons itself into the "hicolor" folder or use a fallback theme during icon loading.

So if an App misses providing these additional icons and relies on having other themes in the Inherit tree, you should report that to the app developers. If these icons are used by multiple applications, it might make sense to request a new icon-name for them on freedesktop.org, so that all themes can ship that icon.

KDE currently is implementing a fallback for their icon-loader, so that breeze will always be used as fallback if the current theme does not ship some KDE specific icons: https://invent.kde.org/frameworks/kiconthemes/-/issues/3

So it probably will make sense to drop breeze from the Inherit list, once that MR got merged and released.

I can confirm that without the Adwaita inheritance I see out-of-place icons in GNOME apps.

Which apps exactly, and which icons are missing? If you do not open an issue against them, I can do so.

Note that we mostly don't provide *-symbolic icons

Yea, symbolic icons currently are a blind spot, since they are not yet part of the fd.org spec. There is currently an issue about adding them to the spec, though so far no concrete MR. Symbolic versions of icons are pretty much used everywhere these days, and most likely will be added to the spec soon. If Numix does not provide them, it probably makes sense to Inherit from some other icon-theme which does so and request a hard package dependency on it.

from numix-icon-theme.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.