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

[Improvement] Wrong folder colour and Audaciou's icons are not in pure Yaru style #1912

Closed
gusbemacbe opened this issue Feb 14, 2020 · 20 comments · Fixed by #1934
Closed

[Improvement] Wrong folder colour and Audaciou's icons are not in pure Yaru style #1912

gusbemacbe opened this issue Feb 14, 2020 · 20 comments · Fixed by #1934
Assignees

Comments

@gusbemacbe
Copy link
Contributor

Actual Behaviour

image

Description

In the folder "/icons/Suru/256×256/legacy", the folder colour (orange) is wrong and should be like the red and grey folders. In the app Audacious, the folder icon is wrong, and the icons are not in pure Yaru style.

@clobrano
Copy link
Member

What is the role of "legacy" icons? According to the name, it looks like something from the past, but where it is used?

@gusbemacbe
Copy link
Contributor Author

@clobrano

Look at the app Audacious. It uses the legacy folder icon. Audacious is not old or outdated. It is updated. The latest stable is in December 2018 and its latest nightly release is in December 2019.

@clobrano
Copy link
Member

I wonder what is the need to use the legacy folder

@gusbemacbe
Copy link
Contributor Author

I am not developer of Audacious, then I can not say why.

But in my opinion, using different folder colour in an app while the system uses another folder colour sounds strange for an user. I did not report it. It is the Italian guy who reported it. He found it strange.

If you want to keep that folder colour in the legacy folder just because it is a legacy icon, I understand and will explain to the Italian guy.

@Feichtmeier
Copy link
Member

Feichtmeier commented Feb 14, 2020

Oh that icon is used in inkscape for example. It is indeed wrong! Ty @gusbemacbe

@gusbemacbe
Copy link
Contributor Author

@Feichtmeier, ah, I understood, I closed the issue because I thought Carlo does not want to change icons, and wants to keep the old icons.

@ghost
Copy link

ghost commented Feb 14, 2020

@gusbemacbe I already asked the question on this icon :)

#1797 (comment)

@clobrano
Copy link
Member

@Feichtmeier, ah, I understood, I closed the issue because I thought Carlo does not want to change icons, and wants to keep the old icons.

No sorry, I was just wondering about the meaning of the "legacy" section.

@gusbemacbe
Copy link
Contributor Author

@clobrano, @Feichtmeier and @FriendlyApple

Reported from the Italian guy, not only Audacious and Inkscape, other apps also use the legacy icons, including the orange folder icon. Observe that you developed a new icon for Transmission, then you also should change the folder icon and other old icons:

  • Transmission:

transmission

  • Puddletag, compare the orange folder (left top) to the different folder icons (left bottom):

puddletag

@madsrh
Copy link
Member

madsrh commented Feb 14, 2020

Thanks @gusbemacbe 👍

A general icon question: Is there a way to tell where an icon is used? (sorry for the lightly off topic comment)

@gusbemacbe
Copy link
Contributor Author

gusbemacbe commented Feb 14, 2020

@madsrh

@gusbemacbe
Copy link
Contributor Author

@madsrh

The post you pinged me is locked and limited to collaborators, then I can not answer there. I have to answer here.

@gusbemacbe
Copy link
Contributor Author

@madsrh

Suru++ uses the symbolic icon which look great but that the downsides that it's monochrome and that it isn't visible in the dark theme.

Which Suru++? @Bonandry's Suru++ Ubuntu is totally outdated and buggy.

My Suru++ is not updated for months because I was too dedicated to two projects Adwaita++ and Yaru++ @Bonandry broke 8 months ago. I took 6 months to repair, correct and redesign Adwaita++, having to redesign actions, apps and status icons, and purging the impure icons copied from Suru++ Ubuntu. I have already forbidden @Bonandry of copying and mixing Suru++ Ubuntu, overwriting Adwaita++ and Yaru++.

Adwaita++ is totally fixed, ready and upgraded last month. At the moment, I am repairing Yaru++ in my branch gusbemacbe.

You may try to test with the upgraded Adwaita++ and the unstable branch gusbemacbe's Yaru++ to see if the symbolic icons work on both dark and light environments.

@madsrh
Copy link
Member

madsrh commented Feb 17, 2020

Hi @gusbemacbe

I've had a look at the Yaru++ branch and WOW! You've done a huge amount of awesome work 🎉 - I'm very impressed 🥇
We're missing a lot of the stuff you have and to be honest, I'm a bit sad that all that awesome work is only found downstream in Yaru++ and not here in Yaru 😥 We really could use your talent here, but I understand that you're buried in work with all your other projects, and fixing issues here would be double the work. Anyway, we're happy that you drop by occasionally 😃

And thanks for the explanation above!

I've made a PR that fixes this issue 😉

@gusbemacbe
Copy link
Contributor Author

gusbemacbe commented Feb 17, 2020

I've had a look at the Yaru++ branch and WOW! You've done a huge amount of awesome work 🎉 - I'm very impressed

Thank you!

We're missing a lot of the stuff you have and to be honest, I'm a bit sad that all that awesome work is only found downstream in Yaru++ and not here in Yaru 😥.

As @Bonandry's projects are under GPL licence, you can copy from them to your project, but the problem is that you use PNG.

I have talked with @Bonandry, he said that 16px legacy icons do not seem good. And he said he would like his Yaru++ project be credited in your project because some of your icons are inspired by his project.

We really could use your talent here, but I understand that you're buried in work with all your other projects, and fixing issues here would be double the work. Anyway, we're happy that you drop by occasionally 😃

I am finishing the Yaru++ project. Then I can work a bit with your project.

@madsrh
Copy link
Member

madsrh commented Feb 17, 2020

Yaru++ project be credited in your project because some of your icons are inspired by his project.

Yes, sorry about that. We really need to update Authors and credit files! Also, we're already borrowing from all the great work you guys are doing!

I am finishing the Yaru++ project. Then I can work a bit with your project.

THAT would be much welcome ❤️ 🎉
We have User Interface Freeze on March 19 for the LTS release, so if you find anything critical please let us know!

And ofc The icons ... Maybe some of them maybe more :)

@Feichtmeier I think gusbemacbe knows that we're not using monochrome icons or thirdparty apps 😉

@Feichtmeier
Copy link
Member

Feichtmeier commented Feb 17, 2020

@gusbemacbe we would need help with the full colour legacy icons (used in old gtk3 and gtk2 apps) and symlinks. That'd be really really nice.

We use a script to create symlinks from a .list file for each icon category to real symlinks

@ubuntujaggers
Copy link
Contributor

And he said he would like his Yaru++ project be credited in your project because some of your icons are inspired by his project.

Which ones...?

@Amr-Ibra
Copy link

What is the role of "legacy" icons? According to the name, it looks like something from the past, but where it is used?

Legacy icons are needed primarily for Qt apps (see #1621 and #1294 for history), and as a side effect they are also picked up by GTK2 and GTK3 apps, which don't use symbolic icons, just because of the icon naming. They are not "strictly" needed there, as Humanity icons are sufficient for them.
They are placed in a "legacy" folder because that's what upstream Adwaita is doing with full-colour icons.

@clobrano
Copy link
Member

Great explaination, thanks a lot

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

Successfully merging a pull request may close this issue.

6 participants