Opened 4 years ago

Last modified 17 months ago

#10658 reopened bug

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!

Change History (8)

comment:1 Changed 4 years ago by Scott González

Resolution: notabug
Status: newclosed

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

comment:2 Changed 4 years ago by Rock

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?

comment:3 Changed 4 years ago by Rock

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

comment:4 Changed 4 years ago by Scott González

Component: [meta] ui.a11yui.slider
Resolution: notabug
Status: closedreopened

comment:5 Changed 23 months ago by Michael Quad

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].

comment:6 Changed 23 months ago by Scott González

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

comment:7 Changed 17 months ago by Ahmed Mustafa

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.

comment:8 Changed 17 months ago by mrashad2017

Dears,

Appreciate if you can review Ahmed code https://github.com/jquery/jquery-ui/pull/1682/commits/c92c2992e38777d92efde91f4b1b8aa8ae244350. As it should fix this issue.

Note: See TracTickets for help on using tickets.