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.
JS file misc/ajax_screen_lock.js uses tabs instead of 2 spaces.
Plus it is not possible to override its behavior cause ajaxScreenLock variable exists only locally in this file.
I will provide a patch.
Comment | File | Size | Author |
---|---|---|---|
#2 | ajax_screen_lock-js_fixes-2705415-2-7.patch | 3.99 KB | duozersk |
Comments
Comment #2
duozerskHere is a patch fixing code standard and changing ajaxScreenLock to Drupal.ajaxScreenLock
With these changes you can do the following in your custom code (I have changed the css value passed to the blockUI plugin):
Thanks
AndyB
Comment #3
nijk CreditAttribution: nijk commentedI see the need to override the internal workings of this module, particularly in order to control the options passed to the jQuery.blockUI plugin.
Personally I would rather see the module implement proper Drupal JS behaviours and extend the usage of Drupal JS settings to allow admin form control over these options, particularly the css property. There are plenty to support, see: http://jquery.malsup.com/block/#options
Comment #5
eugene.ilyin CreditAttribution: eugene.ilyin as a volunteer and at DrupalJedi commentedProblem with indent symbols was not only in JS files. Corrected.
Comment #6
eugene.ilyin CreditAttribution: eugene.ilyin as a volunteer and at DrupalJedi commented