Make WordPress Core

Opened 5 years ago

Closed 4 years ago

#55564 closed defect (bug) (fixed)

Twenty Twenty-One: edit translator comment in Continue Reading text

Reported by: sabernhardt Owned by: davidbaumwald
Priority: normal Milestone: 6.0
Component: Bundled Theme Version: 5.6
Severity: normal Keywords: has-patch commit
Cc: Focuses: docs

Description

In twenty_twenty_one_continue_reading_text(), the translator comment does not explain that the post title is visually hidden.

Additionally, a comment for the the_content_more_link filter mentions the excerpt instead of the content.

Attachments (2)

55564.patch​ (922 bytes ) - added by sabernhardt 5 years ago.
editing translator comment plus content more comment
55564.diff​ (2.7 KB ) - added by SergeyBiryukov 4 years ago.

Download all attachments as: .zip

Change History (9)

@sabernhardt
5 years ago

editing translator comment plus content more comment

#1 @costdev
5 years ago

  • Keywords commit added
  • Version → 5.6

This looks good to me, marking for commit consideration.

#2 @davidbaumwald
5 years ago

  • Owner set to davidbaumwald
  • Status new → reviewing

#3 @davidbaumwald
4 years ago

  • Resolution → fixed
  • Status reviewing → closed

In 53221:

Twenty Twenty-One: Correct translator comment in twenty_twenty_one_continue_reading_text.

This commit updates the translator comment in twenty_twenty_one_continue_reading_text to indicate that the text is visually hidden. This change also corrects the comment above the the_content_more_link filter to reference the content, not the excerpt.

Props sabernhardt, costdev.
Fixes #55564.

@SergeyBiryukov
4 years ago

#4 @SergeyBiryukov
4 years ago

  • Resolution fixed
  • Status closed → reopened

55564.diff​ uses the same translator comment in other files of the theme.

Should we open a ticket to make these changes for other bundled themes too?

#5 @audrasjb
4 years ago

@SergeyBiryukov yeah it would makes a lot of sense to me.
I think you can commit 55564.diff under this ticket, and it would probably be nice to open a new ticket for other bundled themes.

#6 @sabernhardt
4 years ago

The ticket for other themes is opened: #55591

#7 @davidbaumwald
4 years ago

  • Resolution → fixed
  • Status reopened → closed

In 53263:

Twenty Twenty-One: Make translator comments referencing the post title consistent.

Follow-up to [53221].

Props SergeyBiryukov.
Fixes #55564.

Note: See TracTickets for help on using tickets.