Bug Tracker

Opened 5 years ago

Closed 5 years ago

Last modified 5 years ago

#10622 closed bug (fixed)

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

Reported by: warrenparsons Owned by: rwaldron
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 5 years ago by rwaldron

  • Component changed from unfiled to effects
  • Milestone changed from None to 1.7
  • Owner set to rwaldron
  • Priority changed from undecided to blocker
  • Status changed from new to assigned

Confirmed, regression.

Version 0, edited 5 years ago by rwaldron (next)

comment:2 Changed 5 years ago by rwaldron

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

comment:3 Changed 5 years ago by rwaldron

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

comment:4 Changed 5 years ago by timmywil

  • Resolution set to fixed
  • Status changed from assigned to closed

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 5 years ago by rwaldron

#10628 is a duplicate of this ticket.

Note: See TracTickets for help on using tickets.