bug(sidenav/drawer/focustrap): over/push mode do not remove tabindex="0" when drawer closed

[krzysztofpiotrow]

Is this a regression?

• Yes, this behavior used to work in the previous version

The previous version in which this bug was not present was

15

Description

When using mat-sidenav or mat-drawer with mode over or push (and might need backdrop=true) there is a div element:
<div class="cdk-visually-hidden cdk-focus-trap-anchor" aria-hidden="true"></div>
which always has tabindex="0" added to it but should get it only when the sidenav/drawer is opened.
As a result tabbing through the page is confusing.

Reproduction

StackBlitz link: https://stackblitz.com/edit/e4bsuu?file=package.json
StackBlitz was forked from the second example here: SideNav Examples
Steps to reproduce:

1. Select Sidenav mode: Over, Has backdrop
2. Via DevTools check that the mentioned elements have tabindex="0".

Expected Behavior

tabindex="0" is only added when the sidenav/drawer is opened.

Actual Behavior

tabindex="0" is always there and makes tabbing through the page confusing.

Environment

Angular: 16.1.1
CDK/Material: 16.2.0
Browser(s): Chrome
Operating System (e.g. Windows, macOS, Ubuntu): Windows

[krzysztofpiotrow]

Seems like this a duplicate: #27911

[angular-automatic-lock-bot]

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

_{This action has been performed automatically by a bot.}