Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Top of main is hidden behind header when tall logo is used #8

Open
gareththered opened this issue May 14, 2020 · 3 comments
Open

Top of main is hidden behind header when tall logo is used #8

gareththered opened this issue May 14, 2020 · 3 comments
Labels
fixed

Comments

@gareththered
Copy link

The default logo is approx 160x35 and the page layout works. However, if a taller logo is used (160x70) the first line of text (approx h1's worth) becomes hidden behind the now taller header. This seems to be the case on MS Edge and on Firefox. I've not tested on others.

I believe a taller header should push the main body down - not overlap it.
@onweru
Copy link
Owner

onweru commented May 14, 2020

@gareththered,

I will fix this.

@onweru onweru added bug Something isn't working nice 👍 labels May 14, 2020
onweru pushed a commit that referenced this issue May 19, 2020
keep content clear of nav #8
1184e46
@onweru onweru added fixed and removed bug Something isn't working nice 👍 labels May 19, 2020
@onweru
Copy link
Owner

onweru commented May 19, 2020

@gareththered, this is fixed now.

@onweru onweru closed this as completed May 21, 2020
@freakinhippie
Copy link

This issue seems to either still exist, or was reintroduced at some point.

@onweru onweru reopened this Jun 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
fixed
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@gareththered @freakinhippie @onweru