<?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>44515</bug_id>
          
          <creation_ts>2010-08-24 05:01:56 -0700</creation_ts>
          <short_desc>Implement ruby CSS display values</short_desc>
          <delta_ts>2023-10-08 11:17:58 -0700</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>CSS</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>All</rep_platform>
          <op_sys>All</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>DUPLICATE</resolution>
          <dup_id>261468</dup_id>
          
          <bug_file_loc>http://beta.html5test.com/testcases/ruby/index.html</bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>BrowserCompat</keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          <dependson>261468</dependson>
          <blocked>40829</blocked>
          <everconfirmed>1</everconfirmed>
          <reporter name="Niels Leenheer (HTML5test)">info</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>ahmad.saleem792</cc>
    
    <cc>ap</cc>
    
    <cc>eoconnor</cc>
    
    <cc>eric</cc>
    
    <cc>haraken</cc>
    
    <cc>hyatt</cc>
    
    <cc>karlcow</cc>
    
    <cc>mmaxfield</cc>
    
    <cc>ntim</cc>
    
    <cc>paulirish</cc>
    
    <cc>rniwa</cc>
    
    <cc>syoichi</cc>
    
    <cc>zalan</cc>
    
    <cc>zcorpan</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>268484</commentid>
    <comment_count>0</comment_count>
    <who name="Niels Leenheer (HTML5test)">info</who>
    <bug_when>2010-08-24 05:01:56 -0700</bug_when>
    <thetext>When HTML5 ruby elements are rendered their display property values are inline or block. The HTML5 specification *1 suggest the use of &apos;ruby&apos; for &lt;ruby&gt; elements, &apos;ruby-text&apos; for &lt;rt&gt; elements and &apos;none&apos; for &lt;rp&gt; elements. Additionally &apos;ruby-base&apos; is suggested for children of &lt;ruby&gt; elements that are not &lt;rt&gt; or &lt;rp&gt; elements.

These display property values correspond to the values defined in the CSS3 Ruby Module *2. Even though full support for this CSS module is not present at the moment, the HTML5 ruby support should use the suggested display property values.

This change is worth 3 points on the HTML5test *3 which currently does not award any points to Webkit for ruby support.

---
*1 http://www.w3.org/TR/html5/rendering.html#display-types
*2 http://www.w3.org/TR/css3-ruby/
*3 http://html5test.com/</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>307131</commentid>
    <comment_count>1</comment_count>
    <who name="Alexey Proskuryakov">ap</who>
    <bug_when>2010-11-09 19:59:40 -0800</bug_when>
    <thetext>See also: bug 47596.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>312029</commentid>
    <comment_count>2</comment_count>
    <who name="Dave Hyatt">hyatt</who>
    <bug_when>2010-11-20 19:55:55 -0800</bug_when>
    <thetext>The problem here is that ruby can work as both an inline or block construct (or even as an inline-block construct).  A single display type, &quot;ruby&quot;, is insufficient to model this.  The CSS3 Ruby draft has huge flaws, and if you&apos;re basing your HTML5 test suite off it, then you need to fix your test suite.

I think the HTML5 spec should be amended to drop all mention of unique Ruby display types until problems like this can be addressed.

The ruby-base and ruby-text display types are fine, but the Ruby display type needs to be broken into two display types.  We&apos;d be willing to break the inline-block version of Ruby since it would rarely be desired, but I don&apos;t think we want positioning or floating a ruby to suddenly turn off its ruby behavior. Therefore block-level ruby has to be specifiable.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>312030</commentid>
    <comment_count>3</comment_count>
    <who name="Dave Hyatt">hyatt</who>
    <bug_when>2010-11-20 20:19:50 -0800</bug_when>
    <thetext>Also, one of your ruby tests is buggy and is testing for a display value of hidden rather than none for the rp element.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>312034</commentid>
    <comment_count>4</comment_count>
    <who name="Dave Hyatt">hyatt</who>
    <bug_when>2010-11-20 20:33:00 -0800</bug_when>
    <thetext>Note that even if we did add these display types, we would have to add them as -webkit-ruby-text, etc., so we wouldn&apos;t pass your test anyway.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>539979</commentid>
    <comment_count>5</comment_count>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2012-01-20 18:13:07 -0800</bug_when>
    <thetext>http://www.w3.org/TR/css3-ruby/#display also seems relevant.

I agree with Dave, the test should be fixed to not depend on these display types until the specs settle down.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>540008</commentid>
    <comment_count>6</comment_count>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2012-01-20 18:55:57 -0800</bug_when>
    <thetext>https://github.com/NielsLeenheer/html5test/issues/51 was the original bug filed against the test suite.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1745378</commentid>
    <comment_count>7</comment_count>
    <who name="Simon Pieters (:zcorpan)">zcorpan</who>
    <bug_when>2021-03-30 15:51:53 -0700</bug_when>
    <thetext>See https://github.com/whatwg/html/issues/2134 for &apos;display&apos; of &lt;rp&gt; (outside ruby). Chromium and Gecko and the spec have

rp { display: none }

but WebKit uses display: inline here

http://software.hixie.ch/utilities/js/live-dom-viewer/saved/9130</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1875883</commentid>
    <comment_count>8</comment_count>
    <who name="Ahmad Saleem">ahmad.saleem792</who>
    <bug_when>2022-06-14 15:11:35 -0700</bug_when>
    <thetext>This seems to be clarified in Spec discussion and now Blink and Gecko aligns with each other based on Comment 07 (refer to Spec discussion link).

Is this need to be updated within Webkit?

https://github.com/WebKit/WebKit/blob/8afe31a018b11741abdf9b4d5bb973d7c1d9ff05/Source/WebCore/html/RubyElement.cpp#L56

Thanks!</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1888365</commentid>
    <comment_count>9</comment_count>
    <who name="Tim Nguyen (:ntim)">ntim</who>
    <bug_when>2022-08-02 05:03:23 -0700</bug_when>
    <thetext>(In reply to Dave Hyatt from comment #2)
&gt; The problem here is that ruby can work as both an inline or block construct
&gt; (or even as an inline-block construct).  A single display type, &quot;ruby&quot;, is
&gt; insufficient to model this.  The CSS3 Ruby draft has huge flaws, and if
&gt; you&apos;re basing your HTML5 test suite off it, then you need to fix your test
&gt; suite.
&gt; 
&gt; I think the HTML5 spec should be amended to drop all mention of unique Ruby
&gt; display types until problems like this can be addressed.
&gt; 
&gt; The ruby-base and ruby-text display types are fine, but the Ruby display
&gt; type needs to be broken into two display types.  We&apos;d be willing to break
&gt; the inline-block version of Ruby since it would rarely be desired, but I
&gt; don&apos;t think we want positioning or floating a ruby to suddenly turn off its
&gt; ruby behavior. Therefore block-level ruby has to be specifiable.

CSS display module level 3 addresses this by separating display into outer display and inner display.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1903527</commentid>
    <comment_count>10</comment_count>
    <who name="Ahmad Saleem">ahmad.saleem792</who>
    <bug_when>2022-10-05 14:21:31 -0700</bug_when>
    <thetext>Fixed in Safari Technology Preview 155 due to updating UA Stylesheet to make &quot;rp&quot; as display:none aligned with Standard.

All browsers show &quot;in body&quot; in this test now - http://software.hixie.ch/utilities/js/live-dom-viewer/?saved=9130 

Safari 16 show &quot;rp in body&quot;.

Marking this as &quot;RESOLVED CONFIGURATION CHANGED&quot;.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1903749</commentid>
    <comment_count>11</comment_count>
    <who name="Tim Nguyen (:ntim)">ntim</who>
    <bug_when>2022-10-06 07:52:32 -0700</bug_when>
    <thetext>I do think we should implement display: ruby / inline ruby / block ruby like Firefox does, and use them in our UA stylesheet, instead of hacking it in `createElementRenderer`.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1906029</commentid>
    <comment_count>12</comment_count>
    <who name="Karl Dubost">karlcow</who>
    <bug_when>2022-10-16 20:40:11 -0700</bug_when>
    <thetext>Some status 
https://wpt.fyi/results/css?label=master&amp;label=experimental&amp;aligned&amp;view=subtest&amp;q=status%3Afail%20ruby


                chr       edge      fx         saf tp
                108       108.      107        155
css-break/      4 / 4     4 / 4     0 / 4      2 / 4
css-content/    0 / 1     0 / 1     0 / 1      0 / 1
css-ruby/       22 / 100  27 / 100  89 / 100   13 / 100
css-text-decor/	0 / 6     0 / 6     6 / 6      0 / 6
css-ui/         0 / 1     0 / 1     1 / 1      0 / 1
Subtest Total	26 / 112  31 / 112  96 / 112   15 / 112</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1983586</commentid>
    <comment_count>13</comment_count>
    <who name="Tim Nguyen (:ntim)">ntim</who>
    <bug_when>2023-10-08 11:17:34 -0700</bug_when>
    <thetext>Bug 261468 implements some of this.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1983587</commentid>
    <comment_count>14</comment_count>
    <who name="Tim Nguyen (:ntim)">ntim</who>
    <bug_when>2023-10-08 11:17:58 -0700</bug_when>
    <thetext>

*** This bug has been marked as a duplicate of bug 261468 ***</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>