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

[LeftNav] Only transition the transform property #2730

Merged
merged 1 commit into from
Dec 30, 2015
Merged

[LeftNav] Only transition the transform property #2730

merged 1 commit into from
Dec 30, 2015

Conversation

oliviertassinari
Copy link
Member

Fix @mbrookes comment:

It looks like when the Nav opens responsively it first appears in front of the AppBar, then pops behind it. Can that be avoided?

and #2077.

@oliviertassinari oliviertassinari mentioned this pull request Dec 30, 2015
Merged
@alitaheri
Copy link
Member

@oliviertassinari I think you can obliterate modernizr.custom now :D :D

from the webpack, package, file system and all that. This was the only place it was used

@alitaheri
Copy link
Member

Also is-browser

@oliviertassinari
Copy link
Member Author

@alitaheri Yeah, that's good 🎉

@alitaheri
Copy link
Member

+4 -313 True obliteration :D :D

Thanks a lot @oliviertassinari 👍 🎉 🎈 🎉

alitaheri added a commit that referenced this pull request Dec 30, 2015
@alitaheri
Merge pull request #2730 from oliviertassinari/left-nav-fix-transitio…
85a7cba 
…n-transform

[LeftNav] Only transition the transform property
@alitaheri alitaheri merged commit 85a7cba into mui:master Dec 30, 2015
@oliviertassinari oliviertassinari deleted the left-nav-fix-transition-transform branch December 30, 2015 20:58
@EvHaus EvHaus mentioned this pull request Jan 21, 2016
Closed
@zannager zannager added the docs Improvements or additions to the documentation label Mar 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
docs Improvements or additions to the documentation
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@oliviertassinari @alitaheri @zannager