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

https://galaxy.ansible.com favicon is low DPI (www.ansible.com one fine) #161

Closed
tim-seoss opened this issue Nov 7, 2017 · 4 comments
Closed
Assignees
Labels
Milestone

Comments

@tim-seoss
Copy link

Very minor niggle - the favicon (e.g. as displayed in Chrome for tabs under https://galaxy.ansible.com ) is low DPI. The ones used by http://docs.ansible.com/ and https://www.ansible.com/ are both high DPI (although a bit different from each other, strangely).

So the Galaxy ones look pixelated on a 4k (or other high DPI) display.

@cchurch
Copy link
Contributor

cchurch commented Nov 7, 2017

FYI, additional info on what's missing from a site I've used to create favicons: https://realfavicongenerator.net/favicon_checker?protocol=https&site=galaxy.ansible.com

@cutwater cutwater added this to the 2.4 milestone Nov 8, 2017
@cutwater cutwater self-assigned this Nov 8, 2017
@cutwater cutwater added the status/fix-committed Merged to develop \ release branch label Nov 10, 2017
@cutwater
Copy link
Collaborator

@cchurch Thanks for suggestion, it's really useful resource.

@cutwater
Copy link
Collaborator

cutwater commented Nov 10, 2017

favicon.ico has been replaced with multisize version. Fix will land on https://galaxy.ansible.com in the next release.

However there is another problem remains that public site has two different icons /favicon.ico and /static/img/favicon.ico. Created a separate issue for tracking it #174

@chouseknecht
Copy link
Contributor

@chouseknecht chouseknecht removed the status/fix-committed Merged to develop \ release branch label Dec 4, 2017
@cutwater cutwater removed the shipit label Dec 4, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants