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)
Change History (10)
@
4 years ago
resets letter-spacing for Persian and Urdu, changing CSS selector to include any Persian locale
#1
@
4 years ago
- Component Themes → Bundled Theme
- Keywords has-patch needs-testing added
- Milestone Awaiting Review → 6.2
- Severity minor → normal
- Summary Removing letter-spacing from Farsi/Persian alphabet → Twenty Seventeen: Remove letter-spacing from Farsi/Persian alphabet
- Type enhancement → defect (bug)
- Version 6.1 → 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
@
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.
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
Added a letter spacing for html[lang="fa-IR"]