Summary: | Web Inspector: make items in the CanvasOverviewContentView more flexible | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Devin Rousso <hi> | ||||||||
Component: | Web Inspector | Assignee: | Devin Rousso <hi> | ||||||||
Status: | RESOLVED WONTFIX | ||||||||||
Severity: | Normal | CC: | bburg, inspector-bugzilla-changes, joepeck | ||||||||
Priority: | P2 | ||||||||||
Version: | WebKit Nightly Build | ||||||||||
Hardware: | All | ||||||||||
OS: | All | ||||||||||
Bug Depends on: | |||||||||||
Bug Blocks: | 175485 | ||||||||||
Attachments: |
|
Description
Devin Rousso
2017-10-07 00:16:07 PDT
Created attachment 323088 [details]
Patch
Created attachment 323089 [details]
[Image] After Patch is applied
Again there needs to be a before picture, ideally with the exact same content and window size, to really understand what is changing. Created attachment 323103 [details]
[Image] Before
I actually like the Before picture better. The extra padding around the sides makes the contents stand out more. Comment on attachment 323088 [details]
Patch
I prefer the second one, as the first one just looks kind of cramped with the tiny gray border.
Joe, can we land this and revert if it is too annoying or space wasting?
Oops, I misread which attachment is which. I agree with Joe now. :P I suppose we could still force a gutter on both sides and use flex box for the rest of the content area.. I think the consensus is to leave it as is. I've gotten used to it as well :P |