Missing some sort of "confirmation" for 0 errors, empty summary about tota11y HOT 4 CLOSED

Agreed. A simple "Congratulations. All good!" message would suffice. Are you working on the change?

I read the project, I think a simple detection for 0 errors here: https://github.com/Khan/tota11y/blob/master/plugins/shared/info-panel/index.js#L197 would do the job.
@jdan

from tota11y.

I tried to solve this. Does this look good in case of no error? What do you think? @jdan

from tota11y.

@ankitaggarwal011 Thanks for the mockup :) Design-wise it's pretty solid - i'd like the panel to be a little shorter though. Maybe ditch the panel all together for a small green box in the bottom right corner. Unfortunately it's a little confusing because the landmarks plugin will never report any errors.

I think we'll need a higher-level solution here where plugins can claim whether or not they'll be reporting errors. I'll likely end up working this into #20.

tota11y.addPlugin('my-plugin', {
    reportsErrors: true,
    run: function() {
        // ...
    },
    cleanup: function() {
        // ...
    }
});

from tota11y.

@jdan , Thanks! I think keeping the panel uniform for all plugins would be better UX decision, else It would be confusing. Since, in the test case only the Landmarks plugin showed no errors, I used it to demonstrate the design. However, I have just added code in info-panels/index.js to catch the case where the errors are zero (it's not plugin specific). This will be in all the cases where the number of errors are zero.

I agree that ultimately you should move to the approach where developers can add plugin with a specific API which has the flexibility whether to show summary, errors, or some other tabs they would like.

from tota11y.