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

Use MouseEnter/MouseLeave instead of MouseOver/MouseOut #1189

Closed
cgestes opened this issue Jul 17, 2015 · 3 comments
Closed

Use MouseEnter/MouseLeave instead of MouseOver/MouseOut #1189

cgestes opened this issue Jul 17, 2015 · 3 comments
Labels
performance

Comments

@cgestes
Copy link
Contributor

cgestes commented Jul 17, 2015

This applies to a lot of components.

This idea is to improve performances by avoiding useless setState.

Let's take the ListItem example.
If a child component is hovered then the hovered state of ListItem switches back and forth even if the list item is still hovered. This creates a lot of useless rerender.
@cgestes
Copy link
Contributor Author

cgestes commented Jul 17, 2015

React support: https://facebook.github.io/react/docs/events.html

@cgestes
Copy link
Contributor Author

cgestes commented Jul 17, 2015

see #1190

@shaurya947
Copy link
Contributor

#1190 was merged

mnajdova pushed a commit to mnajdova/material-ui that referenced this issue Nov 10, 2020
@igornfaustino @dmtrKovalenko
[docs] Add redux form integration example (mui#1189)
4ef9a9f 
* install redux and redux-form

* configure redux-form

* wrap App with redux provider

* create redux form example

* add react-redux types to dev dependencies

* Update docs/pages/guides/form-integration.mdx

Co-Authored-By: Dmitriy Kovalenko <dmtr.kovalenko@outlook.com>

* move redux to example folder

* Wrap example with redux

* remove redux-thunk

* modify onChange to allow empty field

in the other way, the field always gotten the last valid value, even when explicit deleted.
This way, the user can clear the field!

* move store to inside Form example
This was referenced Apr 13, 2021
Open
Open
This was referenced Apr 26, 2021
Open
Open
Open
Open
@zhaleTaheri zhaleTaheri mentioned this issue Oct 1, 2023
Open
@akashmitra akashmitra mentioned this issue Oct 1, 2023
Open
@hzhz hzhz mentioned this issue Oct 1, 2023
Open
@maniator maniator mentioned this issue Oct 1, 2023
Open
@hzhz hzhz mentioned this issue Oct 2, 2023
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
performance
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@cgestes @shaurya947 @zannager