#10658 reopened bug ()

Opened October 11, 2014 03:39AM UTC

Last modified November 06, 2017 04:43PM UTC

Slider: handle is not mirrored in RTL mode

Reported by: Rock Owned by:
Priority: minor Milestone: none
Component: ui.slider Version: 1.11.1
Keywords: Cc:
Blocked by: Blocking:
Description

Should slider handle start from right in rtl mode? And arrow keyboard support should be reversed also. I'd like to provide a fix if this confirmed as a open defect, thanks!

Attachments (0)
Change History (8)

Changed October 11, 2014 12:55PM UTC by scottgonzalez comment:1

resolution: → notabug
status: new → closed

Please ensure these are necessary before filing all of these bugs.

Changed October 13, 2014 05:35AM UTC by Rock comment:2

Thanks for the comment, and after looking through html rendering spec in w3c, http://www.w3.org/TR/html5/rendering.html#the-input-element-as-a-range-control. It said that- "''...with the lowest value on the right if the 'direction' property on this element has a computed value of 'rtl', and on the left otherwise...''", so I think it's better to mirror slider in rtl mode, right?

Changed October 13, 2014 03:11PM UTC by Rock comment:3

Test case: http://jsfiddle.net/aaq0sdwn/, <body> in testcase has attribute "dir=rtl"

Changed October 23, 2014 02:36PM UTC by scottgonzalez comment:4

component: [meta] ui.a11y → ui.slider
resolution: notabug
status: closed → reopened

Changed May 09, 2017 12:44PM UTC by determin1st comment:5

this issue may require large rewrite of positioning and input routines, so there is no local solution.

you may use modified slider with option: direction=true, which inverts scale from [0..100] to [100..0].

Changed May 09, 2017 02:30PM UTC by scottgonzalez comment:6

I'm not sure what you're talking about. There is no direction option.

Changed October 26, 2017 12:09PM UTC by AhmedMustafa comment:7

I agree with Rock. The Slider handle should start from right in RTL mode. This is the expected behavior for the Arabic/Hebrew users or any RTL speaking countries.

I've proposed some code changes to handle this issue here https://github.com/jquery/jquery-ui/pull/1682/commits/c92c2992e38777d92efde91f4b1b8aa8ae244350. I hope you take a look and review it.

Changed November 06, 2017 04:43PM UTC by mrashad2017 comment:8

Dears,

Appreciate if you can review Ahmed code

https://github.com/jquery/jquery-ui/pull/1682/commits/c92c2992e38777d92efde91f4b1b8aa8ae244350.

As it should fix this issue.