Opened 2 years ago
Closed 2 years ago
#56994 closed defect (bug) (fixed)
Twenty Seventeen: Remove letter-spacing from Farsi/Persian alphabet
Reported by: | amirrezatm | Owned by: | audrasjb |
---|---|---|---|
Milestone: | 6.2 | Priority: | normal |
Severity: | normal | Version: | 4.7 |
Component: | Bundled Theme | Keywords: | has-patch has-screenshots commit |
Focuses: | css | Cc: |
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)
Change History (10)
@
2 years ago
resets letter-spacing
for Persian and Urdu, changing CSS selector to include any Persian locale
#1
@
2 years ago
- Component changed from Themes to Bundled Theme
- Keywords has-patch needs-testing added
- Milestone changed from Awaiting Review to 6.2
- Severity changed from minor to normal
- Summary changed from Removing letter-spacing from Farsi/Persian alphabet to Twenty Seventeen: Remove letter-spacing from Farsi/Persian alphabet
- Type changed from enhancement to defect (bug)
- Version changed from 6.1 to 4.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
@
2 years ago
- Keywords has-screenshots added; needs-testing removed
Thanks @amirrezatm for the ticket.
@sabernhardt 56994.patch LGTM. Attached screenshot for before/after patch.
Added a letter spacing for html[lang="fa-IR"]