Skip to content

solution#539

Open
dqmurre wants to merge 4 commits intomate-academy:masterfrom
dqmurre:develop
Open

solution#539
dqmurre wants to merge 4 commits intomate-academy:masterfrom
dqmurre:develop

Conversation

@dqmurre
Copy link

@dqmurre dqmurre commented Jan 18, 2026

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.

good job

  1. these links should open in a new tab
image
  1. make sure the fields have the same outline colour
image
  1. place these blocks at the same height
image
  1. it would be good to add a hover effect to these icons
image
  1. make sure these links look the same
image

@dqmurre dqmurre requested a review from etojeDenys January 18, 2026 11:44
Copy link

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

Good Job!

To improve:

  1. Please remove the default button border to match the design.
image image
  1. Please double-check these heading styles. They should use #333333 by default.
image image image image image
  1. Please align these icons on one row
image
  1. The email field is currently marked as invalid for this value.
image
  1. On desktop, the image appears slightly larger than in the design—please adjust the size to match.
image image

@dqmurre
Copy link
Author

dqmurre commented Jan 18, 2026

не розумію чому при переході на мій сайт через ссилку яку я записав в коментарях, не змінилось нічого

Copy link

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

Choose a reason for hiding this comment

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

не розумію чому при переході на мій сайт через ссилку яку я записав в коментарях, не змінилось нічого

Did you deploy after the code changes? If you have some questions - feel free to ask in chat.

@dqmurre
Copy link
Author

dqmurre commented Jan 18, 2026

yes

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.

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

4 participants