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

#2209 Footer cutting off Export Users on User page #2239

Closed
wants to merge 23 commits into from

Conversation

courtneyc1
Copy link
Contributor

Updated footer page toot cut off export users under user page.

🗣 Description

go to frontend/src/components/footer/styleFooter.ts

under line 12-17 for corrections

💭 Motivation and context

footer was cutting off content of pages. User page at the bottom where export user was cut off by footer.

🧪 Testing

tested all changes within local crossed and verified before creating the PR.

Before with footer being cut off
264076812-6dd93d38-aa5a-41c3-95e1-81b53dbc3b88

Footer issue resolved
Screenshot 2023-09-13 at 10 58 42 AM

The footer now shows under the content for all pages. The user page under export now shows footer under it instead of cutting it off.
Copy link
Contributor

@Matthew-Grayson Matthew-Grayson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This solution would undo changes that keep the footer at the bottom for pages, such as the Feeds page. I recommend keeping the footer position as absolute and adding padding-bottom to each page body.

@ValeriaReveles
Copy link

I recommend seeing if 'position: fixed' works as better solution first as opposed to 'position: absolute' as it should allow the footer to act as a sticky footer without needed to change padding or margins.

@ValeriaReveles
Copy link

Failing "Docs" build check due to issues with node-module changes possibly made to master. Page able to load locally and no issues on docker logs.

@courtneyc1
Copy link
Contributor Author

moving this issue to #2345

@courtneyc1
Copy link
Contributor Author

closing this issue because of current updates should fix branch conflict and build/docs issues. See #2345.

@courtneyc1 courtneyc1 closed this Oct 24, 2023
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 cutting off "Export Users" section of "Users" page.
3 participants