Summary: | REGRESSION (r205223): Web Inspector: Debugger popover title and code aren't horizontally aligned | ||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Nikita Vasilyev <nvasilyev> | ||||||||||||
Component: | Web Inspector | Assignee: | Nikita Vasilyev <nvasilyev> | ||||||||||||
Status: | RESOLVED FIXED | ||||||||||||||
Severity: | Normal | CC: | bburg, commit-queue, hi, joepeck, mattbaker, nvasilyev, timothy, webkit-bug-importer | ||||||||||||
Priority: | P2 | Keywords: | InRadar | ||||||||||||
Version: | WebKit Nightly Build | ||||||||||||||
Hardware: | All | ||||||||||||||
OS: | All | ||||||||||||||
Attachments: |
|
Regressed in bug 161417: Web Inspector: Remove largest common indentation spacing in debugger popover Created attachment 288531 [details]
Patch
Created attachment 288532 [details]
[Image] With patch applied
Comment on attachment 288531 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=288531&action=review > Source/WebInspectorUI/ChangeLog:11 > + Overwrite left padding defined in CodeMirrorOverrides.css by `.CodeMirror pre` rule. Should be "Override". (In reply to comment #4) > Created attachment 288532 [details] > [Image] With patch applied Personally, I find it odd that the function name has a left padding while the source link doesn't have any right padding. Should we consider removing the padding for the function name altogether? (In reply to comment #6) > (In reply to comment #4) > > Created attachment 288532 [details] > > [Image] With patch applied > > Personally, I find it odd that the function name has a left padding while > the source link doesn't have any right padding. Should we consider removing > the padding for the function name altogether? I didn't notice that! They should be the same. Created attachment 288618 [details]
Patch
Created attachment 288619 [details]
[Image] With patch applied
Comment on attachment 288618 [details] Patch Clearing flags on attachment: 288618 Committed r205832: <http://trac.webkit.org/changeset/205832> All reviewed patches have been landed. Closing bug. |
Created attachment 288530 [details] [Image] Bug The code is shifted a few pixels on the right. Must be a recent regression.