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.

Sign up for GitHub

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

Jump to bottom

JavaScript cleanup #213

Open
duckinator opened this issue Dec 30, 2024 · 0 comments · May be fixed by #215
Open

JavaScript cleanup #213

duckinator opened this issue Dec 30, 2024 · 0 comments · May be fixed by #215

Comments

@duckinator
Copy link
Member

duckinator commented Dec 30, 2024 •
edited
Loading

I noticed there's a lot of room for improvement regarding our JavaScript usage.

Drop IE8 support

IE 10 support ended in January 2025 — half a decade ago — so I don't think we should bother supporting things that are even older than that.

Dropping IE8 support means we can remove the following:

  • modernizr
  • selectivizr
  • html5shiv
  • ie8.css

Avoid using JS for navigation

Our navigation on mobile uses JavaScript. We should migrate to a CSS-only implementation — it can be done more simply, and without JavaScript.

Split out to #214.

Avoid loading things twice

For some reason, application.js is loaded twice. We should stop doing that.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Remove IE8-specific kludges and don't load application.js twice rubygems/rubygems.github.io
1 participant
@duckinator