Skip to content

Develop#2733

Open
MateuszRuszczynski wants to merge 97 commits into
mate-academy:developfrom
MateuszRuszczynski:develop
Open

Develop#2733
MateuszRuszczynski wants to merge 97 commits into
mate-academy:developfrom
MateuszRuszczynski:develop

Conversation

@MateuszRuszczynski
Copy link
Copy Markdown

Cześć. W projekcie zdecydowałem się na niewielkie zmiany w designie w sekcji ze zdjęciami. Mam nadzieję, że takie podejście jest w porządku.

Copy link
Copy Markdown

@natalia-klonowska natalia-klonowska 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! Just a few small fixes needed:

  • title in header mobile version should have bigger font size
  • icons in all versions don’t match sizes from design
Image
  • on tablet and desktop versions fix differences in font size (title), side margins, icons size and gap between them
Image
  • tablet version should have different text layout
Image

Copy link
Copy Markdown

@natalia-klonowska natalia-klonowska left a comment

Choose a reason for hiding this comment

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

make sure that burger icon and close icon are positioned in the exact same spot to allow quick and seamless opening and closing of menu. and change close icon size to match design
https://www.loom.com/share/e05aa983553d4349874cc3374095d26a

Copy link
Copy Markdown

@natalia-klonowska natalia-klonowska left a comment

Choose a reason for hiding this comment

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

ikony ponownie są zbyt duże. ich rozmiar wychodzi nawet poza kontener w którym się znajdują

Image

@MateuszRuszczynski MateuszRuszczynski force-pushed the develop branch 2 times, most recently from 38ba267 to 656f7f1 Compare March 4, 2026 21:08
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.