WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
NEW
Bug 270341
REGRESSION(
270647@main
): vertical form controls causing WPT failures
https://bugs.webkit.org/show_bug.cgi?id=270341
Summary
REGRESSION(270647@main): vertical form controls causing WPT failures
Sam Sneddon [:gsnedders]
Reported
2024-03-01 05:03:29 PST
This may well largely be a test development task, but: * /css/css-align/baseline-rules/grid-item-input-type-text.html:
https://wpt.fyi/results/css/css-align/baseline-rules/grid-item-input-type-text.html?label=master&product=safari%5Bstable%5D&product=safari%5Bexperimental%5D&aligned&diff&filter=ADC
* /css/css-flexbox/flex-item-compressible-002.html:
https://wpt.fyi/results/css/css-flexbox/flex-item-compressible-002.html?label=master&product=safari%5Bstable%5D&product=safari%5Bexperimental%5D&aligned&diff&filter=ADC
* /css/css-inline/baseline-source/baseline-source-first-textarea-002.tentative.html:
https://wpt.fyi/results/css/css-inline/baseline-source/baseline-source-first-textarea-002.tentative.html?label=master&product=safari%5Bstable%5D&product=safari%5Bexperimental%5D&aligned&diff&filter=ADC
* /css/css-inline/baseline-source/baseline-source-first-textarea-003.tentative.html:
https://wpt.fyi/results/css/css-inline/baseline-source/baseline-source-first-textarea-003.tentative.html?label=master&product=safari%5Bstable%5D&product=safari%5Bexperimental%5D&aligned&diff&filter=ADC
* /css/css-inline/baseline-source/baseline-source-last-textarea-002.tentative.html:
https://wpt.fyi/results/css/css-inline/baseline-source/baseline-source-last-textarea-002.tentative.html?label=master&product=safari%5Bstable%5D&product=safari%5Bexperimental%5D&aligned&diff&filter=ADC
* /css/css-inline/baseline-source/baseline-source-last-textarea-003.tentative.html:
https://wpt.fyi/results/css/css-inline/baseline-source/baseline-source-last-textarea-003.tentative.html?label=master&product=safari%5Bstable%5D&product=safari%5Bexperimental%5D&aligned&diff&filter=ADC
* /css/css-writing-modes/forms/checkbox-appearance-native-vertical-lr-baseline.optional.html:
https://wpt.fyi/results/css/css-writing-modes/forms/checkbox-appearance-native-vertical-lr-baseline.optional.html?label=master&product=safari%5Bstable%5D&product=safari%5Bexperimental%5D&aligned&diff&filter=ADC
* /css/css-writing-modes/forms/checkbox-appearance-native-vertical-rl-baseline.optional.html:
https://wpt.fyi/results/css/css-writing-modes/forms/checkbox-appearance-native-vertical-rl-baseline.optional.html?label=master&product=safari%5Bstable%5D&product=safari%5Bexperimental%5D&aligned&diff&filter=ADC
* /css/css-writing-modes/forms/radio-appearance-native-vertical-lr-baseline.optional.html:
https://wpt.fyi/results/css/css-writing-modes/forms/radio-appearance-native-vertical-lr-baseline.optional.html?label=master&product=safari%5Bstable%5D&product=safari%5Bexperimental%5D&aligned&diff&filter=ADC
* /css/css-writing-modes/forms/radio-appearance-native-vertical-rl-baseline.optional.html:
https://wpt.fyi/results/css/css-writing-modes/forms/radio-appearance-native-vertical-rl-baseline.optional.html?label=master&product=safari%5Bstable%5D&product=safari%5Bexperimental%5D&aligned&diff&filter=ADC
All of these regressed with the enablement of vertical form controls. Needless to say, we probably don't want to be failing tests everyone else passes, thus this probably needs test changes to allow for vertical form controls (or does the spec require support?).
Attachments
Add attachment
proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1
2024-03-01 05:03:48 PST
<
rdar://problem/123895894
>
Aditya Keerthi
Comment 2
2024-03-01 11:18:44 PST
Starting to triage these. * /css/css-writing-modes/forms/checkbox-appearance-native-vertical-lr-baseline.optional.html:
https://wpt.fyi/results/css/css-writing-modes/forms/checkbox-appearance-native-vertical-lr-baseline.optional.html
* /css/css-writing-modes/forms/checkbox-appearance-native-vertical-rl-baseline.optional.html:
https://wpt.fyi/results/css/css-writing-modes/forms/checkbox-appearance-native-vertical-rl-baseline.optional.html
* /css/css-writing-modes/forms/radio-appearance-native-vertical-lr-baseline.optional.html:
https://wpt.fyi/results/css/css-writing-modes/forms/radio-appearance-native-vertical-lr-baseline.optional.html
* /css/css-writing-modes/forms/radio-appearance-native-vertical-rl-baseline.optional.html Are not regressions. In fact, I first wrote these tests. Chrome then added to the tests in
https://github.com/web-platform-tests/wpt/commit/715c01dd8ab90c9e402eddf7792f26f243ee843d
Really, the Chrome additions should have been done as a separate ref test. They show that our baseline alignment for checkboxes and radio buttons is incorrect for vertical writing mode + text-orientation: sideways. But that is simply a bug in a new feature. The test was passing pre-
270647@main
simply because it was written in a way such that the test passes when form controls are forced to use a horizontal writing mode.
Aditya Keerthi
Comment 3
2024-03-01 11:21:37 PST
* /css/css-align/baseline-rules/grid-item-input-type-text.html Is a 1px positioning difference in vertical writing mode. The test previously passed as horizontal writing mode does not have the same bug, and form controls were previously forced into horizontal.
Aditya Keerthi
Comment 4
2024-03-01 11:41:10 PST
The same is true of the remaining tests. They are all written in a way such that they pass when a horizontal writing mode is forced. But that itself is incorrect. We need to track related tests separately, as bugs in vertical writing mode.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug