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

favicon gone missing #491

Closed
allenwb opened this Issue Mar 25, 2016 · 2 comments

Comments

Projects
None yet
3 participants
@allenwb
Member

allenwb commented Mar 25, 2016

http://tc39.github.io/ecma262/ seems to have lost it's Ecma favicon.

(It actually might be wise to not use the official Ecma favicon on drafts like this, but something to make it standout in a roll of tabs would be nice.)

@mathiasbynens

This comment has been minimized.

Show comment
Hide comment
@mathiasbynens

mathiasbynens Mar 25, 2016

Member

https://tc39.github.io/ecma262/ has <link rel="icon" href="favicon.ico"> in its source but https://tc39.github.io/ecma262/favicon.ico seems to be missing, indeed.

Update: As @alrra points out, the icon was moved to img/favicon.ico. The reference should be updated.

Member

mathiasbynens commented Mar 25, 2016

https://tc39.github.io/ecma262/ has <link rel="icon" href="favicon.ico"> in its source but https://tc39.github.io/ecma262/favicon.ico seems to be missing, indeed.

Update: As @alrra points out, the icon was moved to img/favicon.ico. The reference should be updated.

@alrra

This comment has been minimized.

Show comment
Hide comment
@alrra

alrra Mar 25, 2016

Contributor

https://tc39.github.io/ecma262/ has in its source but https://tc39.github.io/ecma262/favicon.ico seems to be missing, indeed.

That's because the favicon.ico is now in the img/ directory.

The reference should be updated.

I've opened #493.

Contributor

alrra commented Mar 25, 2016

https://tc39.github.io/ecma262/ has in its source but https://tc39.github.io/ecma262/favicon.ico seems to be missing, indeed.

That's because the favicon.ico is now in the img/ directory.

The reference should be updated.

I've opened #493.

@bterlson bterlson closed this in d62ef7f Mar 25, 2016

bterlson added a commit that referenced this issue Mar 25, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment