Skip to content

Commit

Permalink
feat(landmark-banner-is-top-level: add rule ensuring top level banner
Browse files Browse the repository at this point in the history
  • Loading branch information
sulsanaul authored and WilcoFiers committed Mar 7, 2018
1 parent 3e2acb1 commit 7c0cf8e
Show file tree
Hide file tree
Showing 12 changed files with 335 additions and 0 deletions.
1 change: 1 addition & 0 deletions doc/rule-descriptions.md
Original file line number Diff line number Diff line change
Expand Up @@ -34,6 +34,7 @@
| input-image-alt | Ensures <input type="image"> elements have alternate text | cat.text-alternatives, wcag2a, wcag111, section508, section508.22.a | true |
| label-title-only | Ensures that every form element is not solely labeled using the title or aria-describedby attributes | cat.forms, best-practice | true |
| label | Ensures every form element has a label | cat.forms, wcag2a, wcag332, wcag131, section508, section508.22.n | true |
| landmark-banner-is-top-level | A banner landmark identifies site-oriented content at the beginning of each page within a website | best-practice | true |
| landmark-main-is-top-level | The main landmark should not be contained in another landmark | best-practice | true |
| landmark-no-duplicate-banner | Ensures the document has no more than one banner landmark | best-practice | true |
| landmark-no-duplicate-contentinfo | Ensures the document has no more than one contentinfo landmark | best-practice | true |
Expand Down
19 changes: 19 additions & 0 deletions lib/checks/keyboard/banner-is-top-level.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,19 @@
const landmarks = axe.commons.aria.getRolesByType('landmark');
const sectioning = ['article', 'aside', 'main', 'navigation', 'section'];
const nodeIsHeader = node.tagName.toLowerCase() === 'header' && node.getAttribute('role') !== 'banner';
var parent = axe.commons.dom.getComposedParent(node);

while (parent){
var role = parent.getAttribute('role');
if (!role && (parent.tagName.toLowerCase() !== 'form')){
role = axe.commons.aria.implicitRole(parent);
}
if (role && nodeIsHeader && sectioning.includes(role)){
return true;
}
if (role && landmarks.includes(role)){
return false;
}
parent = axe.commons.dom.getComposedParent(parent);
}
return true;
11 changes: 11 additions & 0 deletions lib/checks/keyboard/banner-is-top-level.json
Original file line number Diff line number Diff line change
@@ -0,0 +1,11 @@
{
"id": "banner-is-top-level",
"evaluate": "banner-is-top-level.js",
"metadata": {
"impact": "moderate",
"messages": {
"pass": "Banner landmark is top level or header element is not a banner",
"fail": "Banner landmark is not top level"
}
}
}
16 changes: 16 additions & 0 deletions lib/rules/landmark-banner-is-top-level.json
Original file line number Diff line number Diff line change
@@ -0,0 +1,16 @@
{
"id": "landmark-banner-is-top-level",
"selector": "[role=banner], header",
"tags": [
"best-practice"
],
"metadata": {
"description": "A banner landmark identifies site-oriented content at the beginning of each page within a website",
"help": "Banner landmark must be at top level"
},
"all": [],
"any": [
"banner-is-top-level"
],
"none": []
}
89 changes: 89 additions & 0 deletions test/checks/keyboard/banner-is-top-level.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,89 @@
describe('banner-is-top-level', function () {
'use strict';

var fixture = document.getElementById('fixture');

var checkSetup = axe.testUtils.checkSetup;
var shadowSupported = axe.testUtils.shadowSupport.v1;

afterEach(function () {
fixture.innerHTML = '';
});

it('should return false if banner landmark is in main element', function() {
var main = document.createElement('main');
var banner = document.createElement('div');
banner.setAttribute('role','banner');
main.appendChild(banner);
fixture.appendChild(main);
assert.isFalse(checks['banner-is-top-level'].evaluate(banner));
});

it('should return false if banner landmark is in main element', function () {
var main = document.createElement('main');
var banner = document.createElement('div');
banner.setAttribute('role','banner');
main.appendChild(banner);
fixture.appendChild(main);
assert.isFalse(checks['banner-is-top-level'].evaluate(banner));
});

it('should return false if banner landmark is in div with role main', function () {
var main = document.createElement('div');
main.setAttribute('role','main');
var banner = document.createElement('div');
banner.setAttribute('role','banner');
main.appendChild(banner);
fixture.appendChild(main);
assert.isFalse(checks['banner-is-top-level'].evaluate(banner));
});

it('should return false if header is not sectioning element and in div with role search', function () {
var search = document.createElement('div');
search.setAttribute('role','search');
var banner = document.createElement('header');
search.appendChild(banner);
fixture.appendChild(search);
assert.isFalse(checks['banner-is-top-level'].evaluate(banner));
});


it('should return true if banner landmark is not contained in another landmark', function () {
var banner = document.createElement('div');
banner.setAttribute('role','banner');
fixture.appendChild(banner);
assert.isTrue(checks['banner-is-top-level'].evaluate(banner));
});

it('should return true if header element is not sectioning element and not contained in landmark', function () {
var header = document.createElement('header');
fixture.appendChild(header);
assert.isTrue(checks['banner-is-top-level'].evaluate(header));
});

it('should return true if header element is in sectioning element', function () {
var header = document.createElement('header');
var article = document.createElement('div');
article.setAttribute('role', 'main');
article.appendChild(header);
fixture.appendChild(article);
assert.isTrue(checks['banner-is-top-level'].evaluate(header));
});

(shadowSupported ? it : xit)('should test if banner in shadow DOM is top level', function () {
var div = document.createElement('div');
var shadow = div.attachShadow({ mode: 'open' });
shadow.innerHTML = '<div role="banner">Banner landmark</div>';
var checkArgs = checkSetup(shadow.querySelector('[role=banner], header'));
assert.isTrue(checks['banner-is-top-level'].evaluate.apply(null, checkArgs));
});

(shadowSupported ? it : xit)('should test if header in shadow DOM is top level', function () {
var div = document.createElement('div');
var shadow = div.attachShadow({ mode: 'open' });
shadow.innerHTML = '<header>Banner landmark</header>';
var checkArgs = checkSetup(shadow.querySelector('[role=banner], header'));
assert.isTrue(checks['banner-is-top-level'].evaluate.apply(null, checkArgs));
});

});
Original file line number Diff line number Diff line change
@@ -0,0 +1,15 @@
<!doctype html>
<html id="violation2">
<head>
<meta charset="utf8">
<script src="/axe.js"></script>
</head>
<body>
<p>This iframe should fail, too</p>
<div role="complementary">
<header>
<p>This banner landmark is in a complementary landmark</p>
</header>
</div>
</body>
</html>
Original file line number Diff line number Diff line change
@@ -0,0 +1,27 @@
<!doctype html>
<html id="pass2">
<head>
<meta charset="utf8">
<script src="/axe.js"></script>
</head>
<body>
<p>This iframe should pass, too</p>

<div role="navigation">
<p>This div has role navigation</p>
</div>
<header>
<p>This banner content is not within another landmark</p>
</header>
<div role="complementary">
<p>This div has role complementary</p>
</div>
<div role="search">
<p>This div has role search</p>
</div>
<div role="form">
<p>This div has role form<p>
</div>
<iframe id="frame2" src="level2.html"></iframe>
</body>
</html>
Original file line number Diff line number Diff line change
@@ -0,0 +1,15 @@
<!doctype html>
<html id="pass3">
<head>
<meta charset="utf8">
<script src="/axe.js"></script>
</head>
<body>
<p>This iframe should pass<p>
<main>
<header>
<p>This header is in a main landmark</p>
</header>
</main>
</body>
</html>
Original file line number Diff line number Diff line change
@@ -0,0 +1,29 @@
<!doctype html>
<html lang="en" id="violation1">
<head>
<title>landmark-banner-is-top-level test</title>
<meta charset="utf8">
<link rel="stylesheet" type="text/css" href="/node_modules/mocha/mocha.css" />
<script src="/node_modules/mocha/mocha.js"></script>
<script src="/node_modules/chai/chai.js"></script>
<script src="/axe.js"></script>
<script>
mocha.setup({
timeout: 10000,
ui: 'bdd'
});
var assert = chai.assert;
</script>
</head>
<body>
<div role="navigation">
<div role="banner">
<p>This is going to fail</p>
</div>
</div>
<iframe id="frame1" src="frames/level1-fail.html"></iframe>
<div id="mocha"></div>
<script src="landmark-banner-is-top-level-fail.js"></script>
<script src="/test/integration/adapter.js"></script>
</body>
</html>
Original file line number Diff line number Diff line change
@@ -0,0 +1,40 @@
describe('landmark-banner-is-top-level test fail', function () {
'use strict';
var results;
before(function (done) {
window.addEventListener('load', function () {
axe.run({ runOnly: { type: 'rule', values: ['landmark-banner-is-top-level'] } }, function (err, r) {
assert.isNull(err);
results = r;
done();
});
});
});

describe('violations', function () {
it('should find 1', function () {
assert.lengthOf(results.violations, 1);
});

it('should find 2 nodes', function () {
assert.lengthOf(results.violations[0].nodes, 2);
});
});

describe('passes', function () {
it('should find none', function () {
assert.lengthOf(results.passes, 0);
});

});


it('should find 0 inapplicable', function () {
assert.lengthOf(results.inapplicable, 0);
});

it('should find 0 incomplete', function () {
assert.lengthOf(results.incomplete, 0);
});

});
Original file line number Diff line number Diff line change
@@ -0,0 +1,39 @@
<!doctype html>
<html lang="en" id="pass1">
<head>
<title>landmark-banner-is-top-level test</title>
<meta charset="utf8">
<link rel="stylesheet" type="text/css" href="/node_modules/mocha/mocha.css" />
<script src="/node_modules/mocha/mocha.js"></script>
<script src="/node_modules/chai/chai.js"></script>
<script src="/axe.js"></script>
<script>
mocha.setup({
timeout: 10000,
ui: 'bdd'
});
var assert = chai.assert;
</script>
</head>
<body>
<div role="navigation">
<p>This div has role navigation</p>
</div>
<div role="banner">
<p>This banner content is not within another landmark</p>
</div>
<div role="complementary">
<p>This div has role complementary</p>
</div>
<div role="search">
<p>This div has role search</p>
</div>
<div role="form">
<p>This div has role form<p>
</div>
<iframe id="frame1" src="frames/level1.html"></iframe>
<div id="mocha"></div>
<script src="landmark-banner-is-top-level-pass.js"></script>
<script src="/test/integration/adapter.js"></script>
</body>
</html>
Original file line number Diff line number Diff line change
@@ -0,0 +1,34 @@
describe('landmark-banner-is-top-level test pass', function () {
'use strict';
var results;
before(function (done) {
window.addEventListener('load', function () {
axe.run({ runOnly: { type: 'rule', values: ['landmark-banner-is-top-level'] } }, function (err, r) {
assert.isNull(err);
results = r;
done();
});
});
});

describe('violations', function () {
it('should find 0', function () {
assert.lengthOf(results.violations, 0);
});
});

describe('passes', function () {
it('should find 3', function () {
assert.lengthOf(results.passes[0].nodes, 3);
});
});

it('should find 0 inapplicable', function () {
assert.lengthOf(results.inapplicable, 0);
});

it('should find 0 incomplete', function () {
assert.lengthOf(results.incomplete, 0);
});

});

0 comments on commit 7c0cf8e

Please sign in to comment.