Skip to content

The work is complete#702

Open
fedirbazalii wants to merge 7 commits into
mate-academy:masterfrom
fedirbazalii:develop
Open

The work is complete#702
fedirbazalii wants to merge 7 commits into
mate-academy:masterfrom
fedirbazalii:develop

Conversation

@fedirbazalii

Copy link
Copy Markdown

@etojeDenys etojeDenys left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

good job

  1. it should be links
Image
  1. 'send' button should simply clear the input without resetting the page
Image
  1. slightly increase the size of the images when they are hovered

@etojeDenys etojeDenys left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

it seems that you forgot to deploy the changes

@fedirbazalii

Copy link
Copy Markdown
Author

it seems that you forgot to deploy the changes

Sorry
Now I have already deployed

@brespect brespect left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Good progress, let`s check this:

  1. The page looks different from the mockup on Desktop 1440px+, should be no margins:
Image
  1. The burger menu icon shouldn`t overflow through whole page:
Image
  1. The second photo onMobile slider should not be visible:
Image

@fedirbazalii

Copy link
Copy Markdown
Author

brespect

Fixed desktop margins and burger menu overflow. Regarding the mobile slider, I kept the next image visible to match the Figma design and indicate scrollability.

Знімок екрана 2026-02-13 192806

@etojeDenys etojeDenys left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

great 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