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

Add several network logos as svg and xml #590

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ialokim
Copy link
Collaborator

@ialokim ialokim commented Feb 27, 2019

This PR is moving forward to complete #238 by replacing several PNG network logos with SVG and XML ones:

  • network_aus_logo.svg (original source: Wikimedia)
  • network_california_logo.svg (original source: Wikimedia)
  • network_mersey_logo.svg (converted with Inkscape from PNG logo)
  • network_ns_logo.svg (converted with Inkscape from PNG logo)
  • network_nz_logo.svg (original source: Wikimedia)
  • network_oregon_logo.svg (original source: Wikimedia)
  • network_pl_logo.svg (converted with Inkscape from PNG logo)
  • network_rt_logo.svg (original source: Wikimedia)
  • network_sbb_logo.svg (original source: Wikimedia)

All logos have been edited to fit exactly 64x64px and simplified as much as possible.

Furthermore, I have been working on three more SVG logos (for paris, se and rtachicago), but while converting them to Android XML using SvgToVectorDrawableConverter, it seems the current minimum API version does not support gradients. How should we handle that case?

@cla-bot cla-bot bot added the cla-signed ✔️ The Contributor Licence Agreement was signed by all contributors. label Feb 27, 2019
@grote
Copy link
Owner

grote commented Feb 27, 2019

Hey, thanks a lot for working on this!

I suggest to simplify the logos or find different ones in cases where they are too complex, have too many or too long paths (resulting in huge file sizes and memory usage), or use non-supported features such as gradients. E.g. the Canadian logos use a simple part of Canada's flag and the state code.

@ialokim
Copy link
Collaborator Author

ialokim commented Feb 27, 2019

I suggest to simplify the logos or find different ones in cases where they are too complex

That's what I've tried, do you think some of the proposed logos are still too complex?

Not sure about simplifying with gradients neither, e.g. the SNCF Logo without a gradient would just look quite different.

@grote
Copy link
Owner

grote commented Feb 27, 2019

That's what I've tried, do you think some of the proposed logos are still too complex?

Yes, some of yours are super large. Look at the Spain logo for example, adding this crown would have made the logo similarly large, so I just left it out.

Not sure about simplifying with gradients neither, e.g. the SNCF Logo without a gradient would just look quite different.

This is Paris, right? SNCF wouldn't be the good logo for that one anyway, because there's many transport agencies there. I'd suggest to find another logo. In the worst case, the eiffel tower.

@ialokim
Copy link
Collaborator Author

ialokim commented Feb 27, 2019

Yes, some of yours are super large.

Okay, which ones would you like to have replaced?

@grote
Copy link
Owner

grote commented Apr 30, 2019

@ialokim Do you still want to complete this PR?

@ialokim
Copy link
Collaborator Author

ialokim commented Apr 30, 2019

Yeah sure, I've just been a bit short of time lately. If you prefer, you could close the PR for now and I report back when I've managed to work on it.

@grote
Copy link
Owner

grote commented Apr 30, 2019

No, if you'll continue to work on it, we can leave it open.

@Altonss
Copy link
Collaborator

Altonss commented May 19, 2023

Just so I can pick up this PR, which logos in this PR need to be changed for it to be merged?

@ialokim
Copy link
Collaborator Author

ialokim commented May 23, 2023

I'd guess the ones that are "super large". Feel free to replace/improve them!

@Altonss
Copy link
Collaborator

Altonss commented Oct 9, 2023

I'd guess the ones that are "super large". Feel free to replace/improve them!

I guess that with #884 removing NZ, Oregon and California, the "super large" logos get removed anyway.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla-signed ✔️ The Contributor Licence Agreement was signed by all contributors.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants