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

Piecon.reset() doesn't restore original favicon #6

Closed
tim-peterson opened this issue Aug 10, 2012 · 5 comments
Closed

Piecon.reset() doesn't restore original favicon #6

tim-peterson opened this issue Aug 10, 2012 · 5 comments

Comments

@tim-peterson
Copy link

I'd like to show the pie chart favicon to chart progress during a file upload but then revert to my site's original favicon when the upload is complete. Piecon.reset() just restores a partially filled pie chart when I call it when the file is done uploading.

How can I modify piecon.js such that Piecon.reset() can restore the original favicon on my site?

thanks,
tim

@lipka
Copy link
Owner

lipka commented Aug 11, 2012

This is implemented.

What browser/version are you running on?

@tim-peterson
Copy link
Author

hi @lipka, I'm using Chrome 20. Here's the html for my site's original favicon:

<link rel='shortcut icon' href='/assets/img/myLogo.png' type='image/png'>

maybe Piecon.reset() is not recognizing the mime-type and needs it to be this: type="image/x-icon"?

@tim-peterson
Copy link
Author

@lipka any thoughts on whether i've specified the mime-type incorrectly?

@crdx
Copy link

crdx commented Aug 29, 2012

If you look at the source, it's because of this line:

if (typeof(links[i]) !== 'undefined' && links[i].getAttribute('rel') === 'icon') {

It's looking for rel="icon" but yours is shortcut icon. If you made yours just icon it would work.

@tim-peterson
Copy link
Author

oh awesome, thanks for finding this, problem solved!

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

3 participants