Skip to main content

Bug Tracker

Side navigation

#5487 closed bug (duplicate)

Opened November 11, 2009 08:42AM UTC

Closed November 14, 2009 03:18AM UTC

animate height or width changes display mode

Reported by: orlleite Owned by:
Priority: major Milestone:
Component: effects Version: 1.3.2
Keywords: animate height width display Cc:
Blocked by: Blocking:
Description

Hi!

When you add a animation to a element, the display css is changed. It's should be the correct way, but not ways. In my case, table-rows get a miss visualization when it's occurs.

I did a little change, for me it's works.

My line 4023:

if ( ( this.prop == "height" || this.prop == "width" ) && this.elem.style && this.elem.style.display != "table-row" )

Yours:

if ( ( this.prop == "height" || this.prop == "width" ) && this.elem.style )

Could be others cases like this, where jquery should not change the display mode. In my way, I verify the value of display style, but it's could be a parameter, pass in animate function, for example.

thx.

Attachments (0)
Change History (2)

Changed November 11, 2009 12:51PM UTC by dmethvin comment:1

description: Hi! \ \ When you add a animation to a element, the display css is changed. It's should be the correct way, but not ways. In my case, table-rows get a miss visualization when it's occurs. \ \ I did a little change, for me it's works. \ \ My line 4023: \ \ if ( ( this.prop == "height" || this.prop == "width" ) && this.elem.style && this.elem.style.display != "table-row" ) \ \ Yours: \ \ if ( ( this.prop == "height" || this.prop == "width" ) && this.elem.style ) \ \ \ Could be others cases like this, where jquery should not change the display mode. In my way, I verify the value of display style, but it's could be a parameter, pass in animate function, for example. \ \ thx.Hi! \ \ When you add a animation to a element, the display css is changed. It's should be the correct way, but not ways. In my case, table-rows get a miss visualization when it's occurs. \ \ I did a little change, for me it's works. \ \ My line 4023: \ {{{ \ if ( ( this.prop == "height" || this.prop == "width" ) && this.elem.style && this.elem.style.display != "table-row" ) \ }}} \ Yours: \ {{{ \ if ( ( this.prop == "height" || this.prop == "width" ) && this.elem.style ) \ }}} \ \ Could be others cases like this, where jquery should not change the display mode. In my way, I verify the value of display style, but it's could be a parameter, pass in animate function, for example. \ \ thx.

Changed November 14, 2009 03:18AM UTC by dmethvin comment:2

resolution: → duplicate
status: newclosed

Duplicate of #4458.