<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.webkit.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.4.1"
          urlbase="https://bugs.webkit.org/"
          
          maintainer="admin@webkit.org"
>

    <bug>
          <bug_id>270341</bug_id>
          
          <creation_ts>2024-03-01 05:03:29 -0800</creation_ts>
          <short_desc>REGRESSION(270647@main): vertical form controls causing WPT failures</short_desc>
          <delta_ts>2024-12-17 20:33:10 -0800</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>Forms</component>
          <version>WebKit Nightly Build</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</op_sys>
          <bug_status>NEW</bug_status>
          <resolution></resolution>
          
          <see_also>https://bugs.webkit.org/show_bug.cgi?id=245355</see_also>
    
    <see_also>https://bugs.webkit.org/show_bug.cgi?id=264692</see_also>
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>InRadar, WPTImpact</keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Sam Sneddon [:gsnedders]">gsnedders</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>akeerthi</cc>
    
    <cc>cdumez</cc>
    
    <cc>karlcow</cc>
    
    <cc>ntim</cc>
    
    <cc>webkit-bug-importer</cc>
    
    <cc>wenson_hsieh</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>2017964</commentid>
    <comment_count>0</comment_count>
    <who name="Sam Sneddon [:gsnedders]">gsnedders</who>
    <bug_when>2024-03-01 05:03:29 -0800</bug_when>
    <thetext>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&amp;product=safari%5Bstable%5D&amp;product=safari%5Bexperimental%5D&amp;aligned&amp;diff&amp;filter=ADC
* /css/css-flexbox/flex-item-compressible-002.html: https://wpt.fyi/results/css/css-flexbox/flex-item-compressible-002.html?label=master&amp;product=safari%5Bstable%5D&amp;product=safari%5Bexperimental%5D&amp;aligned&amp;diff&amp;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&amp;product=safari%5Bstable%5D&amp;product=safari%5Bexperimental%5D&amp;aligned&amp;diff&amp;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&amp;product=safari%5Bstable%5D&amp;product=safari%5Bexperimental%5D&amp;aligned&amp;diff&amp;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&amp;product=safari%5Bstable%5D&amp;product=safari%5Bexperimental%5D&amp;aligned&amp;diff&amp;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&amp;product=safari%5Bstable%5D&amp;product=safari%5Bexperimental%5D&amp;aligned&amp;diff&amp;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&amp;product=safari%5Bstable%5D&amp;product=safari%5Bexperimental%5D&amp;aligned&amp;diff&amp;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&amp;product=safari%5Bstable%5D&amp;product=safari%5Bexperimental%5D&amp;aligned&amp;diff&amp;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&amp;product=safari%5Bstable%5D&amp;product=safari%5Bexperimental%5D&amp;aligned&amp;diff&amp;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&amp;product=safari%5Bstable%5D&amp;product=safari%5Bexperimental%5D&amp;aligned&amp;diff&amp;filter=ADC

All of these regressed with the enablement of vertical form controls. Needless to say, we probably don&apos;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?).</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2017965</commentid>
    <comment_count>1</comment_count>
    <who name="Radar WebKit Bug Importer">webkit-bug-importer</who>
    <bug_when>2024-03-01 05:03:48 -0800</bug_when>
    <thetext>&lt;rdar://problem/123895894&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2018062</commentid>
    <comment_count>2</comment_count>
    <who name="Aditya Keerthi">akeerthi</who>
    <bug_when>2024-03-01 11:18:44 -0800</bug_when>
    <thetext>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.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2018063</commentid>
    <comment_count>3</comment_count>
    <who name="Aditya Keerthi">akeerthi</who>
    <bug_when>2024-03-01 11:21:37 -0800</bug_when>
    <thetext>* /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.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2018065</commentid>
    <comment_count>4</comment_count>
    <who name="Aditya Keerthi">akeerthi</who>
    <bug_when>2024-03-01 11:41:10 -0800</bug_when>
    <thetext>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.</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>