Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[RaisedButton] 0.18.4 adds overflow:hidden; style to containerElement, breaks layout #7256

Closed
Floriferous opened this issue Jun 26, 2017 · 2 comments · Fixed by #7262
Closed

[RaisedButton] 0.18.4 adds overflow:hidden; style to containerElement, breaks layout #7256

Floriferous opened this issue Jun 26, 2017 · 2 comments · Fixed by #7262
Assignees
@oliviertassinari
Labels
component: button This is the name of the generic UI component, not the React module!

Comments

@Floriferous
Copy link
Contributor

Floriferous commented Jun 26, 2017 •
edited

The latest release 0.18.4 adds overflow: hidden; to a containerElement on RaisedButton. This is not happening with 0.18.3. It increases the height of the button and displays a white border at the bottom of the button,

Also checked FlatButton, which still works properly.

Here is a link to a reproduction:
https://www.webpackbin.com/bins/-KnZlC-OMVsBCErCmgGD

EDIT: for convenience, here is a link to webpackbin with 0.18.3
https://www.webpackbin.com/bins/-KnZuTGm_5ZPX8yRYMgR
@Floriferous
Copy link
Contributor Author

#7158 introduced this

@Stupidism
Copy link

Stupidism commented Jun 26, 2017 •
edited

And this bug is in online document http://www.material-ui.com/#/components/raised-button
image

@oliviertassinari oliviertassinari self-assigned this Jun 27, 2017
This was referenced Jun 27, 2017
Merged
Closed
@oliviertassinari oliviertassinari added the status: waiting for maintainer These issues haven't been looked at yet by a maintainer label Dec 21, 2022
@zannager zannager added component: button This is the name of the generic UI component, not the React module! and removed status: waiting for maintainer These issues haven't been looked at yet by a maintainer labels Jan 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@oliviertassinari oliviertassinari

Labels
component: button This is the name of the generic UI component, not the React module!
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix regressions
4 participants
@oliviertassinari @Stupidism @Floriferous @zannager