Make WordPress Core

Opened 14 months ago

Last modified 8 days ago

#60770 assigned enhancement

TimeZone select box compatibility with RTL directions

Reported by: farhad0's profile farhad0 Owned by: audrasjb's profile audrasjb
Milestone: 6.9 Priority: normal
Severity: normal Version: 6.5
Component: I18N Keywords: has-patch has-screenshots has-test-info dev-feedback
Focuses: rtl, administration Cc:

Description

This patch fixes the time zone select box compatibility with RTL directions It sets auto direction which makes the options value compatible with both RTL and LTR directions

Attachments (3)

timezone-select-box-rtl-compatibility.jpg (116.6 KB) - added by farhad0 14 months ago.
timezone-select-box-rtl-compatibility-before-after
rtl_before.png (44.0 KB) - added by drysand 14 months ago.
rtl_after.png (43.1 KB) - added by drysand 14 months ago.

Download all attachments as: .zip

Change History (11)

This ticket was mentioned in PR #6268 on WordPress/wordpress-develop by @farhad0.


14 months ago
#1

This patch fixes the time zone select box compatibility with RTL directions It sets auto direction which makes the options value compatible with both RTL and LTR directions

Trac ticket: https://core.trac.wordpress.org/ticket/60770

---

#2 @swissspidy
14 months ago

  • Keywords needs-testing-info needs-screenshots added

@farhad0 Could you perhaps add before/after screenshots so that others looking at this ticket can better understand the problem you're trying to solve? Thanks!

@farhad0
14 months ago

timezone-select-box-rtl-compatibility-before-after

#3 @farhad0
14 months ago

  • Keywords has-screenshots added; needs-screenshots removed

@swissspidy
Hi,
Here you are

Version 0, edited 14 months ago by farhad0 (next)

@drysand
14 months ago

@drysand
14 months ago

#4 @drysand
14 months ago

i tested the patch and it does indeed align english text LTR and RTL text RTL, however it seems to align all the optiongroup headers LTR no matter the language

sorry about the separate images, im still getting used to trac

#5 @sabernhardt
5 weeks ago

#63184 was marked as a duplicate.

#6 @sabernhardt
5 weeks ago

  • Component changed from Administration to I18N
  • Milestone changed from Awaiting Review to 6.9
  • Owner set to audrasjb
  • Status changed from new to assigned

In RTL (fa-IR):

  • PR 6268 aligns every optgroup plus 'UTC' and all UTC offset options to the left, and the other options align right.
  • PR 8600 aligns the options for 'UTC' and the UTC offsets to the left, but all optgroup labels and most options align to the right.

I would like to consider using the same alignment for the 'UTC' optgroup and its option (I prefer both aligned left in Firefox but both on the right seems better in Chrome).

#7 @farhad0
3 weeks ago

@sabernhardt
Hi,
The Chrome detects the direction of the text and align it automatically,
This is a way which many modern applications use and this works on Chrome and may be some other browsers

#8 @SirLouen
8 days ago

  • Keywords has-test-info dev-feedback added; needs-testing-info removed

Updating keywords to reflect current status.

Note: See TracTickets for help on using tickets.