Bug 16797
Summary: | CSS3: cursor: default not supported (Acid3 bug) | ||
---|---|---|---|
Product: | WebKit | Reporter: | Eric Seidel (no email) <eric> |
Component: | CSS | Assignee: | Eric Seidel (no email) <eric> |
Status: | RESOLVED INVALID | ||
Severity: | Normal | CC: | ap, ian |
Priority: | P2 | ||
Version: | 528+ (Nightly build) | ||
Hardware: | Mac | ||
OS: | OS X 10.4 |
Eric Seidel (no email)
Test 47: FAIL (expected default, got: auto - cursor default not supported)
function () {
// test 47: 'cursor' and CSS3 values
var doc = getTestDocument();
var style = doc.createElement('style');
style.setAttribute('type', 'text/css');
var cursors = ['auto', 'default', 'none', 'context-menu', 'help', 'pointer', 'progress', 'wait', 'cell', 'crosshair', 'text', 'vertical-text', 'alias', 'copy', 'move', 'no-drop', 'not-allowed', 'e-resize', 'n-resize', 'ne-resize', 'nw-resize', 's-resize', 'se-resize', 'sw-resize', 'w-resize', 'ew-resize', 'ns-resize', 'nesw-resize', 'nwse-resize', 'col-resize', 'row-resize', 'all-scroll'];
for (var i in cursors) {
var c = cursors[i];
style.appendChild(doc.createTextNode('#' + c + ' { cursor: ' + c + '; }'));
}
style.appendChild(doc.createTextNode('#bogus { cursor: bogus; }'));
doc.body.appendChild(style);
doc.body.id = "bogus";
assertEquals(doc.defaultView.getComputedStyle(doc.body, '').cursor, "auto", "control failed");
for (var i in cursors) {
var c = cursors[i];
doc.body.id = c;
assertEquals(doc.defaultView.getComputedStyle(doc.body, '').cursor, c, "cursor " + c + " not supported");
}
return 3;
},
Attachments | ||
---|---|---|
Add attachment proposed patch, testcase, etc. |
Eric Seidel (no email)
Should be simple. This should just map to "pointer" for us.
Eric Seidel (no email)
Actually this seems to have nothing to do with cursor: default support. Investigating.
Eric Seidel (no email)
So the major issue here is that Hixie is adding the <style> tag to the body:
doc.body.appendChild(style);
instead of to the head (like he does for every other test). We don't support that for some reason.
We also seem to incorrectly map none -> crosshair and copy -> auto. I'll fix those in a forthcoming patch.
Eric Seidel (no email)
None of these fail in the latest Acid3. My diagnosis was incorrect.