Opened 11 years ago

Closed 10 years ago

Last modified 8 years ago

#7951 closed defect (fixed)

VerticalSlider: rendering broken in IE8 standards (strict) mode

Reported by: Adam Peller Owned by: Douglas Hays
Priority: high Milestone: 1.3
Component: Dijit - Form Version: 1.2.0
Keywords: Cc: davidb
Blocked By: Blocking:

Description

Go to "sliders" tab.

in _setValueAttr:

			progressBar.style[this._progressPixelSize] = (percent*100) + "%";

percent is NaN. Slider is not rendered properly as a result.

Change History (16)

comment:1 Changed 11 years ago by Douglas Hays

Owner: changed from Douglas Hays to haysmark

comment:2 Changed 11 years ago by haysmark

When the parser parses the slider in IE8, the percent is correct and the Slider indeed sets the height of the two bars to the appropriate percents. However, the parent center tag is not being set to the expected height in IE8 so nothing appears and subsequent calls to setValue fail as described in the ticket. I noticed that if you copy the outerhtml of the Slider into an empty page with no Dojo code the height behaves this way on all browsers, so somehow there is a style that is not being applied in IE8.

comment:3 Changed 11 years ago by Douglas Hays

Owner: changed from haysmark to Douglas Hays

comment:4 Changed 11 years ago by Douglas Hays

Milestone: 1.31.4

I don't see a fix or workaround using IE8 Beta 2.

comment:5 Changed 11 years ago by Douglas Hays

(In [15966]) References #7951. Change DOCTYPE to work around IE8 beta 2 bug with height:100%

comment:6 Changed 11 years ago by Douglas Hays

Using this DOCTYPE

<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.0 Transitional//EN">

seems to enable the VerticalSlider? on IE8 beta 2.

comment:7 Changed 11 years ago by dante

forcing quirksmode for a widget to work doesn't seem like a good thing.

comment:8 Changed 11 years ago by Douglas Hays

Priority: normalhigh

Hopefully this ticket can be resolved when the next IE8 build becomes available.

comment:9 Changed 11 years ago by davidb

Cc: davidb added

IE bug report

(Doug, apologies for delay -- I thought there was a similar bug on file, but can no longer find)

comment:10 Changed 10 years ago by bill

Still broken on RC1 :-(.

comment:11 Changed 10 years ago by bill

It might be better to tell people to specify

<meta http-equiv="X-UA-Compatible" content="IE=EmulateIE7" />

rather than changing the doc type, see the microsoft doc about this.

comment:12 Changed 10 years ago by Douglas Hays

(In [16222]) References #7951. Change DOCTYPE back to strict and add META tag that just affects IE8 rendering. Will update the release notes page with this info as well.

comment:13 Changed 10 years ago by bill

Summary: IE8: vertical slider in themeTester is brokenVerticalSlider: rendering broken in IE8 standards (strict) mode

Just updating summary to be more descriptive.

comment:14 Changed 10 years ago by Douglas Hays

Milestone: 1.41.3
Status: newassigned

comment:15 Changed 10 years ago by Douglas Hays

Resolution: fixed
Status: assignedclosed

(In [16488]) Fixes #7951. IE8 public rc1 fixes this IE bug. Removed the META tag to force IE7 mode.

comment:16 Changed 8 years ago by bill

Component: DijitDijit - Form
Note: See TracTickets for help on using tickets.