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

Update landing page UI #40

Merged
merged 2 commits into from
Sep 16, 2018
Merged

Update landing page UI #40

merged 2 commits into from
Sep 16, 2018

Conversation

erickzhao
Copy link
Member

Closes #36 and #37.

  • Removes hero buildings SVG at viewport heights of less than 36rem.
  • Adds white background colour to mailchimp input.
  • Updates bridge and plateau SVGs

image

@erickzhao erickzhao changed the title Landing page UI fixes Update landing page UI Sep 16, 2018
@krubenok
Copy link
Member

The scaling doesn't work on a resize but it does on a refresh (at least in safari). I think this is a huge improvement over what we have now and is probably good enough. But I think we should re-open the issue on super low-pri to keep the issue in mind.

@krubenok krubenok merged commit 0da7132 into master Sep 16, 2018
@loreina
Copy link
Member

loreina commented Sep 16, 2018

I didn't get a chance to review yet... @krubenok

@erickzhao erickzhao deleted the landing-ui-fixes branch September 16, 2018 20:58
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