update color, hover styles and remove active styles on radio btn according to VOCA #1340

hannaharnett · 2023-06-02T08:35:38Z

Changed color from core-purple to black-purple
Changed hover styles to only show thicker border (no shadows) and only apply to enabled radio btn
Removed active effects

mariannemagnussen · 2023-06-09T12:55:13Z

src/atoms/RadioButton/RadioButton.pcss

@@ -35,12 +35,12 @@
  /* If the input is checked, display the inner circle with purple color */
  & > input:checked ~ .radiobutton__svg-container .radiobutton__circle {
    display: block;
-    fill: var(--core-purple);
+    fill: var(--black-purple);


I don't think the correct purple is the black-purple 🤔 For the button we've used this --core-purple-850. I see that they now call that color "deep purple" (like we do in oneportal), or purple 900. So not sure if we should add a color called deep purple? or if we should add the new "purple" colors. Seems like they've changed the purple scale a bit

update color, hover styles and remove active styles on radio btn

5ae0ed3

hannaharnett requested review from mariannemagnussen, leskraas and joakikr June 2, 2023 08:35

mariannemagnussen reviewed Jun 9, 2023

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

update color, hover styles and remove active styles on radio btn according to VOCA #1340

update color, hover styles and remove active styles on radio btn according to VOCA #1340

hannaharnett commented Jun 2, 2023

mariannemagnussen Jun 9, 2023

update color, hover styles and remove active styles on radio btn according to VOCA #1340

Are you sure you want to change the base?

update color, hover styles and remove active styles on radio btn according to VOCA #1340

Conversation

hannaharnett commented Jun 2, 2023

mariannemagnussen Jun 9, 2023

Choose a reason for hiding this comment