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.
The Installation section for the main page of this module instructs you to create modernizr.min.js, but when I enabled this module it created modernizr-latest.js, which appears to work. It's not a problem, it's just a bit confusing.
Comment | File | Size | Author |
---|---|---|---|
#5 | 2395841-5-instructions.patch | 1.01 KB | rupl |
Comments
Comment #1
ruplDid you use Drush? Upon enabling the module, Drush automatically downloads the "development" copy of Modernizr from http://modernizr.com/downloads/modernizr-latest.js which contains all the available tests, but does NOT contain Modernizr.load(). If you go to
/admin/config/development/modernizr
you will see a page which allows you to build a much smaller, customized copy of Modernizr that also includes Modernizr.load()Comment #2
youngelpaso CreditAttribution: youngelpaso commentedI second that, I had the same experience.
Comment #3
ruplTo be honest I need some direction on what it should say. I wrote the module and have had an extremely clear understanding of how it works since the beginning. So if you provide me with the instructions you want to see (or patch them in!) then I'm happy to review and make the installation process more helpful.
Comment #4
versantus.nik CreditAttribution: versantus.nik as a volunteer commentedHow about rather than
"You don't seem to have a custom build of Modernizr installed yet. This page will help generate one for you."
say:
"You don't seem to have a custom build of Modernizr installed yet, or we have detected the wrong version. Remove any existing modernizer-latest.js or modernizer custom builds from sites/all/libraries and generate a new one using the instructions below."
Comment #5
ruplThanks for the feedback, how is this?
Comment #6
versantus.nik CreditAttribution: versantus.nik as a volunteer commentedThat wording looks good and the patch worked for me.
Comment #8
ruplThanks for the review!