Skip to content

Conversation

@reidmore-online
Copy link
Contributor

Updated test results for input-number for all desktop browser + SR combinations.

Some changed test results.

This PR closes issues #261, #260, and #259.

Copy link
Contributor

@mfairchild365 mfairchild365 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for your work on this! please see my comments :)

"feature_assertion_id": "convey_name",
"result": "pass",
"notes": "announced the name when entering the input and did not automatically switch to forms mode."
"result": "fail",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ouch. that's a bummer, but can confirm. Other inputs are fine too.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm wondering if this is intentional though, as when I navigated this way it went from [input label] to [input], so I never missed the label, it just doesn't get a label on its own when you land on it.

"feature_assertion_id": "convey_change_in_value",
"result": "pass"
"result": "fail",
"notes": "each keypress was echoed and the value was changed to \"two11\""
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this might pass. If FF lets you enter invalid characters but still echos them. What do you mean by the value was changed to "two11" - does that match what you typed?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It matches what I typed, I was able to both add text before the value "two11" but also replace the value completely. If its a pass I can switch it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

tech/html/input/input-number narrator/edge tech/html/input/input-number nvda/edge tech/html/input/input-number nvda/chrome

3 participants