Make WordPress Core

Opened 5 months ago

Closed 5 months ago

Last modified 4 months ago

#61714 closed defect (bug) (fixed)

Text "None" disappeared from Align Display in WordPress 6.6

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

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 5 months ago.
adds echo or uses _ex()
61714.update-context.diff (3.3 KB) - added by sabernhardt 5 months ago.
matches context to pre-existing block editor translations (and echoes string)
link-manager-xfn-missing-labels.png (25.1 KB) - added by sabernhardt 5 months ago.
the Link Manager does not show 'none' labels for XFN settings
61714.2.diff (5.7 KB) - added by sabernhardt 5 months ago.
echo strings for link XFN-related labels and media item options/buttons
61714.update-context.2.diff (5.8 KB) - added by sabernhardt 5 months 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
5 months ago

  • Keywords needs-patch added
  • Milestone changed from Awaiting Review to 6.6.2

@sabernhardt
5 months ago

adds echo or uses _ex()

#2 @sabernhardt
5 months ago

  • Keywords has-patch added; needs-patch removed
  • Priority changed from normal to high
  • Severity changed from normal to major

[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 5 months ago by sabernhardt (previous) (diff)

@sabernhardt
5 months ago

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

#3 @sabernhardt
5 months 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
5 months ago

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

#4 @sabernhardt
5 months ago

  • Component changed from Media to I18N

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
5 months ago

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

@sabernhardt
5 months ago

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

#5 @sabernhardt
5 months ago

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

#6 @SergeyBiryukov
5 months ago

  • Owner set to SergeyBiryukov
  • Status changed from new to reviewing

#7 @SergeyBiryukov
5 months ago

  • Resolution set to fixed
  • Status changed from reviewing to closed

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
5 months ago

  • Keywords fixed-major added
  • Resolution fixed deleted
  • Status changed from closed to reopened

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

#9 @hellofromTonya
5 months ago

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

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
5 months ago

  • Resolution set to fixed
  • Status changed from reviewing to closed

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.


4 months ago

Note: See TracTickets for help on using tickets.