From 8a180e6b7749538df1e84d1c2d78c8a0f9d2439c Mon Sep 17 00:00:00 2001 From: ramonjd Date: Mon, 29 Nov 2021 15:10:35 +1100 Subject: [PATCH] Creating a temporary name when we add a new color to the custom palette --- packages/components/src/palette-edit/index.js | 14 ++++++++++---- 1 file changed, 10 insertions(+), 4 deletions(-) diff --git a/packages/components/src/palette-edit/index.js b/packages/components/src/palette-edit/index.js index 8687c4f9ffb19..ddd8903b7a6b9 100644 --- a/packages/components/src/palette-edit/index.js +++ b/packages/components/src/palette-edit/index.js @@ -7,7 +7,7 @@ import { kebabCase } from 'lodash'; * WordPress dependencies */ import { useState, useRef, useEffect } from '@wordpress/element'; -import { __ } from '@wordpress/i18n'; +import { __, sprintf } from '@wordpress/i18n'; import { lineSolid, moreVertical, plus } from '@wordpress/icons'; import { __experimentalUseFocusOutside as useFocusOutside } from '@wordpress/compose'; @@ -64,6 +64,7 @@ function Option( { } ) { const focusOutsideProps = useFocusOutside( onStopEditing ); const value = isGradient ? element.gradient : element.color; + return ( 0; + const elementsLength = elements.length; + const hasElements = elementsLength > 0; return ( @@ -270,13 +271,18 @@ export default function PaletteEdit( { : __( 'Add color' ) } onClick={ () => { + const tempOptionName = sprintf( + /* translators: %s: is a temporary id for a custom color */ + __( 'Color %s ' ), + elementsLength + 1 + ); onChange( [ ...elements, { ...( isGradient ? { gradient: DEFAULT_GRADIENT } : { color: '#000' } ), - name: '', + name: tempOptionName, slug: '', }, ] );