Skip to content

Develop#519

Open
OleksChernikov wants to merge 3 commits intomate-academy:masterfrom
OleksChernikov:develop
Open

Develop#519
OleksChernikov wants to merge 3 commits intomate-academy:masterfrom
OleksChernikov:develop

Conversation

@OleksChernikov
Copy link

Copy link

@maxim2310 maxim2310 left a comment

Choose a reason for hiding this comment

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

Plz fix demo link
image

@OleksChernikov
Copy link
Author

I have fixed that.

Copy link

@etojeDenys etojeDenys left a comment

Choose a reason for hiding this comment

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

  1. make sure this section looks the same with layout
    your:
image

figma:
image

  1. change text color. make sure all inputs are required. send button should clear the inputs instead of leading to an error page
image
  1. add a hover effect to these links
image

Copy link

@etojeDenys etojeDenys left a comment

Choose a reason for hiding this comment

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

good job
however, don't forget to change the text color in the second input field
image

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

Comments