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

#3 v2 header & footer updates #1389

Conversation

edwinjue
Copy link
Member

@edwinjue edwinjue commented Oct 4, 2022

Contributes to #1347

Screenshots

Header (figma)

image

Header (#1389)

image

Footer (figma)

image

Footer (#1389)

image

  • Up to date with dev branch
  • Branch name follows guidelines
  • All PR Status checks are successful
  • Peer reviewed and approved

Any questions? See the getting started guide

@edwinjue edwinjue requested review from nichhk and jekijo October 4, 2022 22:47
@edwinjue edwinjue mentioned this pull request Oct 4, 2022
11 tasks
@edwinjue
Copy link
Member Author

edwinjue commented Oct 4, 2022

@jekijo The third one for you Killen. Please let me know if you have any questions. Thanks again!

@edwinjue edwinjue requested a review from funbunch October 5, 2022 02:18
@edwinjue
Copy link
Member Author

edwinjue commented Oct 5, 2022

@funbunch added you as a reviewer since you have been recently working on these files. Please let me know if you have any questions. Thanks in advance!

@funbunch
Copy link
Member

funbunch commented Oct 5, 2022

Hi @edwinjue Is there a visual change here where we can see a screenshot?

@edwinjue
Copy link
Member Author

edwinjue commented Oct 5, 2022

@funbunch good idea! i just updated the first post with screenshots of what the header and footer will look like after merge along with screenshots taken from the figma v2 design. hope it helps

@funbunch
Copy link
Member

funbunch commented Oct 5, 2022

Thanks! Will approve.

@edwinjue edwinjue merged commit 0f4288b into dev Oct 6, 2022
@edwinjue edwinjue deleted the 1347-thematic-coloring-of-content-pages-3-v2-header-footer-updates branch October 6, 2022 01:29
@edwinjue
Copy link
Member Author

edwinjue commented Oct 6, 2022

@funbunch Thanks, appreciate the quick response!

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.

2 participants