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.

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

Fix footer does not stick to the bottom #577

Closed
wants to merge 1 commit into from

Conversation

simoarpe
Copy link

Resolves #576

This PR does not affect the current behavior of the footer except for the edge case described in the issue: when there is a small content or none the footer now stays at the bottom the page.

Because the height of the footer can change, most of the solutions around didn't work. Using flexbox for a sticky footer not only doesn't require any extra elements, but allows for a variable height footer.

Signed-off-by: Simone Arpe <simon.arpe@gmail.com>
@daattali
Copy link
Owner

daattali commented Feb 3, 2020

Thanks for the PR!

This changes the layout of the entire page. Since this theme is used by tens of thousands of people, I'm extremely cautious in making such changes. I would like to see this tested with different config parameters, to make sure it doesn't break any existing features. How much have you tested this outside of your own project?

@simoarpe
Copy link
Author

simoarpe commented Feb 7, 2020

I ran some tests, but probably not as much as it deserved.
Tried on different browsers, desktop and mobile.

@O-Mutt
Copy link
Contributor

O-Mutt commented Apr 24, 2020

Might be cautious of flex-direction with FF https://caniuse.com/#search=flex-direction

@daattali
Copy link
Owner

daattali commented May 2, 2020

Thank you @Mutmatt

Have you tried this solution to see what it looks like on FF? I just rewrote the majority of the HTML and CSS while upgrading to bootstrap 4, so now seems like a good time to test this as well.

@daattali
Copy link
Owner

@simoarpe the theme recently underwent huge changes, updating to bootstrap 4 which uses flexbox, so if you could update this PR to work with the current codebase, I'd be happy to accept this

Repository owner deleted a comment from Lion2me Aug 22, 2020
@daattali
Copy link
Owner

daattali commented Sep 4, 2020

Closing this PR due to inactivity. If anyone wants to revive it in the future, feel free to submit a new one!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Footer stick to the bottom when contents are short or absent
3 participants