Summary: | Web Inspector: Add icon for collapsed blackboxed section | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Nikita Vasilyev <nvasilyev> | ||||||||
Component: | Web Inspector | Assignee: | Devin Rousso <hi> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | ews-watchlist, hi, inspector-bugzilla-changes, joepeck, pangle, webkit-bug-importer | ||||||||
Priority: | P2 | Keywords: | InRadar | ||||||||
Version: | WebKit Nightly Build | ||||||||||
Hardware: | All | ||||||||||
OS: | All | ||||||||||
Bug Depends on: | 216897 | ||||||||||
Bug Blocks: | 234581 | ||||||||||
Attachments: |
|
Description
Nikita Vasilyev
2020-12-11 15:05:07 PST
No icons needs to be added if you use the existing `[?]` icon, which is my current plan. I’ll reopen if we decide to use another icon. I don't think [?] is a good icon to convey "blackboxed". I think we should either use the eye or some other icon so that it's visually clear that the blackboxed "group" is not a normal call frame. Created attachment 448319 [details]
Patch
Created attachment 448321 [details]
[Image] after Patch is applied (dark)
Created attachment 448322 [details]
[Image] after Patch is applied (light)
Comment on attachment 448319 [details]
Patch
Neat! Looks nice
Comment on attachment 448319 [details]
Patch
r=me
Committed r287586 (245715@main): <https://commits.webkit.org/245715@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 448319 [details]. |