Broken mobile toolbar on pages with non-responsive tables.

From my point of view this kind of change should be at least scoped in:

1. Only mobile breakpoint
2. Only tables in content area.

Also probably we need to use
https://www.npmjs.com/package/tua-body-scroll-lock

It is part of core libraries. So can be useful here.

@finnsky, I have just pushed what could be a solution.
Can you please review it in https://git.drupalcode.org/project/navigation/-/merge_requests/100.diff?

In general, when the user has zoomed out in ex. /admin/content and the navigation menu gets expanded, the tables in the body get limited to the viewport's width which actually causes something like a zoom-in, though not exactly.

vensires → closed merge request !100

Tested and working as intended

Re: #5

I think it is not really secure change.

1.

Regarding point (3) we can't really find a more specific selector.

The default /admin/content provides the following HTML:

<table class="views-table views-view-table cols-7 responsive-enabled sticky-enabled" data-once="tableresponsive table-select">

and we also have its sticky header:

<table class="sticky-header" style="visibility: hidden; position: fixed; top: 0px; left: 312px; width: 1425px;">

I would fix the mobile scope issue in a future commit.

MR code still contains rebase artifact.

finnsky → closed merge request !105

finnsky → reopened merge request !105

Pushed another approach with bodyscroll Lock library from core

Something wrong with Tugboat. Idk how to rebuild it.

Overflow clip for HTML only not fixing initial issue.
But together with BODY overflow clip seems it works.

Please review!

Tested and it's really working as expected! Thank you!

Automatically closed - issue fixed for 2 weeks with no activity.