Summary: | Web Inspector: console.table(): Silently hiding some columns + Key's 'name' can trigger | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | ar <webkitbugs> | ||||
Component: | Web Inspector | Assignee: | Nobody <webkit-unassigned> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | bburg, inspector-bugzilla-changes, joepeck, webkit-bug-importer | ||||
Priority: | P2 | Keywords: | InRadar | ||||
Version: | Safari 10 | ||||||
Hardware: | All | ||||||
OS: | OS X 10.11 | ||||||
Attachments: |
|
Description
ar
2017-02-25 17:08:12 PST
Created attachment 302765 [details]
Test case
8x console.table() examples, with annotations
The test case looks as expected to me in the Safari Technology Preview. Do you still see any issues when using the Safari Technology Preview? Thanks for checking. Unfortunately I can't test in the latest Safari Technology Preview because I haven't upgraded to Sierra yet. The issue *is* there in STP "Release 16 (Safari 10.1, WebKit 11603.1.10)" (which is the last version that runs in 10.11.6) and the release version "10.0.3 (11602.4.8.0.1)". Do you *see* the issue in the release version above and/or the release version in Sierra 10.12.x? 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> 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/> > 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! > 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! Marking as resolved. I'll make sure to note this limitation in documentation of Console API. |