<html>
<head>
<base href="https://bugs.webkit.org/" />
</head>
<body><span class="vcard"><a class="email" href="mailto:joepeck@webkit.org" title="Joseph Pecoraro <joepeck@webkit.org>"> <span class="fn">Joseph Pecoraro</span></a>
</span> changed
<a class="bz_bug_link
bz_status_NEW "
title="NEW - Web Inspector: Unnecessary space added after -webkit- prefixed property values"
href="https://bugs.webkit.org/show_bug.cgi?id=146671">bug 146671</a>
<br>
<table border="1" cellspacing="0" cellpadding="8">
<tr>
<th>What</th>
<th>Removed</th>
<th>Added</th>
</tr>
<tr>
<td style="text-align:right;">Attachment #256316 Flags</td>
<td>review?
</td>
<td>review+
</td>
</tr></table>
<p>
<div>
<b><a class="bz_bug_link
bz_status_NEW "
title="NEW - Web Inspector: Unnecessary space added after -webkit- prefixed property values"
href="https://bugs.webkit.org/show_bug.cgi?id=146671#c7">Comment # 7</a>
on <a class="bz_bug_link
bz_status_NEW "
title="NEW - Web Inspector: Unnecessary space added after -webkit- prefixed property values"
href="https://bugs.webkit.org/show_bug.cgi?id=146671">bug 146671</a>
from <span class="vcard"><a class="email" href="mailto:joepeck@webkit.org" title="Joseph Pecoraro <joepeck@webkit.org>"> <span class="fn">Joseph Pecoraro</span></a>
</span></b>
<pre>Comment on <span class=""><a href="attachment.cgi?id=256316&action=diff" name="attach_256316" title="Patch">attachment 256316</a> <a href="attachment.cgi?id=256316&action=edit" title="Patch">[details]</a></span>
Patch
View in context: <a href="https://bugs.webkit.org/attachment.cgi?id=256316&action=review">https://bugs.webkit.org/attachment.cgi?id=256316&action=review</a>
r=me, thanks for addressing all my comments!
<span class="quote">> Source/WebInspectorUI/Tools/PrettyPrinting/index.html:54
> + // Here, the "calc" keyword is added to allow the do-not-add-whitespace-before-prefixed-property-value</span >
Nit: Double space. Also don't call out a particular test by name, that is the kind of comment that can go stale in the future if that test gets removed or renamed. Just mention its to behave more like the frontend.</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>