-
Notifications
You must be signed in to change notification settings - Fork 60
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
Icons not visible #64
Comments
Could the icons be data URIs or inline SVGs instead of needing a separate download? |
Potentially, yes. But that won't solve the issue on |
You can make them absolute which tends to make development across forks/repos easier: <link href="{{site.github.url}}/css/...."> Locally |
Also while on the topic of URLs - it is preferred to avoid protocol relative URLs (that is, urls without the |
I just totally looked into the wrong file |
I commented on the PR, but why would we not want the paths to always be root-relative - ie, start with a The rationale to avoid protocol-less URLs isn't that important when gh-pages, and individual sites, can be forced onto HTTPS, but it doesn't really matter either way imo. |
When accessing the website with a bad network connection the icons are potentially invisible. The user might miss that there is a navigation icon, because the icon itself could not load.
We should provide a fallback by using native symbols.
Proposals:
Menu icon: ☰ (
☰
)Accordion caret: ▼(
▼
)Edit: they seem to be missing completely right now 🤔
Edit2:
missing on tc39.github.io/beta
visible on tc39-beta.netlify.com
The text was updated successfully, but these errors were encountered: