#1594 closed defect (bug) (fixed)
Add RTL support to BrowseHappy.com
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Milestone: | Priority: | normal | |
| Component: | Browse Happy | Keywords: | |
| Cc: |
Description
Currently the BrowseHappy.com site has no RTL support.
In hebrew, for example, all the text in hebrew is LRT instead of RTL.
It's a simple CSS fix.
Change History (14)
#1
@
10 years ago
- Owner set to coffee2code
- Resolution set to fixed
- Status changed from new to closed
#2
@
10 years ago
- Resolution fixed deleted
- Status changed from closed to reopened
@coffee2code, still no RTL support...
#4
@
8 years ago
- Keywords reporter-feedback added
@ramiy: Any additional feedback and assistance on what remains to be done for this?
#5
@
8 years ago
Testing in Chrome set to Hebrew language, [2731] doesn't seem to work as expected. There's no dir="rtl" attribute on the <html> element, so the selectors have no effect.
#6
@
8 years ago
language_attributes() is supposed to add dir="rtl" if is_rtl() returns true, not sure why it doesn't.
#9
@
8 years ago
- Resolution fixed deleted
- Status changed from closed to reopened
Ok,
So I translated the string to "rtl". See https://translate.wordpress.org/projects/meta/browsehappy/he/default .
But page is still in "ltr" mode: https://browsehappy.com/?locale=he .
As @SergeyBiryukov mentioned, we need to add dir="rtl" to the main HTML tag.
When I add dir="rtl" in my console, it changes the site direction and the site looks as expected in Hebrew!
#10
@
8 years ago
- Keywords reporter-feedback removed
- Resolution set to fixed
- Status changed from reopened to closed
@ramiy You have to wait until the translations have been updated.
In 2731: