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

Retina browser action icon #51

Closed
iainbeeston opened this issue May 4, 2013 · 5 comments
Closed

Retina browser action icon #51

iainbeeston opened this issue May 4, 2013 · 5 comments
Assignees
Labels
low

Comments

@iainbeeston
Copy link

@iainbeeston iainbeeston commented May 4, 2013

As part of upgrading to manfest v2, it would be great if you could add a high-resolution (eg. 38px) browser icon (this depends on issue #50)

@ghost ghost assigned kepi May 4, 2013
@kepi
Copy link
Owner

@kepi kepi commented May 4, 2013

No problem, will be added in next version.

@kepi kepi closed this in 42f7059 May 5, 2013
@iainbeeston
Copy link
Author

@iainbeeston iainbeeston commented May 9, 2013

I have the new version now, and it has a higher resolution icon than before. But I'm curious why you chose a 33px high-resolution icon rather than a 38px one (which is the standard size as explained in the chrome developer documentation). Because for me the icon is stilly blurry. If it helps I'd be happy to downscale one of your larger icons and send you a pull request for a 38px icon?

@kepi
Copy link
Owner

@kepi kepi commented May 10, 2013

That's the right question to ask! :) I'm really not sure how I end with 33 instead of 38 but I'll fix it right away. No need to downscale icon, I have vector version so it is better to use it.

@kepi kepi reopened this May 10, 2013
@kepi kepi closed this in 24c0fe2 May 10, 2013
@kepi
Copy link
Owner

@kepi kepi commented May 10, 2013

Just published on web store, let me know in case of any trouble. Thanks for reporting.

@iainbeeston
Copy link
Author

@iainbeeston iainbeeston commented May 12, 2013

Just wanted to let you know it looks great now! Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.