material-components / material-components-web

Modular and customizable Material Design UI components for the web
https://material.io/develop/web
MIT License
17.11k stars 2.15k forks source link

CSS margin-top should be used instead of padding-top in mdc-top-app-bar--fixed-adjust class #8186

Open mindaugasvcs opened 1 month ago

mindaugasvcs commented 1 month ago

https://github.com/material-components/material-components-web/blob/f80ac92b08dfa1b59cd9faf74f3d19a4b134993e/packages/mdc-top-app-bar/mdc-top-app-bar.scss#L204

  1. CSS margin is used to push element.
  2. CSS padding is used to push element's boundaries.

Now if I use the drawer with top-app-bar then drawer's border will be visible on the top-app-bar.