Skip to content

Gallery: translate string, not results of sprintf#13830

Merged
gziolo merged 1 commit intoWordPress:masterfrom
simison:patch-2
Feb 13, 2019
Merged

Gallery: translate string, not results of sprintf#13830
gziolo merged 1 commit intoWordPress:masterfrom
simison:patch-2

Conversation

@simison
Copy link
Copy Markdown
Member

@simison simison commented Feb 12, 2019

Description

Translation function was previously wrapping results of sprintf, instead of the string passed to sprinf.

How has this been tested?

  • Add Gallery block in the editor
  • Add some images
  • Observe aria label look correct (you should see aria-label="image 1 of 2 in gallery" etc)

Checklist:

  • My code is tested.
  • My code follows the WordPress code style.
  • My code follows the accessibility standards.
  • My code has proper inline documentation.
  • I've included developer documentation if appropriate.

@youknowriad youknowriad added the Good First Review A PR that's suitable for someone looking to contribute for the first time by reviewing code label Feb 12, 2019
@gziolo gziolo added this to the 5.1 (Gutenberg) milestone Feb 12, 2019
@gziolo gziolo added the Internationalization (i18n) Issues or PRs related to internationalization efforts label Feb 12, 2019
@gziolo gziolo merged commit 8f9738d into WordPress:master Feb 13, 2019
@gziolo
Copy link
Copy Markdown
Member

gziolo commented Feb 13, 2019

Nice catch, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Good First Review A PR that's suitable for someone looking to contribute for the first time by reviewing code Internationalization (i18n) Issues or PRs related to internationalization efforts

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants