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

Remove jQuery completely. #9

Closed
solal opened this issue Aug 18, 2020 · 2 comments · Fixed by #14
Closed

Remove jQuery completely. #9

solal opened this issue Aug 18, 2020 · 2 comments · Fixed by #14
Assignees
@solal
Labels
enhancement New feature or request
Projects
Instant Answers Extension

Comments

@solal
Copy link
Owner

solal commented Aug 18, 2020

jQuery is being used for some (not all) DOM access and manipulation.
Remove it entirely in favor of vanilla JS.
@solal solal created this issue from a note in Instant Answers Extension (To do) Aug 18, 2020
@solal solal self-assigned this Aug 18, 2020
@solal solal added the enhancement New feature or request label Aug 18, 2020
@Pustur
Copy link
Contributor

Pustur commented Aug 28, 2020

Hey, I'd like to work on this if you don't mind 🙂

@solal
Copy link
Owner Author

solal commented Aug 30, 2020

Of course I'd love to! Thanks for offering 😃
Do you want to submit a merge request?

@Pustur Pustur mentioned this issue Aug 30, 2020
Merged
@solal solal closed this as completed in #14 Aug 30, 2020
Instant Answers Extension automation moved this from To do to Done Aug 30, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@solal solal

Labels
enhancement New feature or request
Projects
Instant Answers Extension
  
Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Remove jQuery dependency
2 participants
@solal @Pustur