Mobile header is taller than the content

Bug
1

Essentially the mobile header is taller than the actual content and looks off.

It should look like this:

CSS property that needs to be adjusted:

https://github.com/discourse/discourse/blob/911f941da543f7b44ddf457099aa97cb31947076/app/assets/stylesheets/common/base/header.scss#L20

This is the change that actually introduced the “bug”

https://github.com/discourse/discourse/commit/911f941da543f7b44ddf457099aa97cb31947076

The height should be adjusted, or better yet we should find a better way to style it that doesn’t involve hard coded pixel based heights.

2

Thanks for the nudge, not sure why I set that height.

When removed, it seems fine on text-only-logo sites like twittercommunity.com and more typical image-logo sites like here … so that’s what I did.

1 Like