Skip to content

solution - #190

Open
ahromovyi wants to merge 2 commits into
mate-academy:masterfrom
ahromovyi:develop
Open

ahromovyi wants to merge 2 commits into
mate-academy:masterfrom
ahromovyi:develop

Conversation

@ahromovyi

Copy link
Copy Markdown

@etojeDenys etojeDenys left a comment

Copy link
Copy Markdown

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 open the video in a new tab
Image
  1. make the inputs slightly larger or move the title into the placeholder
Image
  1. slightly increase the scale of the images on hover
Image

@ahromovyi

Copy link
Copy Markdown
Author

Thanks. All items have been fixed.

@vitalii-bondarenko-ua vitalii-bondarenko-ua left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Well done, good job!

@ahromovyi

Copy link
Copy Markdown
Author

Well done, good job!

Thanks.

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