Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Collapse has a feature detection script that is now in modernizr (we didn't have modernizr when the details patch was committed).
It fixes the script dependencies and adds the global Modernizr object to the jshint configuration.
There were issues with the details styling and the use of jQuery .prop to deal with the open attribute. (makes the whole thing behave strangely, try the module page on firefox).
Comment | File | Size | Author |
---|---|---|---|
#7 | core-js-clean-collapse-1975426-7.patch | 4.31 KB | nod_ |
#2 | core-js-clean-collapse-1975426-2.patch | 4.06 KB | nod_ |
core-js-clean-collapse.patch | 3.51 KB | nod_ | |
Comments
Comment #1
nod_tag
Comment #2
nod_reroll after #1976964: Add Modernizr to the .jshintrc file
Comment #4
nod_#2: core-js-clean-collapse-1975426-2.patch queued for re-testing.
Comment #5
Wim LeersManually tested in Chrome and Firefox (which doesn't support
<details>
), on the modules page and at node/1/edit. Works as advertised.Comment #6
alexpottif we are truly cleaning up collapse.js let's fix this too...
Comment #7
nod_:D
Comment #8
Wim Leers:)
Comment #9
alexpottCommitted 3b5fd77 and pushed to 8.x. Thanks!