-
Notifications
You must be signed in to change notification settings - Fork 316
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
Replace jQuery-based scripts with vanilla JS #361
Conversation
Signed-off-by: Takuya Noguchi <takninnovationresearch@gmail.com>
Test on Travis-ci |
@sonots What do you think? |
Comments? |
Thoughts? @sonots |
@tomlea Are you still in charge of this gem? See https://guides.rubygems.org/contributing/ updated in rubygems/guides#222 (in 2018) |
@tnir I think it makes sense. I'm usually not the one merging requests here, but since this is not even ruby related and no one's replying I'll give it a go. Thanks! |
Thanks a lot for resolving this security issue on this project, @rikas! |
Fix lints in PR 361: geminabox#361 Signed-off-by: Takuya Noguchi <takninnovationresearch@gmail.com>
Fix lints in PR 361: #361 Signed-off-by: Takuya Noguchi <takninnovationresearch@gmail.com>
jQuery 1.5 is so legacy and insecure.
This PR will drop IE6-8 support from the supported browsers (not tested if this PR actually drops IE6-8 support).
Closes #362
Signed-off-by: Takuya Noguchi takninnovationresearch@gmail.com