classtranscribe / classtranscribe/FrontEnd
Remove jquery
- Dominant language
- JavaScript
- Stars
- 24
- Forks
- 29
- PR merge metrics
- No merged PRs in 30d
Description
A search suggests it appears jquery is only used in
./src/screens/Admin/Instructors/InstructorList.jsx
./src/screens/Watch/Utils/keydown.control.js
as a wrapper for getElementById. Once it is no longer needed it remove it from package.json and yarn.lock.
A simple fix would be to replace jquery with native functions. However working directly with html DOM is not the react way of doing things. Is there a better way?
When fixing this, it would be important to fully-understand (and manually check that the arrow key behavior is unchanged) the focus / UI behavior in keydown.control.js i.e.,
Does the implemented behavior work well for users who use a screen reader and keyboard control?
Contributor guide
No contributing guide indexed for this repository
Assessment
This issue has not been assessed yet.