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

Avoid visual comparisons with lang="ja-jp" #6926

Merged
merged 1 commit into from Sep 1, 2017

Conversation

kahsieh
Copy link
Member

@kahsieh kahsieh commented Aug 17, 2017

The reference file attribute-value-selector-001-ref.xht does not specify lang="ja-jp" on the first div while this file does. This causes the two pages to look different because a Japanese font is selected in this file. Changing "ja-jp" to a language written in Latin letters, "fr-fr", avoids this problem.

@w3c-bots
Copy link

w3c-bots commented Aug 17, 2017

Build PASSED

Started: 2017-08-18 00:01:48
Finished: 2017-08-18 00:04:16

View more information about this build on:

@w3c-bots
Copy link

Build PASSED

Started: 2017-08-17 20:28:37
Finished: 2017-08-17 20:37:43

View more information about this build on:

Copy link
Member

@dbaron dbaron left a comment

Choose a reason for hiding this comment

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

Seems like that's fine if you're running an English build of a browser, but what if you're running a Japanese build? Perhaps it should instead specify the same language on both test and reference?

@gsnedders
Copy link
Member

Or it should be rewritten to not use the lang attribute and avoid this quirkiness altogether.

As an aside, I can't see anything in any spec that would seem to permit what browsers actually do with language-specific generics; spec bug forthcoming.

@kahsieh
Copy link
Member Author

kahsieh commented Aug 17, 2017

For now, if we want to have such consistency, more extensive changes would be needed (please see revised commit).

The reference file `attribute-value-selector-001-ref.xht` does not specify `lang="ja-jp"` on the first `div` while this file does. This causes the two pages to look different because a Japanese font is selected in this file. Changing "ja-jp" to a language written in Latin letters, "fr-fr", avoids this problem.
@sideshowbarker
Copy link
Contributor

w3c-test:mirror

@kojiishi
Copy link
Contributor

kojiishi commented Sep 1, 2017

@dbaron's comment above looks reasonable, was that addressed? I can see only 1 commit.

@kahsieh
Copy link
Member Author

kahsieh commented Sep 1, 2017

Yes, it has been addressed. I applied the same language tags to the reference and all three tests that use it.

@w3c-bots
Copy link

w3c-bots commented Sep 1, 2017

Build PASSED

Started: 2017-08-18 00:01:48
Finished: 2017-08-18 00:04:16

Failing Jobs

  • build_css

View more information about this build on:

@dbaron dbaron merged commit 4fc75f4 into web-platform-tests:master Sep 1, 2017
rachelandrew pushed a commit to rachelandrew/web-platform-tests that referenced this pull request Nov 8, 2017
The reference file `attribute-value-selector-001-ref.xht` does not specify `lang="ja-jp"` on the first `div` while this file does. This causes the two pages to look different because a Japanese font is selected in this file. Changing "ja-jp" to a language written in Latin letters, "fr-fr", avoids this problem.
jakearchibald pushed a commit to jakearchibald/web-platform-tests that referenced this pull request Nov 16, 2017
The reference file `attribute-value-selector-001-ref.xht` does not specify `lang="ja-jp"` on the first `div` while this file does. This causes the two pages to look different because a Japanese font is selected in this file. Changing "ja-jp" to a language written in Latin letters, "fr-fr", avoids this problem.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants