Come together with the global Drupal community in Rotterdam, 28 Sept – 1 Oct 2026. Sessions, contribution, connection, and Early Bird savings until 8 June.Problem/Motivation
Body scroll lock is pretty common task in frontend development. And may be required in different components. And not only Dialog.
Now we have library for this:
assets/vendor/tua-body-scroll-lock/tua-bsl.umd.min.js
But it can be used only as part of drupal.dialog library.
In this issue https://www.drupal.org/project/navigation/issues/3392922#comment-15283105
we need it as is. And i also can imagine another cases.
Proposed resolution
Expand bodyscrolllock as separated library
Issue fork drupal-3395797
Show commands
Start within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
changes
Comments
Comment #2
finnsky commentedComment #3
finnsky commentedComment #4
finnsky commentedComment #6
smustgrave commentedThe idea of separate library makes sense, especially if it can be used by others.
Only question I have is if it will need a change record for announcing the new library.
Comment #7
nod_Discussed at drupalcon, the library was a stopgap measure.
An alternative solution with CSS is explored for this in the navigation module, if all works well we'll replicate the same thing in core and potentially drop the library #3392922: Broken mobile toolbar on pages with non-responsive tables.