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

Keep track of upstream adwaita-icon-theme #1544

Closed
Amr-Ibra opened this issue Sep 21, 2019 · 10 comments · Fixed by #2743
Closed

Keep track of upstream adwaita-icon-theme #1544

Amr-Ibra opened this issue Sep 21, 2019 · 10 comments · Fixed by #2743

Comments

@Amr-Ibra
Copy link

Amr-Ibra commented Sep 21, 2019

Since Yaru is now syncing with upstream adwaita-gtk and gnome-shell theme, I think it would be a good idea to also keep an eye on upstream adwaita-icon-theme.

adwaita-icon-theme mostly contains symbolic and action icons that correspond to code changes in gnome-shell, gnome-control-center and the rest of gnome apps. In every gnome development cycle, adwaita-icon-theme adds, fixes, or enhances its symbolic icons in order to reflect the code changes in the gnome stack. adwaita-icon-theme does not contain app icons since gnome apps ship their own icons.

An example to elaborate:
In the 3.33 development cycle, gnome added new battery icons in order to better represent the battery charge states: https://gitlab.gnome.org/GNOME/adwaita-icon-theme/issues/6

That has resulted in:

Without reflecting the new change in Yaru¹, gnome-shell would still be falling back to the legacy icons, and not showing a better charge state as a result.
¹ #1482

Also, a question:
Does an app (or Yaru itself) fall back to adwaita-icon-theme if a requested icon is not found in Yaru?

Here is the gnome-stencils.svg as a reference:
https://gitlab.gnome.org/GNOME/adwaita-icon-theme/blob/master/src/symbolic/gnome-stencils.svg
grafik

@clobrano
Copy link
Member

Hi @Amr-Ibra, this is a good observation.
When we don't see this updates by ourselves, we usually rely on notifications from Ubuntu desktop guys or other contributors for such important changes. Unfortunately none of us has the time to keep a constant check of these changes and the repo is not organized to get them automatically.

We could keep this ticket opened to discuss a repo change to get automatic info about this (submodules?)

@Amr-Ibra Amr-Ibra changed the title [Discussion] Keep track of upstream adwaita-icon-theme Keep track of upstream adwaita-icon-theme Dec 2, 2019
@Feichtmeier
Copy link
Member

Wouldn't this be something to be made with your GitHub Actions? @clobrano
Ofc not for 20.04 😆

@clobrano clobrano self-assigned this Feb 18, 2020
@clobrano clobrano added this to Backlog in Yaru via automation Feb 19, 2020
@clobrano clobrano removed their assignment Feb 24, 2020
@Feichtmeier
Copy link
Member

I made some Java github actions on my repos, maybe I can do this too

We could use the checkout action on the GitHub gitlab mirror of adwaita
https://github.com/GNOME/adwaita-icon-theme

But I'm not sure if we want so many more files in our repo.
The src folder should be enough

@Feichtmeier Feichtmeier moved this from Backlog to To Do - 30.04 in Yaru Oct 7, 2020
@Feichtmeier
Copy link
Member

@didrocks since we've talked about this, I thought it could not hurt to assign you 👼 Feel free to unassign ofc ^^

@Feichtmeier Feichtmeier moved this from To Do - 21.04 to Backlog in Yaru Feb 18, 2021
@Feichtmeier Feichtmeier assigned Jupi007 and unassigned didrocks Mar 10, 2021
@Feichtmeier
Copy link
Member

@Jupi007 if you have time, maybe you could use your github action knowledge to create an action for checking the adwaita icon theme for changes?

@Jupi007
Copy link
Member

Jupi007 commented Mar 10, 2021

I'll have try 👍

@Jupi007
Copy link
Member

Jupi007 commented Mar 22, 2021

@Feichtmeier I see two way to do that:

  • Use the current Github workflow, but that will need to import a lot of files into this repo.
  • Or create a new workflow which will only do a list file of Adwaita icons.

Which one do you prefer?

@Feichtmeier
Copy link
Member

I think the most critical icons are the symbolics and the file name changes. We don't need the whole adwaita icon set
But I don't know if this is checkable in an action
Basically only the "gnome-stencils.svg" file is needed to be checked plus/or the exported symbolic subfolder

@Jupi007
Copy link
Member

Jupi007 commented Mar 22, 2021

I can try to do an action which list all exported symbolic Adwaita icon. I think it is the best way to have just new files and rename notifications.

Yaru automation moved this from Backlog to Done Apr 1, 2021
@emyoulation
Copy link

emyoulation commented Jul 19, 2021

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

Successfully merging a pull request may close this issue.

6 participants