From f27d71cbf8ebf1755ab413b96635961cf81905cc Mon Sep 17 00:00:00 2001 From: Elio Rivero Date: Sat, 11 Aug 2018 13:50:08 -0300 Subject: [PATCH 1/2] Color palette: solve inconsistency between the Clear link and buttons in other panels like the Reset buttons in Text or Image settings --- packages/components/src/color-palette/index.js | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/packages/components/src/color-palette/index.js b/packages/components/src/color-palette/index.js index f0163630d95a9..c69be9543a35b 100644 --- a/packages/components/src/color-palette/index.js +++ b/packages/components/src/color-palette/index.js @@ -84,7 +84,9 @@ export default function ColorPalette( { colors, disableCustomColors = false, val className="components-color-palette__clear" type="button" onClick={ () => onChange( undefined ) } - isLink + isButton + isSmall + isDefault > { __( 'Clear' ) } From 0d882acda9d5710396fc8d112ddda6fafd57f2d7 Mon Sep 17 00:00:00 2001 From: Elio Rivero Date: Fri, 17 Aug 2018 11:21:12 -0300 Subject: [PATCH 2/2] Update snapshot to reflect code changes so tests pass. --- .../src/color-palette/test/__snapshots__/index.js.snap | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/packages/components/src/color-palette/test/__snapshots__/index.js.snap b/packages/components/src/color-palette/test/__snapshots__/index.js.snap index 636c0132c5a75..d8a1b66119023 100644 --- a/packages/components/src/color-palette/test/__snapshots__/index.js.snap +++ b/packages/components/src/color-palette/test/__snapshots__/index.js.snap @@ -127,7 +127,9 @@ exports[`ColorPalette should allow disabling custom color picker 1`] = `