Mobile header is taller than the content


(Jacob Chapel) #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:

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.


(Jeff Atwood) #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.


(Jeff Atwood) #3