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.
If possible, can another install location be allowed next to /libraries. E.g. to be able to include colorbox in an install profile, without additional requirements on the site including the install profile (e.g. adapt composer.json).
This has been done in dropzonejs for reference.
Comment | File | Size | Author |
---|---|---|---|
#5 | allow_other_install-2842189-5.patch | 823 bytes | geophysicist |
#4 | allow_other_install-2842189-4.patch | 957 bytes | daften |
|
Comments
Comment #2
daften CreditAttribution: daften at Picabit commentedComment #3
daften CreditAttribution: daften at Picabit commentedShould the library definition be updated to be defined with https://api.drupal.org/api/drupal/core%21lib%21Drupal%21Core%21Render%21... ?
Comment #4
daften CreditAttribution: daften at Picabit commentedA patch for the proposed changes.
Comment #5
geophysicist CreditAttribution: geophysicist as a volunteer commentedComment #6
daften CreditAttribution: daften at Picabit commentedThere is no dependency on libraries for colorbox, so the latest patch is not OK IMO.
Libraries support should be in a separate issue and should use libraries to the fullest. Or there should be an adaptation in the README file which explains that other locations are supported, but only with the libraries module installed.
Also, minified => false is the default, so doesn't need to be added explicitly.
Comment #7
Anonymous (not verified) CreditAttribution: Anonymous at Cheeky Monkey Media commentedComment #8
Neslee Canil Pinto