Skip to content

Develop#2968

Open
MBondhub wants to merge 7 commits intomate-academy:mainfrom
MBondhub:develop
Open

Develop#2968
MBondhub wants to merge 7 commits intomate-academy:mainfrom
MBondhub:develop

Conversation

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

  1. on the desktop version, the phone number should appear when you hover that icon. also this icon should be tel link
Image

example:
Image

  1. disable page scrolling when the menu is open (see checklist step 11 for details)
Image
  1. it should be tel, mailto and address links
Image
  1. make sure the fields are the same size
Image
  1. don't forget to implement these steps
Image
  1. at this width, you have a white space that runs across the whole page (this is because of the inputs)
Image
  1. slightly increase the size of images when hovering them

@MBondhub MBondhub requested a review from etojeDenys February 13, 2026 18:22
Copy link

@FaiHamid FaiHamid 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!
Let`s just fix a few details:

  1. Add hover to images:
Image

2.These images shouldn’t have such a large border radius; in the design, they are rectangular on desktop:
Image
Image

3.Between 700 and 580 pixels, a horizontal scroll appears, although it shouldn’t be there

@MBondhub MBondhub requested a review from FaiHamid February 19, 2026 19:05
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. at this width, you have a problem with the page. to fix it, you should look at the sections with inputs
Image
  1. phone number should only appear on the desktop version
Image
  1. it would be good to make these images bigger when you hover them
Image
  1. all fields should be required
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.

  1. size of the images is increasing too much because it looks merged with the other one.slightly decrease the scale of the hovered image
Image
  1. slightly increase size of the button
Image
  1. there is too much space
Image Image

@MBondhub MBondhub requested a review from etojeDenys February 23, 2026 21:18
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 progress 🚀

i hope last changes:

  1. add some padding-top
Image
  1. change input type to email for this input
Image
  1. here should be link to mail. Take a look here
Image
  1. fix gaps in this section on mobile
Image

@MBondhub MBondhub requested a review from 2pasha February 24, 2026 16:59
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.

Almost done!

Just a couple tiny issues need to be fixed:

  1. When User selects one of predefined input values, no other design allowed for CSS Autofill - e.g. background-color or whatever else here:
Image
  1. Would you mind to add some hover effects on links inside the Contact Us section?

  2. Please make hover effect on buttons slightly smoother

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.

5 participants