Opened 9 years ago
Closed 9 years ago
#11234 closed defect (fixed)
dojox.widget.Standby does not hide panels contained in a borderContainer
| Reported by: | Jean-Rubin Leonard | Owned by: | Jared Jurkiewicz |
|---|---|---|---|
| Priority: | high | Milestone: | 1.5 |
| Component: | DojoX Widgets | Version: | 1.5.0b2 |
| Keywords: | Standby bordercontainer z-index | Cc: | |
| Blocked By: | Blocking: |
Description
Hi, there's an issue with the dojox.widget.Standby where the overlay does not cover the content pane widget it's been assigned to cover. The target is assigned to the domNode of a contentPane and when I inspect the target with firebug, the right domNode is highlighted. I believe however that there's some issue in the Zindex that causes the underlay to be below the pane which we are trying to hide. I am on firefox 3.6.3 on Mac OS and I am running from trunks. I have attached a test case where the issue is reproduced. JR
Attachments (3)
Change History (9)
Changed 9 years ago by
| Attachment: | standbyBC.html added |
|---|
Changed 9 years ago by
| Attachment: | standbyBC.2.html added |
|---|
comment:1 Changed 9 years ago by
| Resolution: | → invalid |
|---|---|
| Status: | new → closed |
You need to set the zIndex of the Standby overlay to the level you want it at. Its default is auto.
In this case you need to set it higher, such as 100.
The zIndex is explicitly called out in the documentation as user configurable. See: http://docs.dojocampus.org/dojox/widget/Standby
Attached modified form of testcase showing it working once you configure the zIndex.
User error.
comment:2 Changed 9 years ago by
And by the way, you're not supposed to set width/height on 'center' regions. The bordercontainer calculates center regions and sizes them to fill the remaining space. By setting width/height, it tends to screw it up.
comment:3 Changed 9 years ago by
BTW, BorderContainer currently sets the ContentPanes at z-index:2 and the splitters at z-index: 10. That could almost definitely be changed to remove the ContentPane's z-index settings, and set the splitters' z-index: 1.
Maybe even the splitters' z-index setting can be removed... IIRC it was fixing a rendering glitch where the borders of the splitters weren't visible, but nowadays the borders are on the ContentPane itself.
Changed 9 years ago by
| Attachment: | Standby_zIndexCalc.patch added |
|---|
Patch to look up the containing chain to see if there are zIndexes to overlay, improved 'auto' handling.
comment:4 Changed 9 years ago by
| Milestone: | tbd → 1.5 |
|---|---|
| Resolution: | invalid |
| Status: | closed → reopened |
Looks like the zIndex calcing can be improved. Reopening.
comment:5 Changed 9 years ago by
| Owner: | changed from dante to Jared Jurkiewicz |
|---|---|
| Status: | reopened → new |
comment:6 Changed 9 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
Showing setting of zIndex to overlay ABS/zIndex'ed nodes