#17966 closed defect (worksforme)
dijit/layout/BorderContainer is not rendered when gutters is false
| Reported by: | willchb | Owned by: | |
|---|---|---|---|
| Priority: | undecided | Milestone: | tbd |
| Component: | Dijit | Version: | 1.9.3 |
| Keywords: | Cc: | ||
| Blocked By: | Blocking: |
Description
When we set gutters to false, baseClass property is changed and the entire widget doesn't show at all. That happens in postMixInProperties. When gutters is false, baseClass becomes dijitBorderContainerNoGutter.
I can workaround the issue by just commenting out the line that changes baseClass. Even after commenting out that, it seems to work like expected, i.e. borders are not displayed for non-resizable content panes.
Attachments (1)
Change History (7)
comment:1 Changed 6 years ago by
| Resolution: | → worksforme |
|---|---|
| Status: | new → closed |
Changed 6 years ago by
comment:2 Changed 6 years ago by
Before it didn't work because I had this:
.dijitBorderContainer {
width: 100%;
height: 100%;
}
Now I updated it as below and it worked:
.dijitBorderContainer, .dijitBorderContainerNoGutter {
width: 100%;
height: 100%;
}
For more details, find my full code in attachments.
Anyway, CSS class dijitBorderContainerNoGutter doesn't affect anything at all. So at least for now, it's unnecessary. Also, it doesn't make sense to me that sometimes default baseClass of dijit/layout/BorderContainer is other than dijitBorderContainer.
The below snippet from dijit/layout/BorderContainer can be safely removed.
if(!this.gutters){
this.baseClass += "NoGutter";
}
comment:3 Changed 6 years ago by
OK, I guess you are right, I'll remove that. dijit.css also has references to .dijitBorderContainerNoGutter so they should be removed.
It works fine for me, see the third example of http://downloads.dojotoolkit.org/release-1.9.3/dojo-release-1.9.3/dijit/tests/layout/test_BorderContainer.html.
If you have a test case that fails, write a comment back here and attach the test case.