Skip to content

i18n Make No color selected translatable text. - #54814

Merged
jorgefilipecosta merged 1 commit into
WordPress:trunkfrom
shimotmk:i18n/color-palette
Oct 9, 2023
Merged

i18n Make No color selected translatable text.#54814
jorgefilipecosta merged 1 commit into
WordPress:trunkfrom
shimotmk:i18n/color-palette

Conversation

@shimotmk

Copy link
Copy Markdown
Contributor

What?

Make No color selected translatable text.

Testing Instructions

Place the block that uses the color palette and open the color palette.

Screenshots or screencast

color-palette

@shimotmk
shimotmk requested a review from ajitbohra as a code owner September 26, 2023 04:58

@apeatling apeatling left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@chad1008 Is there any important reason why this text wasn't translated previously? All I can see is that it's expected by tests here, but I don't think that should matter

https://github.com/WordPress/gutenberg/blob/2283d6bc5c33c4c080d8b714939e72475f83a1b2/packages/components/src/color-palette/test/index.tsx#L228C2-L228C2

@chad1008

chad1008 commented Oct 4, 2023

Copy link
Copy Markdown
Contributor

@chad1008 Is there any important reason why this text wasn't translated previously? All I can see is that it's expected by tests here, but I don't think that should matter

2283d6b/packages/components/src/color-palette/test/index.tsx#L228C2-L228C2

None that I recall, I'm pretty sure it was an oversight on my part - glad it got spotted and is being addressed!

@jorgefilipecosta jorgefilipecosta left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for catching this @shimotmk. The change worked well on a simple smoke test I did. I guess we can merge this PR.

@jorgefilipecosta
jorgefilipecosta merged commit 5939b4d into WordPress:trunk Oct 9, 2023
@github-actions github-actions Bot added this to the Gutenberg 16.9 milestone Oct 9, 2023
@shimotmk
shimotmk deleted the i18n/color-palette branch October 10, 2023 00:00
@justintadlock justintadlock added [Type] Bug An existing feature does not function as intended [Feature] Colors Color management labels Oct 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

[Feature] Colors Color management [Type] Bug An existing feature does not function as intended

5 participants