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

Redesign the landing page to look more inline with brand design. #137

Merged
merged 1 commit into from
Apr 13, 2017

Conversation

brockwhittaker
Copy link
Contributor

@brockwhittaker brockwhittaker commented Apr 13, 2017

This redesigns the landing page to incorporate the Zulip style guide.
This also adds two animations:

  1. The error text slides down on error and back up when typing.
  2. The form section shakes when there is invalid input for 0.5s.

New Landing Page

@Lplenka
Copy link
Collaborator

Lplenka commented Apr 13, 2017

@brockwhittaker the page looks great! 👍

you should try running npm run test to check travis errors.

Check this Contributing.md for not failing Travis checks next time. 😄

@brockwhittaker brockwhittaker force-pushed the landing-page-redesign branch from 3879d17 to bb88917 Compare April 13, 2017 18:57
@brockwhittaker
Copy link
Contributor Author

I just fixed all mine but there's still ~25 errors that aren't mine that are present.

This redesigns the landing page to incorporate the Zulip style guide.
This also adds two animations:

1. The error text slides down on error and back up when typing.
2. The form section shakes when there is invalid input for 0.5s.
@brockwhittaker brockwhittaker force-pushed the landing-page-redesign branch from bb88917 to c1f6159 Compare April 13, 2017 19:03
@Lplenka
Copy link
Collaborator

Lplenka commented Apr 13, 2017

@brockwhittaker happens with me too.
npm run test shows JS Lint errors not directly linked with our contribution.
But all of them can be easily fixed.

It's always better if our Pull Request has no JS Lint errors, It takes time to trace those small errors and fix them, but efforts are worth it. you won't ever fail in Travis builds 👍

@akashnimare akashnimare merged commit df91c20 into zulip:master Apr 13, 2017
@akashnimare
Copy link
Member

@brockwhittaker we could make it better :P but for now, this looks good. Merged. Thanks for working on this 🙏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants