Upgrade vendored jQuery UI to 1.14.2#8670
Conversation
5e4e6c8 to
e98b191
Compare
|
@tobischo despite the tab component seems to work at http://localhost:3000/admin/kitchensink, there is a legit failure in cucumber specs I will keep investigating, but if you know the reason feel free to provide information on how to fix |
|
This is actually an issue with activeadmin and should be related to #8616
Actually GitHub does the same for wiki pages, and the behavior works Reproducible test case: |
|
Reported here: jquery/jquery-ui#2344 |
|
Admittedly, this is part of the tests I didn't run before, since it got properly triggered only when I created a PR onto my own repo as well earlier today. We are not using tabs, so it didn't effect the "production test" for us. |
|
Acknowledged as an issue, waiting for 1.14.2 |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## 3-0-stable #8670 +/- ##
=============================================
Coverage ? 99.19%
=============================================
Files ? 194
Lines ? 4971
Branches ? 0
=============================================
Hits ? 4931
Misses ? 40
Partials ? 0 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
I'm going to close here for the moment. I will reopen when 1.14.2 will be released. The effort of downgrading to 1.14.0 and adding rules to package.json is not worth it, because there are other solutions (use I suggest to fork and use this branch on a fork, because I may force push here or delete the branch |
7aa7778 to
30f8d11
Compare
94a8fd8 to
95b039c
Compare
|
There seems to be a little regression against jQuery 4: Datepicker at http://localhost:3000/admin/categories?order=name_ascIt is not being triggered after the first |
|
probably blocked again by jquery/jquery-ui#2385 |
|
But probably we can merge the same, since jQuery 4 is not supported by the current jquery-rails, and it is optional |
|
I explained in jquery/jquery-ui#2385 why it's more an issue in code using jQuery UI, not in jQuery UI or jQuery itself. |
JavaScript files have been manually updated because: - `jquery-ui-rails` is unmaintained and it is bundling 1.13.0 - An active fork of `jquery-ui-rails` is bundling 1.13.2, but there is a more recent version (1.13.3) Files removed because they have been dropped from jQuery UI: - form.js - ie.js - safe-active-element.js - safe-blur.js Additionally: - Allow jQuery 4 - Bump minimum required jQuery to 3.7.1 - Fix an issue with Datepicker not being initialized correctly in jQuery 4 (See jquery/jquery-ui#2385 for more information) Refs: - #8656 - https://jqueryui.com/upgrade-guide/1.14/#removed-apis - jquery/jquery-ui#2385
95b039c to
033ff95
Compare

JavaScript files have been manually updated because:
jquery-ui-railsis unmaintained and it is bundling 1.13.0jquery-ui-railsis bundling 1.13.2, but there isa more recent version (1.13.3)
Files removed because they have been dropped from jQuery UI:
Additionally:
jQuery 4 (See Datepicker created on focus does not open on initial focus with jQuery 4.0.0 (opens after blur/refocus) jquery/jquery-ui#2385 for more information)
Refs:
Manual tests checklist:
$.ui.versionVersion in consolejQuery 4 manual tests
Tested against rails/jquery-rails#307
$.fn.jquery$.ui.versionVersion in console