Skip to main content

Search and Top Navigation

#6727 closed bug (fixed)

Opened December 07, 2010 04:57PM UTC

Closed November 23, 2012 03:45AM UTC

Last modified November 23, 2012 12:09PM UTC

Slider: Visual issues with disabled slider in IE8

Reported by: RwwL Owned by: tj.vantoll
Priority: minor Milestone: 1.10.0
Component: ui.slider Version: 1.8.6
Keywords: Cc:
Blocked by: Blocking:
Description

In IE8: only the slider track gets its opacity adjusted when a slider is disabled, not the thumb.

In IE7: both track and thumb get opacity adjustments when disabled, but the thumb is cropped to the height or width of a vertical or horizontal slider, respectively.

This bug is observable right on http://jqueryui.com/demos/slider/ if you use IE's JavaScript console to disable the sliders.

Attachments (0)
Change History (11)

Changed October 11, 2012 02:53PM UTC by scottgonzalez comment:1

milestone: 1.9.01.11.0

Changed October 15, 2012 06:20PM UTC by davenaeder comment:2

status: newopen

Confirmed, this can be observed here: http://jsfiddle.net/davenaeder/Knyze/ in IE browsers

Changed November 14, 2012 03:15PM UTC by tj.vantoll comment:3

owner: → tj.vantoll
status: openassigned
summary: visual issues with disabled slider in IESlider: Visual issues with disabled slider in IE8

This is an IE8 only issue, here's a better fiddle - http://jsfiddle.net/tj_vantoll/R8Jqx/. This has the same base problem as #6059, in IE8 non static positioned elements do not inherit the opacity of their parents.

I'll come up with something to fix this.

Changed November 15, 2012 02:48AM UTC by tj.vantoll comment:4

I have a fix for this - http://jsfiddle.net/tj_vantoll/R8Jqx/11/show/, but there might be a bigger problem here. In all versions of IE when I disable the slider the handles simply seem to disappear. Anybody else seeing this?

I'm going to try it out on some real IE installations tomorrow to make sure that it's not a BrowserStack thing.

Changed November 15, 2012 01:51PM UTC by scottgonzalez comment:5

They don't actually disappear, they're just very light. Maybe try looking from a different angle to get more contrast from your monitor. If you increase the opacity, you'll see them better.

Changed November 17, 2012 04:25AM UTC by tj.vantoll comment:6

Pull request: https://github.com/jquery/jquery-ui/pull/825

For a walkthrough of my logic that led to this unfortunate solution see the comments in the CSS box on this fiddle - http://jsfiddle.net/tj_vantoll/CFgDL/.

Changed November 23, 2012 03:45AM UTC by TJ VanToll comment:7

resolution: → fixed
status: assignedclosed

Slider: Explicitly apply opacity filter to disabled handles. Fixed #6727 - Slider: Visual issues with disabled slider in IE8

Changeset: 421aeaa08135e747ff9f2a59974c26b350cdcda7

Changed November 23, 2012 03:46AM UTC by mikesherov comment:8

milestone: 1.11.01.10.0

Changed November 23, 2012 10:08AM UTC by jzaefferer comment:9

I've looked at merging this into 1-9-stable, but it won't apply cleanly, so ignoring it.

Changed November 23, 2012 12:03PM UTC by mikesherov comment:10

joern, if you merge the CSS style changes first, it'll merge cleanly (I think).

Changed November 23, 2012 12:09PM UTC by scottgonzalez comment:11

Don't merge the style changes, just leave this for 1.10.0.