Skip to content
This repository has been archived by the owner on Sep 5, 2024. It is now read-only.

Bottomsheet in 1.0.6 generates scrollbar (both chrome and IE) #7410

Closed
jalajc opened this issue Mar 4, 2016 · 5 comments
Closed

Bottomsheet in 1.0.6 generates scrollbar (both chrome and IE) #7410

jalajc opened this issue Mar 4, 2016 · 5 comments
Assignees
Labels
P1: urgent Urgent issues that should be addressed in the next minor or patch release.
Milestone

Comments

@jalajc
Copy link

jalajc commented Mar 4, 2016

I observe that bottomsheet in 1.0.6 generates the scrollbar in Chrome and IE (in IE both horizontal as well vertical scroll bar appears).
When I replace back angular material css to 1.0.5, the scrollbar disappears.

refer pen: http://codepen.io/anon/pen/pyjwNJ (codepen from https://material.angularjs.org/latest/demo/bottomSheet)

Just try changing the css from codepen settings to observe this behavior

@devversion
Copy link
Member

The current issue is now noticeable, because we removed the overflow property from the body for codepens. So yes, can reproduce.

@topherfangio topherfangio added the P1: urgent Urgent issues that should be addressed in the next minor or patch release. label Mar 17, 2016
@mortware
Copy link

+1

@ThomasBurleson
Copy link
Contributor

This issue is closed as part of our ‘Surge Focus on Material 2' efforts.
For details, see our forum posting @ http://bit.ly/1UhZyWs.

@sjlynch
Copy link

sjlynch commented Mar 5, 2017

For anyone running into this issue --

Found a good fix for this by setting the parent of md-bottom-sheet to a div which is relatively positioned and overflow:hidden. See http://stackoverflow.com/questions/19308257/positionabsolute-causes-horizontal-scrollbar

To set the parent, see the bottom of this page -- #640

@Marco-Ishiguro
Copy link

#1211

md-bottom-sheet {
padding: 0px;
bottom: 80px;
overflow: hidden;
}

Should save the day, only tested on safari iPhone, chrome iPhone and chrome desktop but it should also work for the rest.

The value 80px need adjustment

This issue was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
P1: urgent Urgent issues that should be addressed in the next minor or patch release.
Projects
None yet
Development

No branches or pull requests

7 participants