Bug Tracker

Modify

Ticket #4057 (closed bug: invalid)

Opened 4 years ago

Last modified 15 months ago

Color Animation Bug (Highlight Effect)

Reported by: itjam Owned by:
Priority: undecided Milestone: 1.3.2
Component: effects Version: 1.2.6
Keywords: color, effect, jquery Cc: itjam@…
Blocking: Blocked by:

Description

If a highlight effect happens to an hidden panel, an exception is thrown when trying to return to its original color.

Firefox/jQuery Effect library loses information about the hidden panel. When an AJAX update happens, it does the “yellow” highlighting effect, but does not know how to return to its initial background color. It throws an exception so all JS events failed.

It fails on this line: fx.end is undefined.

We override the animation for all of these color styles 1370jQuery.each(['backgroundColor', 'borderBottomColor', 'borderLeftColor', 'borderRightColor', 'borderTopColor', 'color', 'outlineColor'], function(i,attr){ 1371 jQuery.fx.step[attr] = function(fx){ 1372 if ( fx.state == 0 ) { 1373 fx.start = getColor( fx.elem, attr ); 1374 fx.end = getRGB( fx.end ); 1375 } 1376 1377 fx.elem.style[attr] = "rgb(" + [ 1378 Math.max(Math.min( parseInt((fx.pos * (fx.end[0] - fx.start[0])) + fx.start[0]), 255), 0), 1379 Math.max(Math.min( parseInt((fx.pos * (fx.end[1] - fx.start[1])) + fx.start[1]), 255), 0), 1380 Math.max(Math.min( parseInt((fx.pos * (fx.end[2] - fx.start[2])) + fx.start[2]), 255), 0) 1381 ].join(",") + ")"; 1382 } 1383});

Change History

comment:1 Changed 4 years ago by dmethvin

Can you provide a test case? Thanks.

comment:2 Changed 4 years ago by dmethvin

  • Status changed from new to closed
  • Resolution set to invalid

Closed for inactivity; reopen with a test case if the problem still exists.

Please follow the  bug reporting guidlines and use  jsFiddle when providing test cases and demonstrations instead of pasting the code in the ticket.

View

Add a comment

Modify Ticket

Action
as closed
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.