Make WordPress Core

Opened 2 years ago

Closed 2 years ago

Last modified 2 years ago

#61714 closed defect (bug) (fixed)

Text "None" disappeared from Align Display in WordPress 6.6

Reported by: Marius84 Owned by: hellofromTonya
Priority: high Milestone: 6.6.2
Component: I18N Version: 6.6
Severity: major Keywords: has-patch fixed-major commit dev-reviewed
Cc: Focuses:

Description

None text disappeared from Align Display in WordPress 6.6

https://mariushosting.com/wp-content/uploads/2024/07/missing-text-none.png

Attachments (5)

61714.diff (3.2 KB ) - added by sabernhardt 2 years ago.
adds echo or uses _ex()
61714.update-context.diff (3.3 KB ) - added by sabernhardt 2 years ago.
matches context to pre-existing block editor translations (and echoes string)
link-manager-xfn-missing-labels.png (25.1 KB ) - added by sabernhardt 2 years ago.
the Link Manager does not show 'none' labels for XFN settings
61714.2.diff (5.7 KB ) - added by sabernhardt 2 years ago.
echo strings for link XFN-related labels and media item options/buttons
61714.update-context.2.diff (5.8 KB ) - added by sabernhardt 2 years ago.
echo strings for link labels and media items, and reuse context from block editor translations

Download all attachments as: .zip

Change History (16)

#1 @sabernhardt
2 years ago

  • Keywords needs-patch added
  • Milestone Awaiting Review6.6.2

@sabernhardt
2 years ago

adds echo or uses _ex()

#2 @sabernhardt
2 years ago

  • Keywords has-patch added; needs-patch removed
  • Priority normalhigh
  • Severity normalmajor

[58284] switched to contextual translation functions but missed adding echo (or using _ex()).

Adding media in Classic Editor or a Classic block

Add media: Attachment Display Settings section

  • Alignment (line 843)
  • Link To (line 870)

Edit gallery: Gallery Settings section

  • Link To (line 955)

Insert from URL

  • Align (line 1099)
  • Link To (line 1116)

Replacing media in Classic Editor or a Classic block

Image details: Display Settings section

  • Align (line 1163)
  • Link To (line 1239)

Audio details

  • Preload (line 1359)

Video details

  • Preload (line 1458)

Possible additional edits

Line 955 has 'Media link target' for its context, and I think it should be the same for the other three "Link To" -> "None" options (lines 870, 1116, and 1239). If so, I could update the patch.

Last edited 2 years ago by sabernhardt (previous) (diff)

@sabernhardt
2 years ago

matches context to pre-existing block editor translations (and echoes string)

#3 @sabernhardt
2 years ago

I made another patch to reuse contextual translations from the block editor (GB34341, GB40273).

  • 'Alignment option' (created for block alignment settings)
  • 'Media item link option'
  • 'Preload value'

@sabernhardt
2 years ago

the Link Manager does not show 'none' labels for XFN settings

#4 @sabernhardt
2 years ago

  • Component MediaI18N

Oops. The meta-boxes.php file needs updating too. I'll still make two patches because editing the context is not as important or urgent as echoing the strings.

@sabernhardt
2 years ago

echo strings for link XFN-related labels and media item options/buttons

@sabernhardt
2 years ago

echo strings for link labels and media items, and reuse context from block editor translations

#5 @sabernhardt
2 years ago

To test the XFN labels, install and activate the Link Manager plugin.

#6 @SergeyBiryukov
2 years ago

  • Owner set to SergeyBiryukov
  • Status newreviewing

#7 @SergeyBiryukov
2 years ago

  • Resolutionfixed
  • Status reviewingclosed

In 58774:

I18N: Correctly output the None translatable strings.

Includes updating the context to match the pre-existing block editor translations.

Follow-up to [58284].

Props sabernhardt, Marius84.
Fixes #61714.

#8 @SergeyBiryukov
2 years ago

  • Keywords fixed-major added
  • Resolution fixed
  • Status closedreopened

Hi there, thanks for the ticket! Reopening for 6.6.2 consideration.

#9 @hellofromTonya
2 years ago

  • Keywords commit dev-reviewed added
  • Owner changed from SergeyBiryukov to hellofromTonya
  • Status reopenedreviewing

Tested before and after [58774] on the 6.6 branch to confirm can reproduce and [58774] resolves the bugs. LGTM for backporting the 6.6 branch.

Reassigning to me for the backport commit.

#10 @hellofromTonya
2 years ago

  • Resolutionfixed
  • Status reviewingclosed

In 58805:

I18N: Correctly output the None translatable strings.

Includes updating the context to match the pre-existing block editor translations.

Follow-up to [58284].

Reviewed by hellofromTonya.
Merges [58774] to the 6.6 branch.

Props sabernhardt, Marius84.
Fixes #61714.

This ticket was mentioned in Slack in #core-committers by luisherranz. View the logs.


2 years ago

Note: See TracTickets for help on using tickets.