-
-
Notifications
You must be signed in to change notification settings - Fork 447
add zen browser icon #3931
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
add zen browser icon #3931
Conversation
just realized the icons are stretched cuz i was working on a very high monitor... |
yes! you're absolutely right. after that you said i just realized that the shadows make it look it's not perfectly round, like it's streched. i also will do that svg example you sent, it looks better to me too without the plinth. thanks for the tips! |
Looking at the history, there's a freecad update on this branch too, but it's a duplicate of the one in #3928 that's been merged already, so it'll merge cleanly on the principle of the same changes being present. We're chasing February 1st for a release, and it would be good to have this attractive new web browser icon in Papirus before then. I think the plinth design could be accepted as-is if we don't mind the matrix transforms, and time is pressing. It's 5 minutes work to get rid of them too. If slightly clearer redesign is ready on this branch though, that'd be great! I know it's a lot of fiddly circle realignment, though ;) |
no problem, I was already planning to work on them today |
@achadwick i didn't understand exactly what you mean, but i did what you said, is this right now?: |
Yeah, that doesn't contain any matrix transforms now 🎉 Please run "prepare.sh" to clean up the icons before "put-into-theme.sh" and committing+pushing with git. It's all documented here: https://github.com/PapirusDevelopmentTeam/papirus-icon-theme/blob/master/tools/work/README.md#4-final-steps I've just noticed that we're still recommending the wrong dependencies for |
yes! i personally am using linux mint so i downloaded the scour package directly from the app store, i don't know if other distros also have that, but i think it would be cool if you advice that too, once it's more "user friendly" |
No trouble. Most things have some sort of software center now. I'll advise https://repology.org/project/scour/versions too so folks can find an exact package name (I guess Mint get theirs from Ubuntu?) |
yes |
Looks great now! Everything's neat and on the grid without any hidden transforms. If you're happy with the design, let's merge it. @morganist: ready for merge here. Good alignments, tests all pass. |
all right, the alignment is great, but it doesn't feel like a Papirus icon without the layering (no shadows). I guess it's easier that way because the icon only consists of circles, but subtle shadows under every orange ring would definitely give it the Papirus look. I'll merge and rework it from there, thanks! |
@sotha1 this may make the icons look stretched, but when rendered at the size they are supposed to, the shadows only really make the elements look layered and don't distort the shape. Layering is the distinguishing feature of Papirus, the same goes for pixel-perfect alignment, warm, juicy colours, and the base white colour. The icons as they are right now look like pixel-perfect rework of the original, and not like a Papirus icon (base white is too bright, no layering). Just give me a bit of time, I'll commit the fixes and tell me what you think. |
looks great for me! i haven't tested it on-the-go yet so i haven't seen how it would fit in, but at least now for me is great. |
No description provided.