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

[css-ruby] Add ruby-align tests #14498

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

xfq
Copy link
Contributor

@xfq xfq commented Dec 13, 2018

This PR ports https://w3c.github.io/i18n-tests/results/css-ruby#ruby_align to WPT. Currently, the tests are manual tests, since we're unable to come up with a way to automate them.

The -00X.html tests are in horizontal writing mode, and -10X.html tests are in vertical writing mode. Each test has two variants, long base and long annotation. For space-between and space-around tests, there're variants for one Latin word and multiple Latin word as ruby bases/annotations.

/cc @r12a @himorin

@foolip
Copy link
Member

foolip commented Dec 20, 2018

This PR is blocked on the required "Travis CI - Pull Request" check after #14499. In order to trigger it, I will close and reopen this PR.

@foolip foolip closed this Dec 20, 2018
@foolip foolip reopened this Dec 20, 2018
@upsuper
Copy link
Member

upsuper commented Dec 20, 2018

Why is there no test run in the tasks? Log in Taskcluster shows "Identified 0 affected tests", and those in Travis CI also exited immediately after setting up the browsers.

@upsuper
Copy link
Member

upsuper commented Dec 22, 2018

Also I'd really like to see better reftest reviewing tooling (see #14654), otherwise I feel kinda reluctant to review tens of new tests...

@xfq
Copy link
Contributor Author

xfq commented Dec 23, 2018

These are manual tests, instead of reftests.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants