-
Notifications
You must be signed in to change notification settings - Fork 57
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
Update for the application logos of several icons #362
Conversation
Isn't it better to make these changes in this P.R.? |
The icons on this pull request are quite far from the style of the original icon pack, which is based on the Moka icon theme. I see that it's trying to go for a more flat design ("modern", maybe) look since the gradients and almost all the shading have been removed, but I'm not inline with that direction. |
The only icon that I find problematic in Mint-Y is the Firefox one, which is very different from the original, and what the user uses on the system is the internet browser, mainly which comes from Windows, but there is still Yaru and Breeze as an alternative from 21.1 |
Just to be clear, I'm referring to the icons of the pull request you pointed out compared to the icons already in use/the icons of the Moka theme. |
It uses the monochrome version of the Firefox Browser logo (which by the way is provided by Mozilla), but I agree it could have more resemblance to their branding. I've even made a revision with that in mind, but didn't commit it yet. |
According to Mozilla, the icon cannot be gradient. |
Thank you for your work. I hope this gets merged quickly. |
These are nice looking icons! I've added the Firefox icons to my setup manually, and they look good in practice too. |
@Hokiper I really like what you're doing here, but I can't merge it. It's not modular enough... it's an ALL or NOTHING PR. There are changes I'm happy to merge immediately, some I'd like to think about, some I don't think we want to go for, some which conflict with other changes (Sublime icons got their own PR).. etc. Please make individual PRs for individual changes, they'll get merged much more easily. |
#316 |
OK, I will make individual PRs when I get some time.
I will look into that when I'm available. Thanks for the request! |
This looks great, hope it gets merged. |
@clefebvre I'm still waiting for a more detailed response regarding this before I go on with the individual pull requests, just so I have a better idea on how the icons are expected to look like. |
Perhaps they're preparing for LM 22 and didn't prioritize the mint-y-theme as much i myself have made several new icons and would like to submit them but the pull request tab is a mess and a lot of things will have to change Mind checking some of them and telling me what you think? |
@clefebvre a more detailed guide/readme on how to commit to this repo would be nice. Like what criteria new icons need to have? |
I will take a look and comment your PR once I'm available. 🌝
You can generate the PNGs using the Python scripts under |
Err...I'm a bit of a dummy, which one? there are three and i don't have any idea what each one might do Also still wanting to know watcha think, some of these icons might not look as good tho |
Sorry for responding this late, yeah thx a lot ill keep this advice in mind, i do include shading on most of my icons but its mostly a darker colored layer on top of the main background (beneath the logo), i find it more pleasant to work with im sorry if certain problems arise from it |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Hokiper @PHPeon @CZmisaCZ @NikoKrause don't hesitate to join https://matrix.to/#/#linuxmint-artists:matrix.org. We're looking at artwork changes for Mint 22, it'd be good to have a chat about it. |
@vadimk1337 Just added the icon you requested. It took me quite a while to decide the colors… |
@Hokiper |
All merged on master, except the Firefox commit which needs a separate PR. |
I noticed that a lot of icons were using outdated logos of their corresponding applications, so I updated them using the current branding assets of the applications and also took the time to add some new ones.
Maybe I will do the same thing with more icons in the future, but I believe the amount I've already done is enough for a pull request.