role="grid" is not a valid role

[Alonski]

When upgrading an app using this addon the new Ember Template Lint rule no-redundant-role was autofixing code:

  <table
    role="grid"
    {{aria-grid}}
    class="w-full table-fixed {{if @isEmpty 'h-full'}}"
    data-test-selector="table"
  >

To:

  <table
    {{aria-grid}}
    class="w-full table-fixed {{if @isEmpty 'h-full'}}"
    data-test-selector="table"
  >

This causes the assertion that {{aria-grid}} can only be applied to elements with role="grid" to error locally and in tests.

Not sure what the fix is here. Locally I am going to disable this rule.

[joelamb]

Is this an ember-aria-utilities issue, or an ember-template-lint issue?

I'm not sure that role="grid" should be removed by ember template lint.

The implicit role for a table is "table", but any role is allowed (ref: https://www.w3.org/TR/html-aria/#docconformance), so I don't think role="grid" is redundant.

[Alonski]

Ok so maybe the rule is wrong?

[NullVoxPopuli]

Yeah, the rule needs updating. The role is required here