Skip to content

Commit

Permalink
Remove custom secondary color styling from Checkbox and Radio (#…
Browse files Browse the repository at this point in the history
…2842)

There should be no custom secondary styling for radios and checkboxes. Therefore the styling is removed.

Co-authored-by: Julia Wegmayr <[email protected]>
Co-authored-by: Ricky James Smith <[email protected]>
  • Loading branch information
3 people authored Dec 5, 2024
1 parent 7d642ab commit c5d9a47
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 19 deletions.
5 changes: 5 additions & 0 deletions .changeset/soft-horses-ring.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
---
"@comet/admin-theme": major
---

Remove custom `secondary` color styling from `Checkbox` and `Radio`
12 changes: 0 additions & 12 deletions packages/admin/admin-theme/src/componentsTheme/MuiCheckbox.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -65,17 +65,5 @@ export const getMuiCheckbox: GetMuiComponentTheme<"MuiCheckbox"> = (component, {
},
},
},
colorSecondary: {
[`&.${checkboxClasses.checked} .${svgIconClasses.root}`]: {
"& .background": {
fill: palette.secondary.main,
},
},
[`&.${checkboxClasses.indeterminate} .${svgIconClasses.root}`]: {
"& .background": {
fill: palette.grey[300],
},
},
},
}),
});
7 changes: 0 additions & 7 deletions packages/admin/admin-theme/src/componentsTheme/MuiRadio.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -52,12 +52,5 @@ export const getMuiRadio: GetMuiComponentTheme<"MuiRadio"> = (component, { palet
},
},
},
colorSecondary: {
[`&.${radioClasses.checked} .${svgIconClasses.root}`]: {
"& .background": {
fill: palette.secondary.main,
},
},
},
}),
});

0 comments on commit c5d9a47

Please sign in to comment.