-
Notifications
You must be signed in to change notification settings - Fork 31
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
Branding Updates #2593
Branding Updates #2593
Conversation
Instead of a link, the About button in the header now opens up a modal showing details and links.
BiG-CZ presentation should not change, so we flag all changes to only apply for MMW.
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.
Looks good to me visually. Stroud is not itself a funder, so it doesn't belong in that section. I'm hoping we're OK without the wikiwatershed logo.
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.
Looks good. Tested that the change is only visible on MMW and not BIGCZ.
Thanks for reviewing! Merging now. |
LGTM. There's a link to WikiWatershed in the about modal. I think that's a good balance for now. |
Overview
Updates Model My Watershed splash page branding as follows:
Tagging @caseycesari for code review, and @jfrankl and @ajrobbins for visual review.
Connects #2534
Demo
Home / Splash page:
About Modal:
BiG-CZ Splash page:
Notes
Testing Instructions
bundle