[webkit-reviews] review granted: [Bug 166993] Web Inspector: checkboxes in Settings screen use inappropriate layout : [Attachment 306454] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Apr 11 16:17:58 PDT 2017


Devin Rousso <webkit at devinrousso.com> has granted Matt Baker
<mattbaker at apple.com>'s request for review:
Bug 166993: Web Inspector: checkboxes in Settings screen use inappropriate
layout
https://bugs.webkit.org/show_bug.cgi?id=166993

Attachment 306454: Patch

https://bugs.webkit.org/attachment.cgi?id=306454&action=review




--- Comment #24 from Devin Rousso <webkit at devinrousso.com> ---
Comment on attachment 306454
  --> https://bugs.webkit.org/attachment.cgi?id=306454
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=306454&action=review

r=me.  Super awesome job!

> Source/WebInspectorUI/UserInterface/Views/SettingEditor.js:130
> +	       let keyValuePairs = [];

NIT: should be `var` not `let`

https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Statements/le
t#Temporal_dead_zone_and_errors_with_let


More information about the webkit-reviews mailing list