Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Develop Landing Page 'THE MET' #2013

Open
wants to merge 12 commits into
base: main
Choose a base branch
from

Conversation

Viktor-1995
Copy link

Copy link

@anastasiiavorobiova anastasiiavorobiova left a comment

Choose a reason for hiding this comment

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

Great work!

  • Screenshot 2025-02-03 at 12 05 30
  • Consider removing default outlines.
  • The form shouldn't jump on the focus
  • Screenshot 2025-02-03 at 12 05 20
  • Consider using links here
  • Screenshot 2025-02-03 at 12 04 54
  • Consider showing the phone numbers on the hover
  • Consider adding hover styles for all interactive elements on the page
  • Also, it's better to show the cursor pointer for button hover
  • A user should be able to scroll the menu in the landscape mode on a mobile phone

Copy link

@volodymyr-soltys97 volodymyr-soltys97 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. Change title to The MET
image
  1. There is no way to hover over the phone number, the block immediately disappears
image
  1. Add transition when this menu appears on the page
image
  1. Add hover effects for images
image
  1. Remove _ in phone numbers
image
  1. These elements should be links
image
  1. Add hover effects here
image
  1. Fix size for the logo and icons on the mobile
image
  1. Also, the mobile version looks strange, need to fix it

Copy link

@anastasiiavorobiova anastasiiavorobiova left a comment

Choose a reason for hiding this comment

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

Great work!

  • Screenshot 2025-02-07 at 13 05 30
  • Consider using links to Google Maps here, they should be opened in the new tab
  • Screenshot 2025-02-07 at 13 05 01
  • The link leads to the 404 page

Copy link

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