Make WordPress Core

Opened 3 years ago

Closed 3 years ago

#59563 closed enhancement (fixed)

Improve translation strings

Reported by: rabmalin Owned by: SergeyBiryukov
Priority: normal Milestone: 6.5
Component: I18N Version: 6.3
Severity: normal Keywords:
Cc: Focuses:

Description

/wp-includes/class-wp-duotone.php L904

__( 'The duotone id "%s" is not registered in theme.json settings' ),

theme.json is never going to be translated in another languages. Would it make more sense if we take that part out of the translation string and use placeholder?

Another case:

"%s" in theme.json settings.color.duotone is not a hex or rgb string.

settings.color.duotone wont be translated I guess.

Change History (4)

#1 @rabmalin
3 years ago

  • Version → 6.3

#2 @Presskopp
3 years ago

there's one more:

Some of the theme.json settings.spacing.spacingScale values are invalid

#3 @SergeyBiryukov
3 years ago

  • Milestone Awaiting Review → 6.5

#4 @SergeyBiryukov
3 years ago

  • Owner set to SergeyBiryukov
  • Resolution → fixed
  • Status new → closed

In 57135:

I18N: Move code out of translatable strings in a few error messages.

Follow-up to [54272], [56101].

Props rabmalin, Presskopp.
Fixes #59563.

Note: See TracTickets for help on using tickets.