Skip to content

Develop#552

Open
u5135039754-dev wants to merge 12 commits intomate-academy:masterfrom
u5135039754-dev:develop
Open

Develop#552
u5135039754-dev wants to merge 12 commits intomate-academy:masterfrom
u5135039754-dev:develop

Conversation

@u5135039754-dev
Copy link
Copy Markdown

@u5135039754-dev u5135039754-dev commented Mar 28, 2026

Copy link
Copy Markdown

@Denys-Kravchuk9988 Denys-Kravchuk9988 left a comment

Choose a reason for hiding this comment

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

It seems that the project hasn't been deployed yet. DEMO LINK doesn't work (Mentors can't approve without demo link)

Checked locally, a few things to improve:

  1. Please, remove autofill styles
Image
  1. It's better to disable scrolling when menu is open
Image Image

Copy link
Copy Markdown

@brespect brespect 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, check next:

  1. Add hover effect to all interactive elements, including pictures.
  2. To top button in Footer should smoothly scrolls the page.

Copy link
Copy Markdown

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

good job! 👏

here are some improvements:

  1. these links should be anchor links and navigate to specific section
Image
  1. navigation doesn't work
Image

@u5135039754-dev u5135039754-dev requested a review from 2pasha March 31, 2026 19:32
Copy link
Copy Markdown

@brespect brespect left a comment

Choose a reason for hiding this comment

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

Well done!

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.

4 participants