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

Items in the footer are aligned better than before #620

Open
wants to merge 2 commits into
base: dev
Choose a base branch
from

Conversation

jf-kelly
Copy link
Contributor

@jf-kelly jf-kelly commented Aug 4, 2022

Description

This PR fixes the issue with the items in the footer not being aligned properly. I simply removed the 'list-inline' property in the Footer.js file, and the items are more spread out.

This fix does not look exactly like it did before, but it does spread the items out a bit and looks better than what we currently have in dev.

Preview

2022-08-04 (3)

Related Issue

#611

@jf-kelly jf-kelly requested review from a team, rjmara, jdones01 and davidjonesma and removed request for a team August 4, 2022 20:40
@jf-kelly jf-kelly added Frontend Issues being caused via the frontend Discussion needed The issue requires discussion about design before development dev labels Aug 4, 2022
@rjmara
Copy link
Contributor

rjmara commented Aug 9, 2022

@jf-kelly The left definitely does look better than what's in dev currently. Is there a way to make the copyright and privacy policy + terms of service buttons more similar to production? In production, it looks like the privacy policy and terms of service buttons are aligned left while the copyright is aligned at the center. In dev and in your PR, it seems like the privacy policy and terms of service buttons are still a little to the left.

@jf-kelly
Copy link
Contributor Author

I added another commit that makes everything aligned a little better once again. The Bootstrap update changed certain things, like 'left' and 'right' are now 'start' and 'end', so it would possibly be worth looking into these changes to see if more has broken than we expected.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dev Discussion needed The issue requires discussion about design before development Frontend Issues being caused via the frontend
Projects
Status: Needs Review
Development

Successfully merging this pull request may close these issues.

2 participants