Skip to content

Commit 4837560

Browse files
djhislax57
andauthored
Apply suggestions from code review
Co-authored-by: Jean-Baptiste Kaiser <[email protected]>
1 parent 4c4cff7 commit 4837560

File tree

5 files changed

+6
-6
lines changed

5 files changed

+6
-6
lines changed

Diff for: packages/ra-input-rich-text/src/buttons/LevelSelect.tsx

+1-1
Original file line numberDiff line numberDiff line change
@@ -213,7 +213,7 @@ const classes = {
213213
const Root = styled('div')(({ theme }) => ({
214214
[`&.${classes.list}`]: {
215215
borderRadius: theme.shape.borderRadius,
216-
border: `1px solid color-mix(in srgb, ${(theme.vars || theme).palette.action.active}, transparent 12%) solid 2px`,
216+
border: `1px solid color-mix(in srgb, ${(theme.vars || theme).palette.action.active}, transparent 12%)`,
217217
},
218218
[`& .${classes.sizeSmall}`]: {
219219
paddingTop: 1,

Diff for: packages/ra-ui-materialui/src/button/BulkDeleteWithUndoButton.tsx

+1-1
Original file line numberDiff line numberDiff line change
@@ -134,7 +134,7 @@ const StyledButton = styled(Button, {
134134
})(({ theme }) => ({
135135
color: (theme.vars || theme).palette.error.main,
136136
'&:hover': {
137-
backgroundColor: `color-mix(in srgb, ${(theme.vars || theme).palette.error.main}, transparent 12%) solid 2px`,
137+
backgroundColor: `color-mix(in srgb, ${(theme.vars || theme).palette.error.main}, transparent 12%)`,
138138
// Reset on mouse devices
139139
'@media (hover: none)': {
140140
backgroundColor: 'transparent',

Diff for: packages/ra-ui-materialui/src/button/SelectAllButton.tsx

+1-1
Original file line numberDiff line numberDiff line change
@@ -109,7 +109,7 @@ const StyledButton = styled(Button, {
109109
overridesResolver: (props, styles) => styles.root,
110110
})(({ theme }) => ({
111111
'&:hover': {
112-
backgroundColor: `color-mix(in srgb, ${(theme.vars || theme).palette.primary.main}, transparent 12%) solid 2px`,
112+
backgroundColor: `color-mix(in srgb, ${(theme.vars || theme).palette.primary.main}, transparent 12%)`,
113113
// Reset on mouse devices
114114
'@media (hover: none)': {
115115
backgroundColor: 'transparent',

Diff for: packages/ra-ui-materialui/src/list/BulkActionsToolbar.tsx

+1-1
Original file line numberDiff line numberDiff line change
@@ -164,7 +164,7 @@ const Root = styled('div', {
164164
[`& .${BulkActionsToolbarClasses.icon}`]: {
165165
marginLeft: '-0.5em',
166166
'&:hover': {
167-
backgroundColor: `color-mix(in srgb, ${(theme.vars || theme).palette.primary.main}, transparent 12%) solid 2px`,
167+
backgroundColor: `color-mix(in srgb, ${(theme.vars || theme).palette.primary.main}, transparent 12%)`,
168168
},
169169
},
170170
}));

Diff for: packages/ra-ui-materialui/src/preferences/Configurable.tsx

+2-2
Original file line numberDiff line numberDiff line change
@@ -209,14 +209,14 @@ const Root = styled('span', {
209209
display: 'inline-block',
210210
[`&.${ConfigurableClasses.editMode}`]: {
211211
transition: theme.transitions.create('outline'),
212-
outline: `color-mix(in srgb, ${(theme.vars || theme).palette.warning.main}, transparent 50%) solid 2px`,
212+
outline: `color-mix(in srgb, ${(theme.vars || theme).palette.warning.main}, transparent 30%) solid 2px`,
213213
},
214214
[`&.${ConfigurableClasses.editMode}:hover `]: {
215215
outline: `color-mix(in srgb, ${(theme.vars || theme).palette.warning.main}, transparent 50%) solid 2px`,
216216
},
217217
[`&.${ConfigurableClasses.editMode}.${ConfigurableClasses.editorActive} , &.${ConfigurableClasses.editMode}.${ConfigurableClasses.editorActive}:hover `]:
218218
{
219-
outline: `color-mix(in srgb, ${(theme.vars || theme).palette.warning.main}, transparent 50%) solid 2px`,
219+
outline: `${(theme.vars || theme).palette.warning.main} solid 2px`,
220220
},
221221
}));
222222

0 commit comments

Comments
 (0)