Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

When starting most apps: Invalid Context= "Panel" line for icon theme #3277

Closed
goetzc opened this issue Jul 6, 2016 · 17 comments
Closed

When starting most apps: Invalid Context= "Panel" line for icon theme #3277

goetzc opened this issue Jul 6, 2016 · 17 comments
Labels

Comments

@goetzc
Copy link

goetzc commented Jul 6, 2016

I get this output, it seems when starting Qt apps, GTK+ apps don't show this.
Qt 5.7.0, GTK+ 2.24.30, GTK+ 3.20.6

Invalid Context= "Panel" line for icon theme:  "/usr/share/icons/Numix-Circle/16x16/panel/"
Invalid Context= "Panel" line for icon theme:  "/usr/share/icons/Numix-Circle/22x22/panel/"
Invalid Context= "Panel" line for icon theme:  "/usr/share/icons/Numix-Circle/24x24/panel/"
Invalid Context= "Panel" line for icon theme:  "/usr/share/icons/Numix/16/panel/"
Invalid Context= "Panel" line for icon theme:  "/usr/share/icons/Numix/16@2x/panel/"
Invalid Context= "Panel" line for icon theme:  "/usr/share/icons/Numix/22/panel/"
Invalid Context= "Panel" line for icon theme:  "/usr/share/icons/Numix/22@2x/panel/"
Invalid Context= "Panel" line for icon theme:  "/usr/share/icons/Numix/24/panel/"
Invalid Context= "Panel" line for icon theme:  "/usr/share/icons/Numix/24@2x/panel/"
Invalid Context= "Notifications" line for icon theme:  "/usr/share/icons/Numix/48/notifications/"
Invalid Context= "Notifications" line for icon theme:  "/usr/share/icons/Numix/48@2x/notifications/"
@palob
Copy link
Member

palob commented Jul 6, 2016

Can confirm for Miunet (KDE KF5 app) but it's just warnings. Doesn't hurt.

@goetzc
Copy link
Author

goetzc commented Jul 6, 2016

but it's just warnings. Doesn't hurt.

It hurts my eyes :)
No, seriously, it clutters the terminal, journald, and other data sent to stdout or stderr is buried alive.

Imagine all apps with small warnings here and there, all filling the terminal output with uninteresting text mixed with useful data.

@palob
Copy link
Member

palob commented Jul 6, 2016

I think it is an issue for the base theme.

I'm no panel icons expert but I don't think we could do without the panel context.

@wa4557 Any idea?

@Foggalong
Copy link
Contributor

By the looks of things it's an issue with both :(

@Foggalong Foggalong added the bug label Jul 6, 2016
@Foggalong
Copy link
Contributor

Is this because panel is a non-standard context?

@hirschmann
Copy link

You are right, KDE complains when it finds non-standard contexts, see

This warning can be fixed by replacing all occurrences of Context=Panel and Context=Notifications in index.theme with valid contexts (e.g. Context=Status).

@Foggalong
Copy link
Contributor

@wa4557 @palob Would changing these mess with the workaround at all?

@palob
Copy link
Member

palob commented Apr 19, 2017

IIRC we somewhat rely on those invalid contexts. Not sure.

@arthurflor23
Copy link

I get this output too and some softwares "can't find some icons", like this:
image

This warning can be fixed by replacing all occurrences of Context=Panel and Context=Notifications in index.theme with valid contexts (e.g. Context=Status).

Doesn't worked to me.. btw, this package "menda-circle" works fine here (but was depreciated):
https://github.com/manjaro/menda-icon-themes

I'm using Arch Linux with KDE.

@palob
Copy link
Member

palob commented Apr 29, 2017

Please open a seperate issue for the missing icons. We can't however fix this without knowing the icon names.

Menda is an outdated downstream distribution of our theme which we have no leverage on.

@palob
Copy link
Member

palob commented Apr 29, 2017

@arthurflor The missing icons are most probably part of this list numixproject/numix-icon-theme#580.

Can you please navigate to /usr/share/icons/Numix (or whereever your Numix theme is installed to), open index.theme in a text editor and add ,breeze or ,Breeze (don't know which one's correct) to the end of the Inherits= ... line and see if something chnages?

@andia89
Copy link
Contributor

andia89 commented Apr 29, 2017

Changing the context doesn't change anything for the icon lookup btw. So maybe we should change it to "status"

@arthurflor23
Copy link

Can you please navigate to /usr/share/icons/Numix (or whereever your Numix theme is installed to), open index.theme in a text editor and add ,breeze or ,Breeze (don't know which one's correct) to the end of the Inherits= ... line and see if something chnages?

It works, I put "breeze".
All the icons that were missing on system, came back:
image

@palob
Copy link
Member

palob commented Apr 29, 2017

@arthurflor This is related to numixproject/numix-icon-theme#1042.

@Foggalong
Copy link
Contributor

Sam uses Context=Status for panel/ and notifications/ in Faba and Faba-Mono so it would seem it doesn't cause any problems to match up non-standard folders with standard contexts 👍

@bilelmoussaoui
Copy link
Contributor

@Foggalong This should be moved to the packaging repo, as we don't have any index.theme file here to deal with it.

@Foggalong
Copy link
Contributor

Aye

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

No branches or pull requests

7 participants