Customizer on mobile: fixed grey space at the bottom

[iseulde]

Don't remember seeing this in 4.2.

[obenland]

Something is setting the wrong height there

[obenland]

Introduced in [33069].

[obenland]

@ celloexpressions, could you review if that patch is in line with what was intended in [33069]?

[ocean90]

It's content.css( 'height', ( window.innerHeight - 90 ) ); which needs to be removed/changed for screen sizes below 640px IMO.

[celloexpressions]

90 is derived from 45px for the header and 45px for the footer, which is only shown on larger screens. New patch needs to check for screen size < 640px and use 45 instead of 90 in those cases, and should also add inline comments explaining where the magic numbers come from. I've been trying to avoid setting the height in JS like this since panels were first introduced a year ago, but it's just necessary and going to make things easier moving forward to do it this way.

[obenland]

@celloexpressions would you mind creating that patch?

[celloexpressions]

I likely won't have time until after RC, but I could do it then (assuming it's still tracking Wednesday). Should be a fairly straightforward patch.

[westonruter]

In 33454:

Customizer: Hide empty space in footer on small screens since it is only populated on larger screens.

Props obenland.
Fixes #33138.