This repository has been archived by the owner on Jan 13, 2025. It is now read-only.

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

Closed

mindaugasvcs opened this issue May 10, 2024 · 0 comments

Labels

Unresolved (Archived)

mindaugasvcs commented May 10, 2024

material-components-web/packages/mdc-top-app-bar/mdc-top-app-bar.scss

Line 204 in f80ac92

padding-top: variables.$row-height;

CSS margin is used to push element.
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.

asyncLiz added the Unresolved (Archived) label

asyncLiz closed this as completed

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.