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

Incorrect validation on aria-setsize="-1" #1158

Closed
jaimeiniesta opened this issue Jun 2, 2021 · 0 comments
Closed

Incorrect validation on aria-setsize="-1" #1158

jaimeiniesta opened this issue Jun 2, 2021 · 0 comments

Comments

@jaimeiniesta
Copy link
Contributor

URL being validated or code to reproduce error:

<!DOCTYPE html>
<html lang="">
<head>
<title>Test</title>
</head>
<body>
<h2 id="label_fruit"> Available Fruit </h2>
<ul role="listbox" aria-labelledby="label_fruit">
<li role="option" aria-setsize="-1"> apples </li>
</ul>
</body>
</html>

Produces this error:

Error: Bad value -1 for attribute aria-setsize on element li: Expected a digit but saw - instead.

From line 9, column 1; to line 9, column 36

l_fruit"><li role="option" aria-setsize="-1"> apple

Which I believe should not be an error because in https://www.w3.org/TR/wai-aria-1.2/#aria-setsize it says:

Authors MUST set the value of aria-setsize to an integer equal to the number of items in the set. If the total number of items is unknown, authors SHOULD set the value of aria-setsize to -1.

rdeltour added a commit to w3c/epubcheck that referenced this issue Jul 8, 2022
---------------------

docs: switch to higher quality reference for alternative text

Co-authored-by: Jens Oliver Meiert <[email protected]>

---------------------

switch to higher quality reference for alternative text

fixes #1146

Co-authored-by: Steve Faulkner <[email protected]>

---------------------

ARIA: Allow aria-required with special input types

Add aria-required to input types datetime-local / date / month / time / week

Co-authored-by: Daniel <[email protected]>

---------------------

Allow rel attribute on SVG <a> element

Fixes validator/validator#1008
Closes validator/validator#1009

Co-authored-by: MattGarrish <[email protected]>

---------------------

Allow any ARIA role for <svg> element

Fixes validator/validator#1096
Fixes validator/validator#981

Co-authored-by: Michael[tm] Smith <[email protected]>

---------------------

Allow aria.prop.posinset|setsize w/ role=tab

Fixes validator/validator#1060

Co-authored-by: Michael[tm] Smith <[email protected]>

---------------------

Allow “aria-placeholder” where “placeholder” is allowed

Also, report error if element with “placeholder” also has
“aria-placeholder”.

Fixes validator/validator#1098

Co-authored-by: Michael[tm] Smith <[email protected]>

---------------------

Allow only presentation|none for br[role]

Fixes validator/validator#1097

Co-authored-by: Michael[tm] Smith <[email protected]>

---------------------

Update required states for input[type=radio|checkbox]

Fixes validator/validator#1109

Co-authored-by: Michael[tm] Smith <[email protected]>

---------------------

Update allowed roles for figure element

Fixes validator/validator#1108

Co-authored-by: Michael[tm] Smith <[email protected]>

---------------------

Allow th[aria-rowspan] & th[aria-colspan]

Relates to validator/validator#1107

Co-authored-by: Michael[tm] Smith <[email protected]>

---------------------

Allow math element to have global ARIA attributes

Fixes validator/validator#1114

Co-authored-by: Michael[tm] Smith <[email protected]>

---------------------

Add menu, menubar, and tablist to allowed nav roles

Fixes validator/validator#1110

Co-authored-by: Michael[tm] Smith <[email protected]>

---------------------

Align aria-valuemin checking with current spec

Fixes validator/validator#1120

Co-authored-by: Michael[tm] Smith <[email protected]>

---------------------

Align aria-valuemax checking with current spec

Fixes validator/validator#1119

Co-authored-by: Michael[tm] Smith <[email protected]>

---------------------

Allow aria-description

Fixes validator/validator#1155

Co-authored-by: Michael[tm] Smith <[email protected]>

---------------------

Allow -1 for aria-setsize

Fixes validator/validator#1158

Co-authored-by: Michael[tm] Smith <[email protected]>

---------------------

update the allowed roles for wbr

update allowed roles for `wbr` to match `br` element

Co-authored-by: Scott O'Hara <[email protected]>
rdeltour added a commit to w3c/epubcheck that referenced this issue Aug 18, 2022
---------------------

docs: switch to higher quality reference for alternative text

Co-authored-by: Jens Oliver Meiert <[email protected]>

---------------------

switch to higher quality reference for alternative text

fixes #1146

Co-authored-by: Steve Faulkner <[email protected]>

---------------------

ARIA: Allow aria-required with special input types

Add aria-required to input types datetime-local / date / month / time / week

Co-authored-by: Daniel <[email protected]>

---------------------

Allow rel attribute on SVG <a> element

Fixes validator/validator#1008
Closes validator/validator#1009

Co-authored-by: MattGarrish <[email protected]>

---------------------

Allow any ARIA role for <svg> element

Fixes validator/validator#1096
Fixes validator/validator#981

Co-authored-by: Michael[tm] Smith <[email protected]>

---------------------

Allow aria.prop.posinset|setsize w/ role=tab

Fixes validator/validator#1060

Co-authored-by: Michael[tm] Smith <[email protected]>

---------------------

Allow “aria-placeholder” where “placeholder” is allowed

Also, report error if element with “placeholder” also has
“aria-placeholder”.

Fixes validator/validator#1098

Co-authored-by: Michael[tm] Smith <[email protected]>

---------------------

Allow only presentation|none for br[role]

Fixes validator/validator#1097

Co-authored-by: Michael[tm] Smith <[email protected]>

---------------------

Update required states for input[type=radio|checkbox]

Fixes validator/validator#1109

Co-authored-by: Michael[tm] Smith <[email protected]>

---------------------

Update allowed roles for figure element

Fixes validator/validator#1108

Co-authored-by: Michael[tm] Smith <[email protected]>

---------------------

Allow th[aria-rowspan] & th[aria-colspan]

Relates to validator/validator#1107

Co-authored-by: Michael[tm] Smith <[email protected]>

---------------------

Allow math element to have global ARIA attributes

Fixes validator/validator#1114

Co-authored-by: Michael[tm] Smith <[email protected]>

---------------------

Add menu, menubar, and tablist to allowed nav roles

Fixes validator/validator#1110

Co-authored-by: Michael[tm] Smith <[email protected]>

---------------------

Align aria-valuemin checking with current spec

Fixes validator/validator#1120

Co-authored-by: Michael[tm] Smith <[email protected]>

---------------------

Align aria-valuemax checking with current spec

Fixes validator/validator#1119

Co-authored-by: Michael[tm] Smith <[email protected]>

---------------------

Allow aria-description

Fixes validator/validator#1155

Co-authored-by: Michael[tm] Smith <[email protected]>

---------------------

Allow -1 for aria-setsize

Fixes validator/validator#1158

Co-authored-by: Michael[tm] Smith <[email protected]>

---------------------

update the allowed roles for wbr

update allowed roles for `wbr` to match `br` element

Co-authored-by: Scott O'Hara <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants