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

[Pagination] Fix background color on hover and keyboard focus when using CSS theme variables #39220

Merged
merged 2 commits into from
Oct 5, 2023

Conversation

ValkonX33
Copy link
Contributor

@ValkonX33 ValkonX33 commented Sep 29, 2023

Fixes #38944

Fixes the background color of the default pagination component to change to slightly dark when hovered.

Before: https://codesandbox.io/s/nervous-chaplygin-jg76jc?file=/src/mui-theme/ThemeProvider.tsx:190-201
After: https://codesandbox.io/s/keen-pare-8yrw7n?file=/src/mui-theme/ThemeProvider.tsx

@mui-bot
Copy link

mui-bot commented Sep 29, 2023

Netlify deploy preview

https://deploy-preview-39220--material-ui.netlify.app/

Bundle size report

Details of bundle changes (Toolpad)
Details of bundle changes

Generated by 🚫 dangerJS against 0f837d8

@danilo-leal danilo-leal changed the title fix #38944 [material-ui][Pagination] Fix bgColor on hover with extendedTheme Oct 1, 2023
@danilo-leal danilo-leal added package: material-ui Specific to @mui/material component: pagination This is the name of the generic UI component, not the React module! labels Oct 1, 2023
@zannager zannager requested a review from mnajdova October 2, 2023 16:30
@mj12albert mj12albert requested review from ZeeshanTamboli and removed request for mnajdova October 5, 2023 13:59
@ZeeshanTamboli ZeeshanTamboli added the bug 🐛 Something doesn't work label Oct 5, 2023
@ZeeshanTamboli ZeeshanTamboli changed the title [material-ui][Pagination] Fix bgColor on hover with extendedTheme [material-ui][Pagination] Fix background color on hover when using CSS theme variables Oct 5, 2023
@ZeeshanTamboli ZeeshanTamboli changed the title [material-ui][Pagination] Fix background color on hover when using CSS theme variables [Pagination] Fix background color on hover when using CSS theme variables Oct 5, 2023
@ZeeshanTamboli ZeeshanTamboli changed the title [Pagination] Fix background color on hover when using CSS theme variables [Pagination] Fix background color on hover and when keyboard focused when using CSS theme variables Oct 5, 2023
@ZeeshanTamboli ZeeshanTamboli changed the title [Pagination] Fix background color on hover and when keyboard focused when using CSS theme variables [Pagination] Fix background color on hover and keyboard focus when using CSS theme variables Oct 5, 2023
Copy link
Member

@ZeeshanTamboli ZeeshanTamboli left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ValkonX33 Looks good! Thank you for working on it!

@ZeeshanTamboli ZeeshanTamboli merged commit dd6297d into mui:master Oct 5, 2023
6 checks passed
mnajdova pushed a commit to mnajdova/material-ui that referenced this pull request Oct 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🐛 Something doesn't work component: pagination This is the name of the generic UI component, not the React module! package: material-ui Specific to @mui/material
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Pagination][material-ui] Bg color on hover not working properly while using extendTheme
4 participants