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
Attachments (0)
Change History (3)
Changed March 09, 2011 02:51PM UTC by comment:1
component: | unfiled → effects |
---|---|
owner: | → rwaldron |
priority: | undecided → low |
status: | new → assigned |
type: | bug → enhancement |
Changed March 09, 2011 02:58PM UTC by 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 comment:3
resolution: | → worksforme |
---|---|
status: | assigned → closed |
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.
http://jsfiddle.net/rwaldron/8MdFy/2/