Sure, that&#39;s fine.<div><br></div><div>But please make sure not to ship these CSS properties until editing code is properly fixed.<br><div><br></div><div><div class="gmail_quote">On Wed, Aug 1, 2012 at 4:47 AM, Bruno Abinader <span dir="ltr">&lt;<a href="mailto:brunoabinader@gmail.com" target="_blank">brunoabinader@gmail.com</a>&gt;</span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Ryosuke,<br>
<br>
Indeed. On the proposed implementation I&#39;ve assured that the original<br>
&quot;text-decoration&quot; from CSS 2.1 spec remains with exactly same behavior<br>
as before (though I&#39;ve managed to get some shared code to avoid<br>
duplication). Layout tests results from my local run points no new<br>
failing tests due to these changes so far.<br>
<br>
As for the editing &amp; execCommand, these will be handled later (you can<br>
see none of the patches touches any files inside<br>
Source/WebCore/editing directory). I know there are some management<br>
with CSSPropertyTextDecoration and<br>
CSSPropertyWebkitTextDecorationsInEffect going on there, but these<br>
stay untouched (in edting) until I acquire more  information about<br>
-webkit-text-decorations-in-effect usage.<br>
<div class="HOEnZb"><div class="h5"><br>
On Tue, Jul 31, 2012 at 10:40 PM, Ryosuke Niwa &lt;<a href="mailto:rniwa@webkit.org">rniwa@webkit.org</a>&gt; wrote:<br>
&gt; Please make sure the new text decoration values don&#39;t break editing (rich<br>
&gt; text editing &amp; execCommand). In particular, we probably need to update<br>
&gt; ApplyStyleCommand/EditingStyle to handle -webkit-text-decoration*.<br>
&gt;<br>
&gt; On Tue, Jul 31, 2012 at 7:35 PM, Bruno Abinader &lt;<a href="mailto:brunoabinader@gmail.com">brunoabinader@gmail.com</a>&gt;<br>
&gt; wrote:<br>
&gt;&gt;<br>
&gt;&gt; Hi all :)<br>
&gt;&gt;<br>
&gt;&gt; As suggested by Ojan, I am writing a mail to you about my intention to<br>
&gt;&gt; implement all updated and missing text-decoration* properties from<br>
&gt;&gt; CSS3 spec (currently in development), named below:<br>
&gt;&gt;<br>
&gt;&gt; -webkit-text-decoration ( <a href="https://bugs.webkit.org/show_bug.cgi?id=92000" target="_blank">https://bugs.webkit.org/show_bug.cgi?id=92000</a> )<br>
&gt;&gt; CSS3 dev spec: <a href="http://dev.w3.org/csswg/css3-text/#text-decoration" target="_blank">http://dev.w3.org/csswg/css3-text/#text-decoration</a><br>
&gt;&gt; Mozilla ref: <a href="https://developer.mozilla.org/en/CSS/text-decoration" target="_blank">https://developer.mozilla.org/en/CSS/text-decoration</a><br>
&gt;&gt; Status: Proposed patch / pending review<br>
&gt;&gt;<br>
&gt;&gt; -webkit-text-decoration-line (<br>
&gt;&gt; <a href="https://bugs.webkit.org/show_bug.cgi?id=90959" target="_blank">https://bugs.webkit.org/show_bug.cgi?id=90959</a> )<br>
&gt;&gt; CSS3 dev spec: <a href="http://dev.w3.org/csswg/css3-text/#text-decoration-line" target="_blank">http://dev.w3.org/csswg/css3-text/#text-decoration-line</a><br>
&gt;&gt; Mozilla ref: <a href="https://developer.mozilla.org/en/CSS/text-decoration-line" target="_blank">https://developer.mozilla.org/en/CSS/text-decoration-line</a><br>
&gt;&gt; Status: Proposed patch / pending review<br>
&gt;&gt;<br>
&gt;&gt; -webkit-text-decoration-style (<br>
&gt;&gt; <a href="https://bugs.webkit.org/show_bug.cgi?id=90958" target="_blank">https://bugs.webkit.org/show_bug.cgi?id=90958</a> )<br>
&gt;&gt; CSS3 dev spec: <a href="http://dev.w3.org/csswg/css3-text/#text-decoration-style" target="_blank">http://dev.w3.org/csswg/css3-text/#text-decoration-style</a><br>
&gt;&gt; Mozilla ref: <a href="https://developer.mozilla.org/en/CSS/text-decoration-style" target="_blank">https://developer.mozilla.org/en/CSS/text-decoration-style</a><br>
&gt;&gt; Status: Proposed patch / pending review<br>
&gt;&gt;<br>
&gt;&gt; -webkit-text-decoration-color (<br>
&gt;&gt; <a href="https://bugs.webkit.org/show_bug.cgi?id=91638" target="_blank">https://bugs.webkit.org/show_bug.cgi?id=91638</a> )<br>
&gt;&gt; CSS3 dev spec: <a href="http://dev.w3.org/csswg/css3-text/#text-decoration-color" target="_blank">http://dev.w3.org/csswg/css3-text/#text-decoration-color</a><br>
&gt;&gt; Mozilla ref: <a href="https://developer.mozilla.org/en/CSS/text-decoration-color" target="_blank">https://developer.mozilla.org/en/CSS/text-decoration-color</a><br>
&gt;&gt; Status: Proposed patch / pending review<br>
&gt;&gt;<br>
&gt;&gt; -webkit-text-decoration-skip (<br>
&gt;&gt; <a href="https://bugs.webkit.org/show_bug.cgi?id=92801" target="_blank">https://bugs.webkit.org/show_bug.cgi?id=92801</a> )<br>
&gt;&gt; CSS3 dev spec: <a href="http://dev.w3.org/csswg/css3-text/#text-decoration-skip" target="_blank">http://dev.w3.org/csswg/css3-text/#text-decoration-skip</a><br>
&gt;&gt; Status: Work in progress<br>
&gt;&gt;<br>
&gt;&gt; As each patch touches similar places and sometimes requires previous<br>
&gt;&gt; work, this patch series requires the patches to be reviewed / commited<br>
&gt;&gt; in the same order as described above ( 92000, 90959, 90958, 91638 and<br>
&gt;&gt; 92801 ).<br>
&gt;&gt; The decision to use &quot;-webkit-&quot; prefix was suggested by Alexis on<br>
&gt;&gt; <a href="https://bugs.webkit.org/show_bug.cgi?id=90959#c13" target="_blank">https://bugs.webkit.org/show_bug.cgi?id=90959#c13</a> . There is also a<br>
&gt;&gt; suggestion to use a specific flag to enable this feature at compile<br>
&gt;&gt; time, however other CSS3 properties such as text-emphasis, for<br>
&gt;&gt; example, do not use such flags, so I am a bit uncertain about it<br>
&gt;&gt; still.<br>
&gt;&gt;<br>
&gt;&gt; I am really looking forward for comments and review on the proposed<br>
&gt;&gt; patches :)<br>
&gt;&gt;<br>
&gt;&gt; Best regards,<br>
&gt;&gt;<br>
&gt;&gt; --<br>
&gt;&gt; Bruno de Oliveira Abinader<br>
&gt;&gt; _______________________________________________<br>
&gt;&gt; webkit-dev mailing list<br>
&gt;&gt; <a href="mailto:webkit-dev@lists.webkit.org">webkit-dev@lists.webkit.org</a><br>
&gt;&gt; <a href="http://lists.webkit.org/mailman/listinfo/webkit-dev" target="_blank">http://lists.webkit.org/mailman/listinfo/webkit-dev</a><br>
&gt;<br>
&gt;<br>
<br>
<br>
<br>
--<br>
Bruno de Oliveira Abinader<br>
</div></div></blockquote></div><br></div></div>