Opened 13 years ago
Closed 13 years ago
#5626 closed defect (fixed)
TabContainer: gap under TabContainer on themeTester.html (Safari3/mac)
Reported by: | guest | Owned by: | bill |
---|---|---|---|
Priority: | high | Milestone: | 1.2 |
Component: | Dijit | Version: | 1.0 |
Keywords: | TabContainer, gap | Cc: | |
Blocked By: | Blocking: |
Description
Try http://archive.dojotoolkit.org/nightly/dojotoolkit/dijit/tests/layout/test_TabContainer.html There is intermittently a bug where there's a gap under the tab, maybe same issue as #5591?
Attachments (2)
Change History (17)
comment:1 Changed 13 years ago by
Resolution: | → worksforme |
---|---|
Status: | new → closed |
comment:2 Changed 13 years ago by
Resolution: | worksforme |
---|---|
Status: | closed → reopened |
I'm seeing this in a bunch of places on Safari 3.0.4 against the linked test page.
comment:3 Changed 13 years ago by
Owner: | set to bill |
---|---|
Priority: | normal → high |
Status: | reopened → new |
Must be related to having a built dojo.js file, since it happens for me now too on http://archive.dojotoolkit.org/nightly/dojotoolkit/dijit/tests/layout/test_TabContainer.html, but *doesn't* happen on http://dojotoolkit.org/~bill/svn/dijit/tests/layout/test_TabContainer.html. Of course, that's not so surprising as it's the nature of race conditions.
comment:4 Changed 13 years ago by
On FF I'm seeing a small gap, apparently caused because dojo.css loads too late and thus the font-size: 13px
takes effect after the size of the TabController? has been computed. On Safari there's a big gap, presumably because TabContainer?.css float: left
on the TabButtons? hasn't loaded when we measure the height, and each button is on a separate line.
comment:5 Changed 13 years ago by
More notes: On the build, the CSS actually loads faster since the @imports are mostly collapsed (everything except dojo.css). However, templates are also inlined into the JS files, which means that widget construction also happens faster. A sort of arms-race between the CSS loading and the JS execution.
On FF test_TabContainer.html, the first TabContainer? in the file is set at height 20em, which at the time TabContainer?.layout() runs is 320px but then presumably after loading dojo.css, it becomes 260px. That problem will probably be lessened when #3887 is fixed but still, need someway to defer TabContainer? startup until all the CSS finishes loading.
comment:6 Changed 13 years ago by
Resolution: | → fixed |
---|---|
Status: | new → closed |
Changed 13 years ago by
comment:7 Changed 13 years ago by
Resolution: | fixed |
---|---|
Status: | closed → reopened |
I am still finding this error - screenshot attached. Also in my lasy tutorial a comment on ajaxian confirmed the bug :) http://ajaxian.com/archives/rounded-tabs-with-dijit#comments
comment:8 Changed 13 years ago by
Priority: | high → normal |
---|
comment:10 Changed 13 years ago by
Maybe try polling for all CSS link nodes on the page and check for existence of linkNode.sheet.cssRules. This is used in the current patch for requireCss in ticket #5402. Note that this test should not be done for Firefox because it will error out if the link node's href is on a different path from the page. But perhaps it is enough to use it to get Safari to work. Although, I'm not sure that the existence of cssRules means the rules have been applied. It may or may not. Just have not had a good test to know one way or the other.
comment:11 Changed 13 years ago by
comment:12 Changed 13 years ago by
comment:13 Changed 13 years ago by
comment:14 Changed 13 years ago by
Milestone: | 1.1 → 1.2 |
---|---|
Priority: | normal → high |
Move all milestone 1.1 tickets to 1.2, except for reopened tickets and tickets opened after 1.1RC1 was released.
Note that the fixes above ([13148], [13149], and [13157]) may not be necessary, as the test files are "incorrect" in that they load dojo.js before loading the CSS files, whereas the recommended way is to the opposite. The test files do it strangely to allow on-the-fly theme switching w/out benefit of a server.
comment:15 Changed 13 years ago by
Resolution: | → fixed |
---|---|
Status: | reopened → closed |
Summary: | gap under TabContainer on themeTester.html (Safari3/mac) → TabContainer: gap under TabContainer on themeTester.html (Safari3/mac) |
Fixed by [13696], thanks dante! (This is another ticket that should have been closed automatically but wasn't)
Ugh, can't reproduce this now. Probably same problem as #4058, and could probably be fixed by upping the timer in the code
but as for now can't reproduce. If someone else can reproduce then trying changing 0 to 250 and see if it make a difference.