max-mapper / max-mapper/javascript-for-cats
Use favicon to avoid error message
Nobody has claimed this yet.
- Dominant language
- JavaScript
- Stars
- 794
- Forks
- 139
- PR merge metrics
- No merged PRs in 30d
Description
The site's lack of a `favicon` generates a 404 error in the JS console:
Since once of the tutorial's first steps is to open the console, the red error message is shaping first-time students' experience. In fact, I'm reporting this issue on behalf of a first-time student who encountered the error and assumed the rest of the tutorial would not work.
In #55 it was noted that the site uses GitHub Pages for hosting. It looks like [custom favicons are possible even with GitHub Pages](https://stackoverflow.com/a/42966673):
```
```
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Inspect the site's HTML entry point and confirm how GitHub Pages serves its root assets. Add the referenced favicon link and a favicon.ico asset, then open the tutorial and browser console to verify that the favicon request succeeds without the 404 error.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- javascript
- Domain
- documentation
- Issue type
- Feature
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 55/100