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

ListItem Joy - hover effect not applied in beta as it was in alpha #38404

Closed
2 tasks done
Icehunter opened this issue Aug 10, 2023 · 1 comment
Closed
2 tasks done

ListItem Joy - hover effect not applied in beta as it was in alpha #38404

Icehunter opened this issue Aug 10, 2023 · 1 comment
Assignees
Labels
component: list This is the name of the generic UI component, not the React module! package: joy-ui Specific to @mui/joy status: expected behavior Does not imply the behavior is intended. Just that we know about it and can't work around it

Comments

@Icehunter
Copy link

Duplicates

  • I have searched the existing issues

Latest version

  • I have tested the latest version

Steps to reproduce 🕹

Link to live example:
https://codesandbox.io/p/sandbox/wispy-bush-nz49xs

Steps:

  1. install latest joy
  2. use baseline example in docs https://mui.com/joy-ui/react-list/#actionable
  3. UI doesn't have noticable hover like in docs pages

This extends to other components as well, but after the theme changes in beta.0, many items don't seem to have the correct styling.

You can see how the alpha looks here:

https://codesandbox.io/p/sandbox/lively-monad-8chdhh

Current behavior 😯

UI has limited visible hover/selected change on some components.

Expected behavior 🤔

UI state should show hover/selected states correctly.

Context 🔦

No response

Your environment 🌎

npx @mui/envinfo
  Don't forget to mention which browser you used.
  Output from `npx @mui/envinfo` goes here.
@Icehunter Icehunter added the status: waiting for maintainer These issues haven't been looked at yet by a maintainer label Aug 10, 2023
@zannager zannager added component: list This is the name of the generic UI component, not the React module! package: joy-ui Specific to @mui/joy labels Aug 10, 2023
@siriwatknp
Copy link
Member

Thanks for reporting the issue but the hover background is applied. It's just that the color is the same as the body background. If you wrap the list with Sheet component or a white background container, you should see the hover background.

image

Here is the proof:

Screen.Recording.2566-08-10.at.17.21.58.mov

@siriwatknp siriwatknp added status: expected behavior Does not imply the behavior is intended. Just that we know about it and can't work around it and removed status: waiting for maintainer These issues haven't been looked at yet by a maintainer labels Aug 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: list This is the name of the generic UI component, not the React module! package: joy-ui Specific to @mui/joy status: expected behavior Does not imply the behavior is intended. Just that we know about it and can't work around it
Projects
None yet
Development

No branches or pull requests

3 participants