Bug Tracker

Opened 12 years ago

Closed 12 years ago

Last modified 12 years ago

#10622 closed bug (fixed)

.show() does not properly restore CSS-set "display" value

Reported by: warrenparsons Owned by: Rick Waldron
Priority: blocker Milestone: 1.7
Component: effects Version: 1.7rc1
Keywords: Cc:
Blocked by: Blocking:


I've run into an issue in RC1 (and seemingly B2 as well) where using .show() does not restore an element's original, CSS-specified "display" value. Instead, it sets the element's "display" type to that element's browser-default value.

B1 does not have this issue for me.

I've set up a test case on jsfiddle at: http://jsfiddle.net/6JbHh/

Change History (5)

comment:1 Changed 12 years ago by Rick Waldron

Component: unfiledeffects
Milestone: None1.7
Owner: set to Rick Waldron
Priority: undecidedblocker
Status: newassigned

Confirmed: Regression.

Patch to follow.

Last edited 12 years ago by Rick Waldron (previous) (diff)

comment:2 Changed 12 years ago by Rick Waldron

I believe this was caused by the patch that timmywil and I wrote here: https://github.com/jquery/jquery/commit/22f2e8b3dc18dede5f1ccb28cbdf8cb5bcde115f

comment:3 Changed 12 years ago by Rick Waldron

Further reduced, easier to understand: http://jsfiddle.net/rwaldron/s7k4D/

comment:4 Changed 12 years ago by Timmy Willison

Resolution: fixed
Status: assignedclosed

Revert "Landing pull request 530. Fixes coniditional path for tr, td defaultDisplay() calls. Fixes #10416." Fixes #10622.

This reverts commit 22f2e8b3dc18dede5f1ccb28cbdf8cb5bcde115f.

Changeset: 0752687612d190f608e64181148ced1a4adfa5d6

comment:10 Changed 12 years ago by Rick Waldron

#10628 is a duplicate of this ticket.

Note: See TracTickets for help on using tickets.