<html>
<head>
<base href="https://bugs.webkit.org/" />
</head>
<body>
<p>
<div>
<b><a class="bz_bug_link
bz_status_NEW "
title="NEW - Web Inspector: Console SearchBar should work more like ContentBrowser FindBanner"
href="https://bugs.webkit.org/show_bug.cgi?id=149505#c6">Comment # 6</a>
on <a class="bz_bug_link
bz_status_NEW "
title="NEW - Web Inspector: Console SearchBar should work more like ContentBrowser FindBanner"
href="https://bugs.webkit.org/show_bug.cgi?id=149505">bug 149505</a>
from <span class="vcard"><a class="email" href="mailto:dcrousso+webkit@gmail.com" title="Devin Rousso <dcrousso+webkit@gmail.com>"> <span class="fn">Devin Rousso</span></a>
</span></b>
<pre>Comment on <span class=""><a href="attachment.cgi?id=261931&action=diff" name="attach_261931" title="patch">attachment 261931</a> <a href="attachment.cgi?id=261931&action=edit" title="patch">[details]</a></span>
patch
View in context: <a href="https://bugs.webkit.org/attachment.cgi?id=261931&action=review">https://bugs.webkit.org/attachment.cgi?id=261931&action=review</a>
<span class="quote">> Source/WebInspectorUI/UserInterface/Main.html:85
> + <link rel="stylesheet" href="Views/ConsoleFindBanner.css"></span >
We organize CSS files by alphabetical order. Please move this to its appropriate place.
<span class="quote">> Source/WebInspectorUI/UserInterface/Views/LogContentView.js:918
> + this._findBanner.numberOfResults = numberOfResults;</span >
You have a findBanner parameter for this function. Is there a reason that you don't use it? If so, why not remove the parameter?</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>