[Webkit-unassigned] [Bug 168873] Web Inspector: console.table(): Silently hiding some columns + Key's 'name' can trigger

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=168873

--- Comment #7 from ar <webkitbugs at rmnet.com> ---
> I suspect this was fixed by:
> <https://webkit.org/b/167175> Web Inspector: console.table only works for the first 5 properties
> <http://trac.webkit.org/changeset/210910>

Yeah. I referenced in my original report. It looked promising.

> Which is only a few months old. So probably past STP 16.

> You can try a WebKit Nightly on El Capitan. Click the download links in the "WebKit Nightly" section on: <https://webkit.org/downloads/>

Ahh. I didn't realize that.

A few things:
- Tested in Nightly r214393 (March 25, 2017 8:37 AM GMT)

- I can confirm all of my original test cases PASS now.

- I can now see up to 15 columns, which makes sense given:
        - var maxColumnsToRender = 15;
        - via: https://bugs.webkit.org/attachment.cgi?id=299184&action=diff#a/Source/WebInspectorUI/UserInterface/Views/ConsoleMessageView.js_sec1
        - 16+ columns are silently suppressed (as before).

- Setting the "Column Display Restriction" argument to a column above 15 (front end cut-off) works correctly now too! (My tests: 2b, 3b).

This is great.
Thanks!

Now my only request would be: More documentation about this.

- I wasn't able to find anything about this limitation, except in <https://bugs.webkit.org/show_bug.cgi?id=167175> and the source. So adding a note about this to some documentation, somewhere, would be nice :-)

- Even better: A message that appears when column display has been suppressed. Even if it's just a warning or note about the `Max=15` limitation.

Thanks again!

--- Comment #7 from ar <webkitbugs at rmnet.com> ---
> I suspect this was fixed by:
> <https://webkit.org/b/167175> Web Inspector: console.table only works for the first 5 properties
> <http://trac.webkit.org/changeset/210910>

Yeah. I referenced in my original report. It looked promising.

> Which is only a few months old. So probably past STP 16.

> You can try a WebKit Nightly on El Capitan. Click the download links in the "WebKit Nightly" section on: <https://webkit.org/downloads/>

Ahh. I didn't realize that.

A few things:
- Tested in Nightly r214393 (March 25, 2017 8:37 AM GMT)

- I can confirm all of my original test cases PASS now.

- I can now see up to 15 columns, which makes sense given:
        - var maxColumnsToRender = 15;
        - via: https://bugs.webkit.org/attachment.cgi?id=299184&action=diff#a/Source/WebInspectorUI/UserInterface/Views/ConsoleMessageView.js_sec1
        - 16+ columns are silently suppressed (as before).

- Setting the "Column Display Restriction" argument to a column above 15 (front end cut-off) works correctly now too! (My tests: 2b, 3b).

This is great.
Thanks!

Now my only request would be: More documentation about this.

- I wasn't able to find anything about this limitation, except in <https://bugs.webkit.org/show_bug.cgi?id=167175> and the source. So adding a note about this to some documentation, somewhere, would be nice :-)

- Even better: A message that appears when column display has been suppressed. Even if it's just a warning or note about the `Max=15` limitation.

Thanks again!

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.webkit.org/pipermail/webkit-unassigned/attachments/20170325/49199b2b/attachment.html>