fix(material/list): nav list item border-radius

[wagnermaciel]

• https://m2.material.io/components/navigation-drawer#states
• https://m3.material.io/components/navigation-drawer/specs

[github-actions]

Deployed dev-app for e6e6614 to: https://ng-dev-previews-comp--pr-angular-components-28789-dev-brlh8s9b.web.app

Note: As new commits are pushed to this pull request, this link is updated after the preview is rebuilt.

[crisbeto]

The list change LGTM. I wonder if we don't need to make a change to either the inner padding of the list or the sidenav since it currently looks a bit weird because it's edge-to-edge 🤔

[wagnermaciel]

The list change LGTM. I wonder if we don't need to make a change to either the inner padding of the list or the sidenav since it currently looks a bit weird because it's edge-to-edge 🤔

I was wondering this too. I think we do want this, but that change is probably so breaking that I'll want to do it in a separate PR 😭

Also, I never knew about that dev-app-preview label. That is awesome!

[crisbeto]

I think that if we scope it to only nav lists inside sidenavs on M3 it shouldn't be too breaking.

[karmasakshi]

I'm on M2 and this change has introduced border-radius to menu items. Is this expected? Screenshot from my local:

[wagnermaciel]

@karmasakshi Hi Karma, yes this is intentional. Here's what the m2 nav list (or navigation drawer as Material Design calls it) is meant to look like

https://m2.material.io/components/navigation-drawer

[karmasakshi]

Thank you, I added an overall padding to .mdc-list instead of vertical only. Looks good.

[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.}