Skip to content

Solution#169

Open
lishvets wants to merge 5 commits intomate-academy:masterfrom
lishvets:develop
Open

Solution#169
lishvets wants to merge 5 commits intomate-academy:masterfrom
lishvets:develop

Conversation

@lishvets
Copy link

Copy link

@Anton-Kuchmasov Anton-Kuchmasov left a comment

Choose a reason for hiding this comment

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

DEMO link is not working,
Please deploy your solution and re-request a review again

@lishvets
Copy link
Author

DEMO 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.

  1. add smooth scrolling to the page

  2. add a hover effect to these links

image
  1. use a higher-quality image for the desktop version
image
  1. fix the input style. it should be a placeholder
image
  1. add a hover effect to these links
image

@lishvets lishvets requested a review from etojeDenys November 17, 2025 19:05
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.

Good work, but lets make some improvement

  1. all hovers and animations have to be smooth
Image Image
  1. here is wrong font
Image
  1. remove scroll when menu is opened
Image
  1. here should be some indent i guess
Image
  1. and here styles should be according design
Image

@lishvets lishvets requested a review from maxim2310 November 19, 2025 18:18
Copy link

@2pasha 2pasha left a comment

Choose a reason for hiding this comment

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

you have done really good job, fix this comments and project will be perfect

  1. it is still actual issue with smooth scroll (here is some info)

  2. fix transition-duration for hover effect on this button. For nav links, it's good

image
  1. when i am clicking on FAQ - site brokes( (the same with Help link)
image
  1. also if you are interesting in this, this landing page contain specific form flow. If you can implement this - it will be great!
image

@lishvets lishvets requested a review from 2pasha December 4, 2025 15:47
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.

lgtm

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.

5 participants