Skip to content

Commit

Permalink
Fix Default Styles for Radio input, Revert unmounting of conditional …
Browse files Browse the repository at this point in the history
…display container.
  • Loading branch information
HTMLBurger-NG committed Mar 10, 2023
1 parent 8c9cd54 commit 409afd9
Show file tree
Hide file tree
Showing 10 changed files with 14 additions and 12 deletions.
3 changes: 2 additions & 1 deletion build/classic/core.css
Original file line number Diff line number Diff line change
Expand Up @@ -2044,7 +2044,8 @@ span.flatpickr-weekday {
}
.cf-radio__input[type=checkbox]:checked:before {
content: "";
background: var(--wp-admin-theme-color);
background-color: #3582c4;
background-color: var(--wp-admin-theme-color, #3582c4);
border-radius: 50%;
width: 0.5rem;
height: 0.5rem;
Expand Down
2 changes: 1 addition & 1 deletion build/classic/core.min.css

Large diffs are not rendered by default.

4 changes: 2 additions & 2 deletions build/classic/metaboxes.js

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion build/classic/metaboxes.min.js

Large diffs are not rendered by default.

3 changes: 2 additions & 1 deletion build/gutenberg/core.css
Original file line number Diff line number Diff line change
Expand Up @@ -2044,7 +2044,8 @@ span.flatpickr-weekday {
}
.cf-radio__input[type=checkbox]:checked:before {
content: "";
background: var(--wp-admin-theme-color);
background-color: #3582c4;
background-color: var(--wp-admin-theme-color, #3582c4);
border-radius: 50%;
width: 0.5rem;
height: 0.5rem;
Expand Down
2 changes: 1 addition & 1 deletion build/gutenberg/core.min.css

Large diffs are not rendered by default.

4 changes: 2 additions & 2 deletions build/gutenberg/metaboxes.js

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion build/gutenberg/metaboxes.min.js

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion packages/core/fields/radio/style.scss
Original file line number Diff line number Diff line change
Expand Up @@ -56,7 +56,7 @@

&[type="checkbox"]:checked:before {
content: "";
background: var(--wp-admin-theme-color);
background-color: var(--wp-admin-theme-color, #3582c4);
border-radius: 50%;
width: 0.5rem;
height: 0.5rem;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -114,7 +114,7 @@ export default function handler( { containers, context } ) {
// due to https://github.com/facebook/react/issues/13690.
// TODO: Conditionally render the fields in the container, this way
// we can move away from mount/unmount cycles.
//containerNode._reactRootContainer.unmount(); // -- causes a memory leak bug
containerNode._reactRootContainer.unmount();
}
}
} );
Expand Down

0 comments on commit 409afd9

Please sign in to comment.