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.
Although it's written in the module README.txt, it's not very clear that the Range Widget depends on a the jQuery-UI Slider-Pips library to render correctly.
I propose to:
- Write it down in the module description (at least it's clear for every kind of user)
- Run a basic check on install, and optionally display an warning on the status report page
I will write down a patch if ok.
Comment | File | Size | Author |
---|---|---|---|
#3 | 2942558.patch | 1.4 KB | borisson_ |
Comments
Comment #2
borisson_Checking this on install of the submodule is a great idea. Not sure if the module description is a good idea. I like that the webform module provides a message on the status page for these kinds of libraries. I think we should do something like that as well.
In any case, thank you so much for bringing this up and offering to write a patch. Very much looking forward to seeing that.
PS: It seems you (like many others – it's really easy to misinterpret) are confused by the "Issue tags" field. As the guidelines state, they aren't meant for free text tags related to the issue, but only for specific categorization purposes, usually by module maintainers.
So, if you aren't sure your current usage is correct, please just leave the field empty.
Comment #3
borisson_Comment #4
StryKaizerLooks good, moving it to .module to show this error also in status report page, and committing
Comment #6
StryKaizer