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.

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

[Modal] Add a keepMounted property #7072

Merged
merged 1 commit into from
Jun 8, 2017
Merged

[Modal] Add a keepMounted property #7072

merged 1 commit into from
Jun 8, 2017

Conversation

oliviertassinari
Copy link
Member

@oliviertassinari oliviertassinari commented Jun 7, 2017

  1. So we have the top level information in the index of the search provided by Algolia:
    That should result in a much better search experience.

Expected

unnamed 1
Actual

unnamed

  1. So we maximise the responsiveness of the Drawer in the documentation. We save around 100ms by using the keepMounted option on a high-end device. That should definitely be perceivable by any user. @nathanmarks That's something we have been discussing a long time ago :).

Before
capture d ecran 2017-06-07 a 21 34 14

After
capture d ecran 2017-06-07 a 21 35 36

  • PR has tests / docs demo, and is linted.
  • Commit and PR titles begin with [ComponentName], and are in imperative form: "[Component] Fix leaky abstraction".
  • Description explains the issue / use-case resolved, and auto-closes the related issue(s) (http://tr.im/vFqem).

@oliviertassinari oliviertassinari added new feature New feature or request next labels Jun 7, 2017
@oliviertassinari oliviertassinari merged commit 62f6801 into mui:next Jun 8, 2017
@oliviertassinari oliviertassinari deleted the drawer-keep-in-dom branch June 8, 2017 07:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new feature New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant