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

With Version 0.6 the Favicon looks like it is zoomed #47

Open
nbartels opened this issue Sep 25, 2013 · 10 comments
Open

With Version 0.6 the Favicon looks like it is zoomed #47

nbartels opened this issue Sep 25, 2013 · 10 comments

Comments

@nbartels
Copy link

With Version 0.5 everything was fine and after switching to the new 0.6 I have a problem with the favicon. As soon as I call the setBubble Method the favicon looks like someone zoomed in. You can see only a part and even see the single pixels of the original favicon.

I cannot provide a screenshot for legal reasons - at least not on this public way :)

@ben-eb
Copy link

ben-eb commented Oct 8, 2013

I'm also getting this behaviour: I've attached a screenshot showing the 0.6 behaviour alongside the 0.5 behaviour and the favicon I'm using for reference. I use https://github.com/gleero/grunt-favicons to generate the favicon if that helps.

tinycon-glitch

@romanr
Copy link

romanr commented Oct 8, 2013

I am also having this problem, on Retina it looks even more weird

@cantonweb
Copy link

@ben-eb look at your third icon with the "23" count in it (the 0.5 version). Look at how ragged the icon is in comparison to the second one (without the bubble). That is the issue that made me discontinue using this script. Looks like 0.6 just makes it worse with the zoomed favicon. I'm hoping this script can be fixed since I like it, which is why I'm lurking. :)

@ben-eb
Copy link

ben-eb commented Oct 8, 2013

@cantonweb Yeah, I noticed that too. I also like the script and the idea, it's lightweight and flexible, plus it's a useful addition if coding notifications/messaging in your app. Ideally it needs the best of both worlds; the better aliasing from 0.6 but the icon behaviour of 0.5. :-)

@tommoor
Copy link
Owner

tommoor commented Oct 8, 2013

Hey gang, unfortunately I don't have a retina screen to repro this..
perhaps you could take a look?
On 8 Oct 2013 21:28, "Ben Briggs" notifications@github.com wrote:

@cantonweb https://github.com/cantonweb Yeah, I noticed that too. I
also like the script and the idea, it's lightweight and flexible, plus it's
a useful addition if coding notifications/messaging in your app. Ideally it
needs the best of both worlds; the better aliasing from 0.6 but the icon
behaviour of 0.5. :-)


Reply to this email directly or view it on GitHubhttps://github.com//issues/47#issuecomment-25924586
.

@cantonweb
Copy link

@tommoor I am seeing this behavior on both a desktop and laptop computer in Chrome (Win8), so I'm thinking you may be able to see it as well, without needing any specific device. I specialize in other technologies, and with JS not being my forte, I wish I could contribute more...

@tommoor
Copy link
Owner

tommoor commented Oct 11, 2013

Any chance this is fixed with 0.6.1?

@ben-eb
Copy link

ben-eb commented Oct 11, 2013

Anti-aliasing isn't great still, but the icon is no longer stretched. 👍

screen shot 2013-10-11 at 23 50 53

@tommoor
Copy link
Owner

tommoor commented Oct 11, 2013

Fantastic - it's a shame about the antialiasing, does your site only provide the large size favicon? Could be that we could draw from a 16px icon where available or something like that...

@ben-eb
Copy link

ben-eb commented Oct 12, 2013

My favicon is dynamically generated from an SVG which is then rasterized and sent to grunt-favicons - I think that compiles 16x16, 32x32 and 48x48 sizes in the one file. I don't really have a clue about how .ico works though I'm afraid!

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

No branches or pull requests

5 participants