Make WordPress Core

Opened 10 years ago

Closed 10 years ago

#30752 closed defect (bug) (fixed)

Twenty Fifteen: Consider to use Tahoma or Arial for RTL.

Reported by: iamtakashi's profile iamtakashi Owned by: johnbillion's profile johnbillion
Milestone: 4.1 Priority: normal
Severity: normal Version: 4.1
Component: Bundled Theme Keywords: commit fixed-major
Focuses: Cc:

Description (last modified by iamtakashi)

Feedback from yoav on fonts in RTL.

fonts don't look too good in Hebrew on Windows, and I assume this might be a problem in arabic/farsi too. The mac substitute for a serif font is OK, but horrible on windows. Too late to fix?

Hebrew on Mac
https://cldup.com/Fw0FdJuvrp.png

Arabic on Mac
https://cldup.com/y-PA9f5s5u.png

We could change it in all RTL to something more universal.

Attachments (2)

30752.diff (1.9 KB) - added by iamtakashi 10 years ago.
Use Arial for RTL
30752.1.diff (1.9 KB) - added by iamtakashi 10 years ago.
Use Arial as the primary and Tahoma for the secondary font.

Download all attachments as: .zip

Change History (11)

#1 @iamtakashi
10 years ago

  • Description modified (diff)

@iamtakashi
10 years ago

Use Arial for RTL

#2 @iamtakashi
10 years ago

With Arial.

https://cldup.com/2kd2HeE6uP.png

https://cldup.com/_WOszrr0k5.png

@iamtakashi
10 years ago

Use Arial as the primary and Tahoma for the secondary font.

#3 @iamtakashi
10 years ago

I've added Tahoma as the second choice in 30752.1.diff. This is how it was done in Twenty Ten.

Ref. https://core.trac.wordpress.org/browser/trunk/src/wp-content/themes/twentyten/rtl.css#L38

Last edited 10 years ago by iamtakashi (previous) (diff)

This ticket was mentioned in Slack in #core-themes by iamtakashi. View the logs.


10 years ago

This ticket was mentioned in Slack in #core by johnbillion. View the logs.


10 years ago

#6 @johnbillion
10 years ago

  • Keywords commit has-patch added
  • Milestone changed from Awaiting Review to 4.1

#7 @lancewillett
10 years ago

In 30961:

Twenty Fifteen: use better, universal sans-serif fonts for RTL, similar to Twenty Ten which also had serif fonts throughout.

Props iamtakashi, see #30752.

#8 @lancewillett
10 years ago

  • Keywords fixed-major added; has-patch removed

#9 @johnbillion
10 years ago

  • Owner set to johnbillion
  • Resolution set to fixed
  • Status changed from new to closed

In 30962:

Twenty Fifteen: use better, universal sans-serif fonts for RTL, similar to Twenty Ten which also had serif fonts throughout.

Merges [30961] to the 4.1 branch.

Props iamtakashi
Fixes #30752.

Note: See TracTickets for help on using tickets.