Skip to content

MET landing#3015

Open
kseniiavasylenko wants to merge 5 commits intomate-academy:mainfrom
kseniiavasylenko:develop
Open

MET landing#3015
kseniiavasylenko wants to merge 5 commits intomate-academy:mainfrom
kseniiavasylenko:develop

Conversation

@kseniiavasylenko
Copy link

Copy link

@2pasha 2pasha 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! 👏

here are some improvements:

  1. add favicon
Image
  1. add some transition for menu opening

  2. add hover effects for your images

Image
  1. fix this button styles
Image
  1. here should be links to phone and maps
Image
  1. fix form styles
Image
  1. make sure that you have done these points
Image
  1. this section on mobile looks broken
Image

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!

A few things to improve:

  1. I would recommend to make inputs' sizes to fit the buttons size and also it's better to remove autofill styles
Image
  1. Please add favicon for this page
Image
  1. This title should be in one row with content
Image Image
  1. It's better to add cursor: pointer for buttons
Image Image
  1. I would recommend to add smooth scaling on these images
Image
  1. For better quality of raster images I would recommend to export them in 2x scale from Figma and then convert to .avif or webp formats
Image Image

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