WordPress.org
Get WordPress

Make WordPress Core

Opened 12 years ago

Closed 12 years ago

#30641 closed defect (bug) (fixed)

Twenty Thirteen/Fourteen: arrows in "Continue Reading" link can't be reversed for RTL

Reported by: yoavf Owned by:
Priority: normal Milestone: 4.1
Component: Bundled Theme Version: 4.1
Severity: normal Keywords: has-patch i18n-change commit
Cc: Focuses: rtl

Description

Since [30389], [30390] the → is no longer part of the translatable string. Previously, translators of RTL languages would have replaced it with ←

Attachments (1)

30641.diff​ (16.7 KB ) - added by johnbillion 12 years ago.

Download all attachments as: .zip

Change History (11)

#1 @ramiy
12 years ago

Nice catch!

#2 @SergeyBiryukov
12 years ago

  • Milestone Awaiting Review → 4.1

@johnbillion
12 years ago

#3 @johnbillion
12 years ago

  • Keywords has-patch i18n-change added; needs-patch removed

Patch. Use of esc_html__() was introduced in r30389 and r30390 but it's not really necessary. Patch replaces it with __() in order to include the markup around the arrow character.

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

This ticket was mentioned in ​Slack in #core-themes by lancewillett. ​View the logs.
12 years ago

#6 @lancewillett
12 years ago

  • Keywords commit added

Patch looks good to commit.

#7 @davidakennedy
12 years ago

Looks good to me too. Thanks, @yoavf for catching that!

#8 @johnbillion
12 years ago

In 30796:

In Twenty Fourteen, move the "Continue Reading" arrows back inside the translatable strings so they can be localised.

See #30641

#9 @johnbillion
12 years ago

r30795 missed the ticket. Sad times.

#10 @johnbillion
12 years ago

  • Resolution → fixed
  • Status new → closed
Note: See TracTickets for help on using tickets.