Opened 11 years ago

Closed 11 years ago

#7714 closed defect (fixed)

IE unit test failure in NodeList::style

Reported by: Adam Peller Owned by: James Burke
Priority: high Milestone: 1.2
Component: Core Version: 1.2beta
Keywords: Cc:
Blocked By: Blocking:

Description

seen on ie6 & ie7

    _AssertFailure: [object Error]: assertEqual() failed:  expected 1 but got 0.5  [object Error]     ERROR IN:  function style(){ var tnl = new dojo.NodeList(s, t); tnl.style("opacity", 1); doh.is(1, tnl.style("opacity")[0]); dojo.style(t, "opacity", 0.5); doh.is(1.0, tnl.style("opacity")[0]); doh.is(0.5, tnl.style("opacity")[1]); // reset things tnl.style("opacity", 1); }

Change History (3)

comment:1 Changed 11 years ago by bill

Owner: changed from anonymous to James Burke

James, please take a look (is this a regression and if so when did it regress?)

comment:2 Changed 11 years ago by James Burke

Milestone: tbd1.2

Related to changes in [13991], but I have a fix.

comment:3 Changed 11 years ago by James Burke

Resolution: fixed
Status: newclosed

(In [15330]) Fixes #7714. The change to switching to setting Enabled on a filter (to preserve other filters on the node) left a previously set value of opacity as the opacity value even though it was not active. Make sure we update the opacity value to 1, but still make sure the filter is set to Enabled = false for the opacity == 1 case. \!strict

Note: See TracTickets for help on using tickets.