Skip to content

solution#535

Open
LinaDo1609 wants to merge 8 commits intomate-academy:masterfrom
LinaDo1609:develop
Open

solution#535
LinaDo1609 wants to merge 8 commits intomate-academy:masterfrom
LinaDo1609:develop

Conversation

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

great job

  1. send button leads to an error page. make sure that it only clears the inputs

  2. fix image here

image
  1. on the phone version, that section should be the full width of the screen
    your:
image

fegma:
image

@LinaDo1609 LinaDo1609 requested a review from etojeDenys December 5, 2025 15:26
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.

Amazing job!
Just one tiny fix to get approved!

When I'm opening your Landing on my Laptop, there is a horizontal scrollbar here.
image

Would you mind to fix it?

@LinaDo1609
Copy link
Author

Amazing job! Just one tiny fix to get approved!

When I'm opening your Landing on my Laptop, there is a horizontal scrollbar here. image

Would you mind to fix it?

could you, check please
because i don't have this scrollbar on my laptop to find out what is the problem of it appearance, but i try all my best

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.

3 participants

Comments