Change Details

Currently in the codebase, we have callsites where we're using the useMemo hook with the classNames library. This is unnecessary as useMemo returns a memoized value, which in this case is always a string, which we don't see as being computationally expensive, and hence can run on every render.

Currently in the codebase, we have callsites where we're using the useMemo hook with the classNames library. This is unnecessary as useMemo returns a memoized value, which in this case is just a string and hence safe to run on every render. https://linear.app/comm/issue/ENG-2287/[web]-do-not-wrap-classnames-in-reactusememo

Currently in the codebase, we have callsites where we're using the useMemo hook with the classNames library. This is unnecessary as useMemo returns a memoized value, which in this case is just a string and hence safe to run on every render. always a string, which we don't see as being computationally expensive, and hence can run on every render.https://linear.app/comm/issue/ENG-2287/[web]-do-not-wrap-classnames-in-reactusememo