Skip to content

Develop#731

Open
itNepes wants to merge 5 commits into
mate-academy:masterfrom
itNepes:develop
Open

Develop#731
itNepes wants to merge 5 commits into
mate-academy:masterfrom
itNepes:develop

Conversation

@itNepes

@itNepes itNepes commented Jul 14, 2026

Copy link
Copy Markdown

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

Good progress, almost done, but let`s check this:

  1. Page after submit should not jump to the top.

  2. Remove focus default blue outline from input:

Image
  1. All links & buttons should be clickable and have hover effect:
Image

@itNepes
itNepes requested a review from brespect July 14, 2026 14:59
Vite needs type=module so the subscribe and gallery swipe script is included in the production build.

Co-authored-by: Cursor <cursoragent@cursor.com>

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

good job, to improve:

  1. disable page scrolling when the menu is open
Image
  1. when the menu opens, the page content should be covered, as shown here
Image
  1. it should be tel links
Image
  1. after clicking on these links, the menu should close
Image

@itNepes
itNepes requested a review from etojeDenys July 14, 2026 21:31

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

Almost done, but nothing happens when I click on phone icons/tel. They should have hover effects and initiate a call.

@itNepes
itNepes requested a review from brespect July 15, 2026 06:40
@itNepes

itNepes commented Jul 15, 2026

Copy link
Copy Markdown
Author

Почти готово, но ничего не происходит при нажатии на значки телефона/телефона. Должны быть эффекты при наведении курсора и возможность инициировать звонок.

я забув деплойнути! Зараз все ок має бути.

@brespect brespect 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!

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