WordPress.org
Get WordPress

Make WordPress Core

Opened 3 years ago

Closed 2 years ago

#60827 closed defect (bug) (fixed)

Patterns menu item, put back the context parameter.

Reported by: kebbet Owned by: audrasjb
Priority: normal Milestone: 6.5.3
Component: I18N Version: 6.5
Severity: normal Keywords: good-first-bug has-patch fixed-major i18n-change dev-reviewed
Cc: Focuses:

Description

This is a follow up for the next minor release, 6.5.1 and the previous ticket #60825

The commit message by @audrasjb in [57864] stated: A follow-up changeset will be needed to replace the current __() function with _x() and put back the context parameter.

This is what should replace the call to __():
_x( 'Patterns', 'site editor menu item' )

Change History (18)

#1 @swissspidy
3 years ago

  • Focuses ui removed
  • Milestone Awaiting Review → 6.5.1

#2 @audrasjb
3 years ago

  • Component Administration → I18N
  • Status assigned → accepted

Related: #60825

#3 @narenin
3 years ago

Hi,

I have fixed this in my PR.

Please find my PR : ​https://github.com/WordPress/wordpress-develop/pull/6326

Last edited 3 years ago by narenin (previous) (diff)

This ticket was mentioned in ​PR #6326 on ​WordPress/wordpress-develop by ​@narenin.
3 years ago #4

  • Keywords has-patch added

#6 @audrasjb
3 years ago

  • Keywords commit added

Thanks for the PR @narenin and @nestea29950. Approving PR6327 for commit.

​@audrasjb commented on ​PR #6326:
3 years ago #7

Closing in favor of PR#6327

#8 @audrasjb
3 years ago

  • Resolution → fixed
  • Status accepted → closed

In 57887:

I18n: Add context to the "Patterns" translatable string for the related admin menu item.

This changeset put back the context parameter of the "Patterns" string which was previously removed to fix a translation issue without introducing a string change during WP 6.5 string freeze period.

Follow-up to [57864].

Props kebbet, narenin, nestea29950.
Fixes #60827.

#10 @audrasjb
3 years ago

  • Keywords fixed-major i18n-change added; commit removed
  • Resolution fixed
  • Status closed → reopened

Reopening for 6.5.1 backport consideration.

#11 @audrasjb
3 years ago

  • Keywords dev-feedback added

#12 @swissspidy
3 years ago

  • Keywords dev-reviewed added; dev-feedback removed

This ticket was mentioned in ​Slack in #core by jorbin. ​View the logs.
3 years ago

#14 @davidbaumwald
3 years ago

  • Milestone 6.5.1 → 6.5.2

Milestone renamed

#15 @jorbin
3 years ago

  • Milestone 6.5.2 → 6.5.3

This ticket was mentioned in ​Slack in #core by jorbin. ​View the logs.
2 years ago

This ticket was mentioned in ​Slack in #core by jorbin. ​View the logs.
2 years ago

#18 @audrasjb
2 years ago

  • Resolution → fixed
  • Status reopened → closed

In 58038:

I18n: Add context to the "Patterns" translatable string for the related admin menu item.

This changeset put back the context parameter of the "Patterns" string which was previously removed to fix a translation issue without introducing a string change during WP 6.5 string freeze period.

Follow-up to [57864].

Reviewed by audrasjb, jorbin.
Merges [57887] to the to the 6.5 branch.
Props kebbet, narenin, nestea29950.
Fixes #60827.

Note: See TracTickets for help on using tickets.