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

AccName: Fix several expected results #10321

Merged
merged 1 commit into from Apr 5, 2018

Conversation

Projects
None yet
4 participants
@joanmarie
Contributor

joanmarie commented Apr 4, 2018

No description provided.

@joanmarie joanmarie self-assigned this Apr 4, 2018

@joanmarie joanmarie requested a review from halindrome Apr 4, 2018

@wpt-pr-bot wpt-pr-bot added the accname label Apr 4, 2018

@joanmarie

This comment has been minimized.

Contributor

joanmarie commented Apr 4, 2018

Regarding the results "a test This is", that is the correct result when the AccName algorithm is applied to the test case. (Which is what we're testing here.) So, yes, it might look weird as far as prose (or authored content) is concerned, but it is not a mistake. There is also a similar test case where the elements are reversed and the more normal-looking "This is a test" is the result.

@w3c-bots

This comment has been minimized.

w3c-bots commented Apr 5, 2018

Build PASSED

Started: 2018-04-05 00:08:06
Finished: 2018-04-05 00:16:58

View more information about this build on:

@Ms2ger

Ms2ger approved these changes Apr 5, 2018

@Ms2ger Ms2ger merged commit 08906cb into web-platform-tests:master Apr 5, 2018

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment