<html>
    <head>
      <base href="https://bugs.webkit.org/">
    </head>
    <body>
      <p>
        <div>
            <b><a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - Implement parsing and animation support for offset-distance, offset-position, offset-anchor"
   href="https://bugs.webkit.org/show_bug.cgi?id=231491#c7">Comment # 7</a>
              on <a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - Implement parsing and animation support for offset-distance, offset-position, offset-anchor"
   href="https://bugs.webkit.org/show_bug.cgi?id=231491">bug 231491</a>
              from <span class="vcard"><a class="email" href="mailto:tho22@apple.com" title="Kiet Ho <tho22@apple.com>"> <span class="fn">Kiet Ho</span></a>
</span></b>
        <pre>Comment on <span class=""><a href="attachment.cgi?id=440736&action=diff" name="attach_440736" title="Patch">attachment 440736</a> <a href="attachment.cgi?id=440736&action=edit" title="Patch">[details]</a></span>
Patch

View in context: <a href="https://bugs.webkit.org/attachment.cgi?id=440736&action=review">https://bugs.webkit.org/attachment.cgi?id=440736&action=review</a>

<span class="quote">>> Source/WebCore/ChangeLog:8
>> +        No new tests; WPT parsing tests are sufficient.

> If bits of this patch are from external contributors, you should acknowledge them here.</span >

I'll mention WPT tests that cover the implemented properties.

<span class="quote">>> Source/WebCore/css/CSSComputedStyleDeclaration.cpp:3180
>> +            return valueForPositionOrAuto(style, style.offsetAnchor());

> You can also use your helper for CSSPropertyPerspectiveOrigin, maybe others.</span >

In the current code perspective-origin is parsed as a shorthand for some reason :/ If we ever change perspective-origin to use this helper, then it'd be for another patch.</pre>
        </div>
        <div>
            <b><a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - Implement parsing and animation support for offset-distance, offset-position, offset-anchor"
   href="https://bugs.webkit.org/show_bug.cgi?id=231491#c6">Comment # 6</a>
              on <a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - Implement parsing and animation support for offset-distance, offset-position, offset-anchor"
   href="https://bugs.webkit.org/show_bug.cgi?id=231491">bug 231491</a>
              from <span class="vcard"><a class="email" href="mailto:tho22@apple.com" title="Kiet Ho <tho22@apple.com>"> <span class="fn">Kiet Ho</span></a>
</span></b>
        <pre>Comment on <span class=""><a href="attachment.cgi?id=440736&action=diff" name="attach_440736" title="Patch">attachment 440736</a> <a href="attachment.cgi?id=440736&action=edit" title="Patch">[details]</a></span>
Patch

View in context: <a href="https://bugs.webkit.org/attachment.cgi?id=440736&action=review">https://bugs.webkit.org/attachment.cgi?id=440736&action=review</a>

<span class="quote">>> Source/WebCore/ChangeLog:8
>> +        No new tests; WPT parsing tests are sufficient.

> If bits of this patch are from external contributors, you should acknowledge them here.</span >

I'll mention WPT tests that cover the implemented properties.

<span class="quote">>> Source/WebCore/css/CSSComputedStyleDeclaration.cpp:3180
>> +            return valueForPositionOrAuto(style, style.offsetAnchor());

> You can also use your helper for CSSPropertyPerspectiveOrigin, maybe others.</span >

In the current code perspective-origin is parsed as a shorthand for some reason :/ If we ever change perspective-origin to use this helper, then it'd be for another patch.</pre>
        </div>
      </p>


      <hr>
      <span>You are receiving this mail because:</span>

      <ul>
          <li>You are the assignee for the bug.</li>
      </ul>
    </body>
</html>