Skip to content

Footer - #444

Open
lauracastrovenegas wants to merge 19 commits into
masterfrom
Footer
Open

Footer#444
lauracastrovenegas wants to merge 19 commits into
masterfrom
Footer

Conversation

@lauracastrovenegas

Copy link
Copy Markdown
Contributor

Finished footer for website

CarolineHoang
CarolineHoang previously approved these changes May 22, 2020

@CarolineHoang CarolineHoang left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Footer functions perfectly on test
Discovered that the whole page is not centered flush in the mobile view, but this is not an issue with the footer.

Comment thread src/client/src/css/Footer.css Outdated
Comment thread src/client/src/css/Footer.css Outdated
Comment thread package.json
percents not needed in this situation
@CarolineHoang
CarolineHoang dismissed their stale review May 24, 2020 15:29

I've realized that there are sizing issues in this code on mobile. I am fixing them now.

@CarolineHoang CarolineHoang left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I have fixed the mobile compatibility issues. This should be good to go.

@CarolineHoang CarolineHoang left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I have fixed the mobile compatibility issues. This should be good to go.

@erin2722

erin2722 commented Jan 9, 2022

Copy link
Copy Markdown
Contributor

Looks good, only thing is deciding how/when to update credits

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.

3 participants