material-components / material-components-web

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

[TopAppBar] scrollbar gap with short variant #4770

Open moog16 opened 5 years ago

moog16 commented 5 years ago

See https://storage.googleapis.com/mdc-web-screenshot-tests/travis/2019/05/29/19_08_53_246/report/report.html?utm_source=cli&utm_medium=test_results

Occuring in firefox 66/windows 10

Result: top app bar does not account for gap on right side of window. Expected: top app bar spans entire window.

abhiomkar commented 5 years ago

Seems like regression, Do you want to track this under #4527?

abhiomkar commented 5 years ago

Never mind. PRs related to #4527 never merged to master.

I'm not able to repro this issue on Win 10, FF 64 (64bit)?

image

moog16 commented 5 years ago

Thanks for checking! ~Working on https://github.com/material-components/material-components-web/pull/4745. Can you check after?~

I've merged this into develop branch. Ready for another check.