Skip to content

Develop#528

Open
bagirovoleg wants to merge 33 commits intomate-academy:masterfrom
bagirovoleg:develop
Open

Develop#528
bagirovoleg wants to merge 33 commits intomate-academy:masterfrom
bagirovoleg:develop

Conversation

@bagirovoleg
Copy link

Copy link

@danon321 danon321 left a comment

Choose a reason for hiding this comment

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

image I can see burger menu and normal menu on laptops image image

Try to create container with max-width: 1194px (as on figma) to stop spreading ur context on whole desktop view (as on the screen)

image

Didnt see this section

image

On tablet hero image cut the text

@bagirovoleg bagirovoleg requested a review from danon321 October 14, 2025 10:56
@bagirovoleg
Copy link
Author

Poprawiłem

Copy link

@Zibi95 Zibi95 left a comment

Choose a reason for hiding this comment

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

Hey.
Firstly go trough this checklist https://github.com/bagirovoleg/Kickstarter/blob/develop/checklist.md. You have to complete these points.

and now so styling issues

  • Hamburger menu have different bg

Image****

The hero image on mobile has poor quality near the tablet breakpoint, and there's also a strange artifact on it.

Image
  • many images are too small for screens
Image
  • some images have bg when they shouldnt
Image
  • I believe that on tablets there also should be hamburger menu button
Image
  • Are you sure that on desktop these changing subtitles in the first section are correct? I think it's actually a navigation menu meant to let users quickly jump to different sections.
Image

@bagirovoleg
Copy link
Author

image

The task isn't detailed enough, and you're probably right—you have more experience with this. However, I'd like to leave it as is, as it might be useful later as a cheat sheet for subsequent tasks.

@bagirovoleg bagirovoleg requested a review from Zibi95 October 28, 2025 09:19
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