Add getValue method for getting value when we need inside an event

[behnammodi]

handleChange just redefines when onChange change

Also, we can use useStateRef where we need state's value inside an event that we won't redefines by useCallback

[mui-pr-bot]

Details of bundle changes (experimental)

Generated by 🚫 dangerJS against bd032f7

[eps1lon]

Could you expand a bit with concrete examples why this change is needed?

The pattern introduced in useStateRef is not safe in concurrent rendering and React 18 is now in sight. I suspect that there's a problem you have with React in general so it might be better to open a React RFC. Because right now, I can't see how this is a Material-UI issue.

[behnammodi]

Could you expand a bit with concrete examples why this change is needed?

The pattern introduced in useStateRef is not safe in concurrent rendering and React 18 is now in sight. I suspect that there's a problem you have with React in general so it might be better to open a React RFC. Because right now, I can't see how this is a Material-UI issue.

Thanks for the review, this is not an issue, this is an improvement for reducing execution of useCallback, useMemo and also useEffect,...

ex:

const [count, setCount] = useState();

useEffect(() => {
    const handleVisibiltuy = () => {
      console.log(count)
    }

   document.addEventListener('visibilitychange', handleVisibiltuy);
   
   return () =>  document.removeEventListener('visibilitychange', handleVisibiltuy);
}, [count]);

or:

const [count, setCount, getCount] = useStateRef();

useEffect(() => {
    const handleVisibiltuy = () => {
      console.log(getCount())
    }

   document.addEventListener('visibilitychange', handleVisibiltuy);
   
   return () =>  document.removeEventListener('visibilitychange', handleVisibiltuy);
}, []);

Which one is better you think?

Also your point is can be good, I will open a RFC inside React

[eps1lon]

Which one is better you think?

The first one since it's safe with concurrent rendering.

Though it's clear now that this is an enhancement you'd wish to see in React. We are not equipped to run such experiments.

There's already a discussion about this topic on the React repo: facebook/react#14543

[behnammodi]

All right, I opened another facebook/react#22163