Opened 12 years ago

Closed 11 years ago

#6229 closed defect (fixed)

[cla/patch] ProgressBar: glitch on IE6 for test2

Reported by: bill Owned by: bill
Priority: high Milestone: 1.2
Component: Dijit - LnF Version: 1.1b1
Keywords: Cc:
Blocked By: Blocking:

Description (last modified by dylan)

See test_ProgressBar.html, the second example. On IE6 it draws strangely: the vertical bar extends below the bottom of the progress bar rectangle. Works on other browsers.

Attachments (4)

progressBar.gif (1.8 KB) - added by bill 12 years ago.
progress.patch (2.7 KB) - added by dante 11 years ago.
a patch, which might not be acceptable
ie6patch.png (8.0 KB) - added by dante 11 years ago.
this is what ie6 looks like w/ the patch
ie7.png (6.4 KB) - added by dante 11 years ago.
... and ie7 for fun

Download all attachments as: .zip

Change History (9)

Changed 12 years ago by bill

Attachment: progressBar.gif added

comment:1 Changed 12 years ago by bill

Description: modified (diff)

comment:2 Changed 11 years ago by dylan

Component: DijitDijit - LnF
Description: modified (diff)
Owner: set to dante

Changed 11 years ago by dante

Attachment: progress.patch added

a patch, which might not be acceptable

Changed 11 years ago by dante

Attachment: ie6patch.png added

this is what ie6 looks like w/ the patch

Changed 11 years ago by dante

Attachment: ie7.png added

... and ie7 for fun

comment:3 Changed 11 years ago by dante

Owner: changed from dante to bill
Summary: ProgressBar: glitch on IE6 for test2[cla/patch] ProgressBar: glitch on IE6 for test2

it's actually a lot worse in the nightly than the screenshot. attached are two screenshots and a patch, which basically sets the height to 1.6 em, so it might not scale up well, but the background image isn't large (tall) enough to scale up so that may be something else that needs to be addressed.

please review attached.

comment:4 Changed 11 years ago by dante

ps: tested ie6, ie7, opera 9.2, safari 3.1, ff2 - tundra only. can't test test_ProgressBar.html via ?theme= anymore (or at least that test specifically) because of the programmatic examples in the addOnLoad + the new way of loading dynamic themes.

comment:5 Changed 11 years ago by bill

Resolution: fixed
Status: newclosed

(In [13854]) Patch (thanks Peter!) to fix ProgressBar? glitches on IE6, plus removed some apparently unneeded CSS rules. Fixes #6229.

Note: See TracTickets for help on using tickets.