Skip to content

created kat vr page#171

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

created kat vr page#171
MykolaFatkullin wants to merge 3 commits intomate-academy:masterfrom
MykolaFatkullin:develop

Conversation

@MykolaFatkullin
Copy link

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.

great job!

to improve:

  1. it would be better to use that text as a placeholder
image
  1. disable text area resizing
image
  1. add a hover effect to those images
image

@MykolaFatkullin
Copy link
Author

Comments 2 and 3 are fixed.
Comment 1 left unchanged as in this case I'm not sure how to handle error messages, as I made them fro blur and button pressing

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.

almost done

just fix the text area because it is too small, and the user cannot see what text he is typing
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.

great job

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