diff options
Diffstat (limited to 'guidelines/accessibility/guideline-1.js')
| -rw-r--r-- | guidelines/accessibility/guideline-1.js | 38 |
1 files changed, 31 insertions, 7 deletions
diff --git a/guidelines/accessibility/guideline-1.js b/guidelines/accessibility/guideline-1.js index fdb2c42..61e1384 100644 --- a/guidelines/accessibility/guideline-1.js +++ b/guidelines/accessibility/guideline-1.js @@ -42,10 +42,12 @@ exports.rule = function(report) { // TODO: what's the best way to classify content that's `complex`? - // All `area` elements of an image map should have alt attributes. This isn't - // quite a faithful implementation, as it doesn't take into account the case - // where an `a` tag is wrapped around the `area` tag. - $5('map').find('area').each(hasAlt); + // All `area` elements of an image map should have alt attributes. It's also a + // bit overzealous, as it looks at all maps, not just maps that are referenced + // from images. + $5('map').find('area') + .add($5('map').find('a')) + .each(hasAlt); // TODO: figure out a good way to handle frames. // TODO: figure out a good way to handle scripts. @@ -53,19 +55,41 @@ exports.rule = function(report) { /* Checkpoint 1.2 [Priority 1] **********************************************/ - // TODO + // `ismap` is a boolean attribute. If it's present on an image, require that + // there's also a corresponding `usemap` attribute that can be used in lieu of + // having the server information present. This is a bit of an under + // approximation, as if you can provide a client-side map that doesn't cover + // everything that the server does. It's more of a sanity check, that this + // has been thought of. + $5('img').filter('[ismap]').each(function(ix) { + if(_.isEmpty($(this).attr('usemap'))) { + report.error('No usemap attribute to supplement a use of ismap', this); + } + }); /* Checkpoint 1.3 [Priority 1] **********************************************/ - // TODO + // TODO: Not really sure if this is something that we can check; the guideline + // seems to be more of a subjective check. /* Checkpoint 1.4 [Priority 1] **********************************************/ - // TODO + // TODO: Again, not sure if this is something we can check here. /* Checkpoint 1.5 [Priority 3] **********************************************/ + // Make sure that every link in an image map has a corresponding text link + // somewhere else in the document. + var hrefs = $5('a').map(function() { return $(this).attr('href'); }); + $5('map').find('area').each(function() { + var href = $(this).attr('href'); + if(!_.contains(href, hrefs)) { + report.error('Image map contains a link not present in a text link', + this); + } + }); + }; |