#1594 closed defect (bug) (fixed)
Add RTL support to BrowseHappy.com
Reported by: | ramiy | Owned by: | coffee2code |
---|---|---|---|
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)
#2
@
9 years ago
- Resolution fixed deleted
- Status changed from closed to reopened
@coffee2code, still no RTL support...
#4
@
7 years ago
- Keywords reporter-feedback added
@ramiy: Any additional feedback and assistance on what remains to be done for this?
#5
@
7 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
@
7 years ago
language_attributes()
is supposed to add dir="rtl"
if is_rtl()
returns true, not sure why it doesn't.
#9
@
7 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
@
7 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: