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

Yarufy EarTag Icon #4169

Merged
merged 3 commits into from
Dec 16, 2024
Merged

Yarufy EarTag Icon #4169

merged 3 commits into from
Dec 16, 2024

Conversation

ochi12
Copy link
Contributor

@ochi12 ochi12 commented Nov 21, 2024

I tried to be as close as possible to the original design of eartag icon while still following most of Yaru's design guidelines.

Tasks:

  • finished initial designs
  • emailed eartag repo owner knuxify
  • Permission granted
  • finalized symbolic icon.

screenshot:

Screenshot from 2024-11-21 22-25-26

@ochi12
Copy link
Contributor Author

ochi12 commented Nov 21, 2024

The music note in the symbolic icon seems to be unnecessary I might consider just going for a solid symbolic with the original silhouette.

image

PS.

2nd Iterations 3rd Iteration
image image

@Jupi007
Copy link
Member

Jupi007 commented Dec 3, 2024

Great!
But imo without the music note, it looks too much like a padlock 🤔

@Muqtxdir
Copy link
Member

Muqtxdir commented Dec 3, 2024

nitpick: could the music note be similar to yaru's music note as well?

@ochi12
Copy link
Contributor Author

ochi12 commented Dec 5, 2024

Great! But imo without the music note, it looks too much like a padlock 🤔

Lol I notice that now

@ochi12
Copy link
Contributor Author

ochi12 commented Dec 5, 2024

nitpick: could the music note be similar to yaru's music note as well?

@ochi12
Copy link
Contributor Author

ochi12 commented Dec 5, 2024

Great! But imo without the music note, it looks too much like a padlock 🤔

before
image

fixed.
Screenshot from 2024-12-05 15-18-44

Copy link
Member

@Muqtxdir Muqtxdir left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks! love the new icon's look 😃

@Muqtxdir Muqtxdir requested a review from Jupi007 December 16, 2024 14:45
Copy link
Member

@Jupi007 Jupi007 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome work, thanks a lot @ochi12 👍

@Feichtmeier Feichtmeier merged commit efe91fc into ubuntu:master Dec 16, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants