-
Notifications
You must be signed in to change notification settings - Fork 208
Closed
Labels
enhancementNew feature or requestNew feature or request
Description
Context
Sphinx 6.0 is going to remove the jquery dependency by default. We use jquery a little bit in this theme, though likely not in a way that strictly requires jquery.
Here's a GitHub CS search for the $( symbol
Note that our parent theme removed JQuery as a dependency here:
Proposal
We should remove all of our code that depends on JQuery, and only manually add the JQuery dependency if we really need it.
Tasks and updates
- Remove instances of
$(in the theme as well as JQuery notes anywhere - Remove
tooltipfunctionality and popper.js - Remove the styling of
div.tooltipsince we no longer use Bootstrap for this - Double-check functionality is the same
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
enhancementNew feature or requestNew feature or request