Skip to main content

Bug Tracker

Side navigation

#8459 closed enhancement (worksforme)

Opened March 07, 2011 11:31AM UTC

Closed April 16, 2011 11:54PM UTC

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

Reported by: cobexer Owned by: rwaldron
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/

Attachments (0)
Change History (3)

Changed March 09, 2011 02:51PM UTC by rwaldron comment:1

component: unfiledeffects
owner: → rwaldron
priority: undecidedlow
status: newassigned
type: bugenhancement

Changed March 09, 2011 02:58PM UTC by cobexer comment:2

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

Changed April 16, 2011 11:54PM UTC by john comment:3

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.