Bug Tracker

Opened 12 years ago

Closed 12 years ago

#8459 closed enhancement (worksforme)

hide/show explicitly set display and leak data on the element

Reported by: Christoph Obexer Owned by: Rick Waldron
Priority: low Milestone: 1.next
Component: effects Version: 1.5.1
Keywords: Cc:
Blocked by: Blocking:

Description

calling hide and show on an element explicitly sets the display property of that element and additionally fails to remove the olddisplay data property

see http://jsfiddle.net/8MdFy/

Change History (3)

comment:1 Changed 12 years ago by Rick Waldron

Component: unfiledeffects
Owner: set to Rick Waldron
Priority: undecidedlow
Status: newassigned
Type: bugenhancement

comment:2 Changed 12 years ago by Christoph Obexer

rwaldron did you set it to enhancement by accident? to me this seems like a bug

comment:3 Changed 12 years ago by john

Resolution: worksforme
Status: assignedclosed

I don't see the problem here. We want to make sure that the display is being explicitly set and it's not being affected by something in a stylesheet somewhere (and checking for that premise would make the code a lot slower). Additionally, we want to maintain olddisplay so that we don't have to re-query for it again later. I think we're OK here. Please feel free to provide another test case that you think demonstrates otherwise.

Note: See TracTickets for help on using tickets.