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.on the /admin/config/media/colorbox page:
Advanced Settings
Mobile detection
On
Off
If on (default) Colorbox will not be active for devices with a the max width set below.
Emphasis is on the incorrect verbiage.
Maybe read "devices with the max width" or "devices with a max width" Looks like a simple oversight to me.
I apologize for not being able to create a proper patch, that is beyond my skill level.
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | Colorbox-Fixed_typo_on_admin_settings-2593101-3.patch | 995 bytes | nupur.lohokare |
Comments
Comment #2
nupur.lohokare CreditAttribution: nupur.lohokare as a volunteer and at Faichi Solutions Pvt Ltd commentedComment #3
nupur.lohokare CreditAttribution: nupur.lohokare as a volunteer and at Faichi Solutions Pvt Ltd commentedHi philsward,
Found the issue you mentioned and modified the message to -
'If On (default), Colorbox will not be active for devices with the max width set below.'
Thanks
Nupur
Comment #4
gaurav.bajpai CreditAttribution: gaurav.bajpai as a volunteer and at Faichi Solutions Pvt Ltd commentedHi nupur.lohokare,
I checked your patch and looks good.
Thanks,
Gaurav
Comment #5
philsward CreditAttribution: philsward commentedApplied nupur.lohokare patch with no errors. Looks good to me too.
Comment #7
frjo CreditAttribution: frjo commentedCommitted to 7.x-2.x.
Thank you for helping to improve the Colorbox module!