Skip to content

add task solution#167

Open
AndrewPrszn wants to merge 6 commits intomate-academy:masterfrom
AndrewPrszn:develop
Open

add task solution#167
AndrewPrszn wants to merge 6 commits intomate-academy:masterfrom
AndrewPrszn:develop

Conversation

@AndrewPrszn
Copy link

@AndrewPrszn AndrewPrszn commented Oct 14, 2025

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.

Good job!

To improve:

  1. Clicking on this button should erase all date in form. Please review #15 from this checklist
image
  1. Hover effect (scale) should overlay the background. Use z-index to deal with it:
image
  1. Would you mind to slightly reduce your hover effect here?
    Uploading image.png…

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. after providing input and pressing the "contact us" button, horizontal scrolling appears
image
  1. fix the style for the auto-filled inputs
image
  1. add favicon

Copy link

@h-amster h-amster left a comment

Choose a reason for hiding this comment

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

  1. Make sure that when you hover over the cards, they don't overlap each other when scaled. Also add a hover effect to the button.
image
  1. The hero section (the first screen of your landing page) should occupy 100vh. The “more” button should be clickable and navigate the user to the next section.
image
  1. The outline that appears when focused should not overlap the dropdown arrow.
image
  1. The “close” button is very close to the right edge. According to the design, it should have an indent. The menu should cover the entire screen according to the design. That is, when the menu is open, nothing should be visible under it. Also, when hovering over a link, the entire line should change its background color. The indent to the left of the link needs to be removed.
image
  1. Work with transition. Currently, when removing the cursor from an element, there is no smoothness, the state changes instantly.

@AndrewPrszn AndrewPrszn requested a review from h-amster October 24, 2025 18:53
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.

almost done

  1. disable scrolling when the menu is open. The menu also has horizontal scrolling
image

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