Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Section508 - Unknown Provider #18

Open
JCron245 opened this issue Oct 25, 2016 · 3 comments
Open

Section508 - Unknown Provider #18

JCron245 opened this issue Oct 25, 2016 · 3 comments
Assignees
Labels

Comments

@JCron245
Copy link

For some reason the Phantom.js file that comes with the gulp plugin is missing the proper switch statement to support Section508

Here is the relevant portion of code in the phantom.js file in accesssniff included with the gulp plugin

  switch (options) {
    case 'WCAG2A':
      page.evaluate(function () {
        return HTMLCS_RUNNER.run('WCAG2A');
      });
      break;
    case 'WCAG2AA':
      page.evaluate(function () {
        return HTMLCS_RUNNER.run('WCAG2AA');
      });
      break;
    case 'WCAG2AAA':
      page.evaluate(function () {
        return HTMLCS_RUNNER.run('WCAG2AAA');
      });
      break;
    default:
      console.log('Unknown standard.');
  }

and here is the switch statement from the accesssniff project

  switch (options) {
    case 'WCAG2A':
      page.evaluate(function () {
        return HTMLCS_RUNNER.run('WCAG2A');
      });
      break;
    case 'WCAG2AA':
      page.evaluate(function () {
        return HTMLCS_RUNNER.run('WCAG2AA');
      });
      break;
    case 'WCAG2AAA':
      page.evaluate(function () {
        return HTMLCS_RUNNER.run('WCAG2AAA');
      });
      break;
    case 'Section508':
      page.evaluate(function () {
        return HTMLCS_RUNNER.run('Section508');
      });
      break;
    default:
      console.log('Unknown standard.');
  }

The version number in the accesssniff files pulled into the gulp plugin seem to indicate it is the latest version so I am not quite sure what is happening here.

@JCron245
Copy link
Author

It looks like the relevant Section508 stuff in the HTMLCS stuff is missing as well.

@yargalot
Copy link
Owner

@JCron245 Just pushed out 2.4.5 for AccessSniff, Totally forgot to publish it.

Can you give it a burl and see if section508 works?

@JCron245
Copy link
Author

It's doing a Section508 check now but I can't actually get any html to violate it, so it looks like something isn't working. I'll do some more testing with html that should violate 508.

@yargalot yargalot self-assigned this Oct 26, 2016
@yargalot yargalot added the bug label Oct 26, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants