Make WordPress Core

Opened 4 years ago

Closed 4 years ago

#56994 closed defect (bug) (fixed)

Twenty Seventeen: Remove letter-spacing from Farsi/Persian alphabet

Reported by: amirrezatm Owned by: audrasjb
Priority: normal Milestone: 6.2
Component: Bundled Theme Version: 4.7
Severity: normal Keywords: has-patch has-screenshots commit
Cc: Focuses: css

Description

Farsi/Persian has a non-Latin alphabet and the letter-spacing property in style.css really exhausts eyes when reading pages in twenty-seventeen theme

there is a section starting with:

/* Remove letter-spacing for all non-latin alphabets */

in style.css that lists the languages that have letter-spacing omitted from their properties and it lacks Persian, simply add a

html[lang="fa-IR"] * 

and almost 120mln more people can use 2017 for their blog including me :D

Attachments (4)

style.css (86.4 KB ) - added by itpathsolutions 4 years ago.
Added a letter spacing for html[lang="fa-IR"]
56994.patch (1.4 KB ) - added by sabernhardt 4 years ago.
resets letter-spacing for Persian and Urdu, changing CSS selector to include any Persian locale
Before patch.png (1.6 MB ) - added by mukesh27 4 years ago.
After patch.png (1.6 MB ) - added by mukesh27 4 years ago.

Change History (10)

@itpathsolutions
4 years ago

Added a letter spacing for html[lang="fa-IR"]

@sabernhardt
4 years ago

resets letter-spacing for Persian and Urdu, changing CSS selector to include any Persian locale

#1 @sabernhardt
4 years ago

  • Component ThemesBundled Theme
  • Keywords has-patch needs-testing added
  • Milestone Awaiting Review6.2
  • Severity minornormal
  • Summary Removing letter-spacing from Farsi/Persian alphabetTwenty Seventeen: Remove letter-spacing from Farsi/Persian alphabet
  • Type enhancementdefect (bug)
  • Version 6.14.7

Hi and thanks for the report!

Urdu was missing from that list, too. I also changed the selector for all Persian styles to include the Afghanistan locale.

#2 @mukesh27
4 years ago

  • Keywords has-screenshots added; needs-testing removed

Thanks @amirrezatm for the ticket.

@sabernhardt 56994.patch LGTM. Attached screenshot for before/after patch.

@mukesh27
4 years ago

@mukesh27
4 years ago

#3 @amirrezatm
4 years ago

It looks great now thank you!

#4 @sabernhardt
4 years ago

  • Keywords commit added

#5 @audrasjb
4 years ago

  • Owner set to audrasjb
  • Status newaccepted

Looks good to me. Self assigning for commit consideration.

#6 @audrasjb
4 years ago

  • Resolutionfixed
  • Status acceptedclosed

In 54838:

Twenty Seventeen: Improve letter-spacing reset for non-latin alphabets.

This changeset resets letter-spacing for Persian and Urdu, and changes the related CSS selector to include any Persian locale.

Props amirrezatm, itpathsolutions, sabernhardt, mukesh27.
Fixes #56994.

Note: See TracTickets for help on using tickets.