Side navigation
#7162 closed enhancement (invalid)
Opened October 12, 2010 12:00PM UTC
Closed October 12, 2010 12:57PM UTC
animate: un-needed code line
| Reported by: | Taai | Owned by: | |
|---|---|---|---|
| Priority: | low | Milestone: | 1.4.3 |
| Component: | effects | Version: | 1.4.2 |
| Keywords: | animate | Cc: | |
| Blocked by: | Blocking: |
Description
It seems that there is an un-needed code line in jQuery 1.4.2 in line 5611.
5611. self.style[ name ] = (end || 1) + unit; // isn't this un-needed ???
5612. start = ((end || 1) / e.cur(true)) * start;
5613. self.style[ name ] = start + unit; // because statement is set here again !!!
Attachments (0)
Change History (1)
Changed October 12, 2010 12:57PM UTC by comment:1
| resolution: | → invalid |
|---|---|
| status: | new → closed |
Umm, no. If you look at the middle line we're calling e.cur(true) which is getting us the computed value of the element for that property. After it's done we reset the display to the original value.