Problem view padding is inconsistent

[Tyriar]

#69204

The top and bottom padding is inconsistent.

Looks like it's related to "invisible" items inside the body:

Version: 1.32.0-insider (user setup)
Commit: 393b48d
Date: 2019-02-25T01:34:48.785Z
Electron: 3.1.3
Chrome: 66.0.3359.181
Node.js: 10.2.0
V8: 6.6.346.32
OS: Windows_NT x64 10.0.18841

[Tyriar]

Also part of this, the scrollbar is hidden:

[sandy081]

Fixed the scroll bar and focus issues. But it is hard to have consistent padding because it involves computing height using number of lines. Actually this looks same also before and the difference in padding is not so much to consider.

[Tyriar]

@sandy081 I beg to differ with the difference in padding, it makes it look sloppy to me. If we can only show the padding if the scroll bar is needed that would be best like in #69391

[sandy081]

@Tyriar This is how it looks with and without a link. I don't see any sloppiness

[Tyriar]

@sandy081 the bottom padding of the container should match the top padding of the container, just like in the second screenshot?

[roblourens]

I'm guessing this should be reopened?

I also spun another issue off this: #69495

[sandy081]

Since this is not a regression, moving this to March

[sandy081]

Sorry @Tyriar. Looked into it again to make the padding symmetric. But I am sorry to say this need bigger changes to peek widget which I am not willing to do for the value it gives. Moreover it is like this from beginning. Hence closing this as wont fix.