-
Notifications
You must be signed in to change notification settings - Fork 56
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
Hybrid sticky/relative positioned footer #2390
base: main
Are you sure you want to change the base?
Conversation
✅ Deploy Preview for boosted ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These changes feel right to me. One small comment and let's wait for another review
@@ -22,6 +22,8 @@ You can choose to display each part or not, except the last one which is mandato | |||
|
|||
If these available parts don't meet your specific needs, feel free to develop your own custom part accordingly to the Orange Design System. | |||
|
|||
The footer is positioned at the bottom of the window when the page content is smaller than the window, and at the bottom of the page when it is bigger. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Might not be true for some DOM reasons. But for simple syntax where <footer>
is a direct child of <body>
it seems to work pretty well. Any opinion on this @julien-deramond ?
Note: Please transform
- [ ]
into- (NA)
in the description when things are not applicableRelated issues
Description
Add hybrid mode between sticky and relative positioning for footer into library and not only in docs since it is asked by many projects in Orange
Motivation & Context
Avoid empty pages like this:
Types of change
Live previews
Checklist
Contribution
Accessibility
Design
Development
Documentation
Checklist (for Core Team only)
After the merge