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.
Working on this to make colorbox work with drupal 8 beta 1.
Some questions:
Can i require the colorbox 1.5.14 as minimun for drupal 8? I do not see a reason to use one older.
I will move the code from ColorboxSubriscriber to hook_page_build, it is the correct place for code that needs to add javascript, ok ?
I plan to make a minimum working patch, just to it work on beta 1, after this work on other parts, like fields, views and drush.
Comments
Comment #1
frjo CreditAttribution: frjo commentedThanks for helping out!
Unsure where to best add js in D8. What I do is normally to have a look how core and other contrib modules does it.
Comment #2
Sam152 CreditAttribution: Sam152 as a volunteer and at PreviousNext commented