Opened 15 years ago
Closed 15 years ago
#4334 closed defect (wontfix)
dojo 0.9: dojo.fx.wipeIn() and dojo.fx.wipeIn() cross browser issues
Reported by: | guest | Owned by: | Bryan Forbes |
---|---|---|---|
Priority: | high | Milestone: | |
Component: | fx | Version: | 0.9 |
Keywords: | Cc: | [email protected]… | |
Blocked By: | Blocking: |
Description
In order for a div to properly wipe in and out on various platforms I needed to set its display style to "block" in strategic locations. I'm aware (or rather I presume) that this prevents display:inline elements from being displayed properly, but afaik there is no way of reliably setting the height of an inline element anyway...
Patch (Works for me TM) attached.
Thanks in advance!
Attachments (2)
Change History (11)
Changed 15 years ago by
Attachment: | dojo.fx.patch added |
---|
comment:1 Changed 15 years ago by
comment:3 follow-up: 5 Changed 15 years ago by
Component: | General → fx |
---|---|
Owner: | changed from anonymous to Bryan Forbes |
The bug description is rather strange. You said "In order for a div to properly wipe in and out on various platforms I needed to set its display style to "block"", but a <div> is already block.
Wipe in and wipe out are only supposed to work on block level elements. I don't consider that a bug, just a requirement.
comment:4 Changed 15 years ago by
Replying to alex:
who flied this? Is there a CLA on file?
yes, you should have my CLA, look for hannes<dot>wyss<at>gmail<dot>com
comment:5 Changed 15 years ago by
Replying to bill:
The bug description is rather strange. You said "In order for a div to properly wipe in and out on various platforms I needed to set its display style to "block"", but a <div> is already block.
yes, but if you look at the patch you'll see that the existing code actually sets display = "". And afaics that just doesn't work on several platforms. It's possible that my problem has to do with the block's display attribute being set to "none" initially - the block is invisible, and wiped in later. An example (dojo 0.3, being updated to 0.9) is here: http://www.davaz.com/en/personal/home/pretty
an aside: could you put me (hannes<dot>wyss<at>gmail<dot>com) onto the CC-list? I failed to do so when filing the bug initially and I can't find a way to do it now?
comment:6 Changed 15 years ago by
Cc: | [email protected]… added |
---|
comment:7 Changed 15 years ago by
OK, please attach a small test case against 0.9, using the "attach file" button, and let us know what browser was having problems with
s.display="";
comment:8 Changed 15 years ago by
ok, attaching..
Writing the test-case I was able to drill down a bit further; the problem arises from the fact that I've used the div-id to define display:hidden in an external css-file (or, in the case of the test-case, inline), instead of directly writing a style-property into the tag. Setting style.display = "" then sets the display-property to the ids default, which is "hidden".
Not sure if that still qualifies as a bug...
The testcase demonstrates the problem in Firefox 2.0.0.8 and Opera 9.24, both on Linux x86_64
Changed 15 years ago by
Attachment: | wipeInOut.html added |
---|
comment:9 Changed 15 years ago by
Resolution: | → wontfix |
---|---|
Status: | new → closed |
Ah right, we've had two other bugs about this recently, perhaps from you. It's just a limitation of the system that you can't hide stuff via CSS rules. It's pretty easy to do display="block" in our code but the problem is that things like tables don't work well if you set display to block (on some browsers, anyway).
I forgot: Tested on Linux (Firefox 2.0.0.6, Opera 9.23, Konqueror 3.5.7), Mac OS X (Safari 2.0), Windows XP (IE6, IE7)