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

Icon Theme: Possible new icon for Transmission #1687

Closed
wants to merge 5 commits into from

Conversation

ubuntujaggers
Copy link
Contributor

Just an idea, ignore if you don't like it - we have our own icon for Transmission but it's not great at all. I wonder if we'd be better doing what some other themes do (e.g., Paper) and just have a simple glyph. The compression isn't very flattering on the screenshots so have a look at the actual files to compare.

image

image

image

@clobrano
Copy link
Member

clobrano commented Jan 3, 2020

I am not sure about this. For my point of view, since it is quite abstract, it is not immediately recognizable as Transmission. Should we involve the maintainer first?

@madsrh
Copy link
Member

madsrh commented Jan 3, 2020

...it is not immediately recognizable as Transmission...

I had the same thought

@ubuntujaggers
Copy link
Contributor Author

ubuntujaggers commented Jan 3, 2020

Good point, I forgot Transmission was one of the ones where we had maintainer permission. Maybe we can keep the same design as current but make it a bit "slicker"?

Edit: for instance, IMO the double-headed arrow used here is nicer than the one used in the current grey and red design.

@ubuntujaggers
Copy link
Contributor Author

ubuntujaggers commented Jan 3, 2020

@madsrh @clobrano What if I push a version like this:

image

image

Same basic design as current in master but just a bit "sleeker"?

EDIT: Finished that design and pushed.

image

Current icon in Master:

image

New icon in this PR:

image

@clobrano
Copy link
Member

clobrano commented Jan 3, 2020

I personally prefer the current one ☺️

@ubuntujaggers
Copy link
Contributor Author

I personally prefer the current one

Colours or shapes or both...?

@clobrano
Copy link
Member

clobrano commented Jan 3, 2020

Right, I said nothing useful :D

About the shapes, I see the new one a slightly better, but overall I prefer the colors and the fold effect of the current one

@ubuntujaggers
Copy link
Contributor Author

Hmm, maybe I can fuse elements of both 🤔

@ubuntujaggers
Copy link
Contributor Author

ubuntujaggers commented Jan 3, 2020

@clobrano How about

Current:

image

Suggested:

image

image

@clobrano
Copy link
Member

clobrano commented Jan 3, 2020

Very nice for me 👍
Do you remember the transmission maintainer's nick (I'm on mobile right now)?

@ubuntujaggers
Copy link
Contributor Author

I'll find it and tag him in after I've edited the first post.

@clobrano
Copy link
Member

clobrano commented Jan 4, 2020

I found the nicks in the previous PR :)

Hey @livings124, @mikedld,

sorry if I ping you here, but I found your nicks in the previous Transmission icon PR (#1426)
@ubuntujaggers worked on an iteration of the icon, which I think it's really good, what's your opinion, can we use it inside Yaru?

Thanks!

@Feichtmeier
Copy link
Member

Imho the outlines are too strong on this one

@ubuntujaggers
Copy link
Contributor Author

Ah, let's leave it for now, the latest version isn't massively different anyway :)

@ubuntujaggers ubuntujaggers deleted the IconTheme_TransmissionImprovement branch March 3, 2020 18:11
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