Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Develop #2675

Open
wants to merge 46 commits into
base: master
Choose a base branch
from
Open

Develop #2675

wants to merge 46 commits into from

Conversation

Farrelzum
Copy link

@Farrelzum Farrelzum commented Dec 29, 2024

@Farrelzum Farrelzum requested a review from danon321 January 4, 2025 16:21
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

Co do telefonu i hovera myslalem ze pozmienialismy na QnA tak aby telefon nie był absolutem bo przez to on jakos dziwnie chodził. Czemu koniec koncow wrociłes do tego rozwiazania?

image ten fragment z prawej strony na figmie jest na srodku

@Farrelzum
Copy link
Author

Farrelzum commented Jan 5, 2025 via email

@Farrelzum Farrelzum requested a review from danon321 January 21, 2025 10:09
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.

"Tutaj w nawigacji hover dla ikonki telefonu nie działa. Nie wiem dlaczego."
Nie no działa tylko masz szary tekst na ciemnym tle i go nie widac. Jak sie lepiej przyjrzysz to go zobaczysz

"Jeszcze jedno zastrzeżenie mam do headera i faktu że ikony się nie rozjeżdzają przy zmianie szerokosci. Czy tak ma być?"
Twoje ikonki zwężają się do srodka bo przy szerokosci 744px zmieniasz grida
image

Jak to poprawisz to wszystko juz bedzie cacy

@Farrelzum Farrelzum requested a review from danon321 January 23, 2025 17:14
Copy link

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

In 'Contact Us' section all form inputs must be required to prevent submission when they are empty
checklist:

  • Change text color on hover for phone, email and address
  • Placeholders in the forms suggest what to enter; apply validation of the form fields (required, email / tel etc.), then it is clear in what format to enter the data
  • Form shouldn't be submitted if some of the fields are not filled

Copy link

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

  • Change text color on hover for phone, email and address
    image

Choose a reason for hiding this comment

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

you should use whole picture with better quality

[figma](image)

@natalia-klonowska
Copy link

all your layout is fixed with just breakpoints for 3 screen sizes. elements remain in place while margins expand as the page resizes. they should adjust dynamically, as the current behavior results in disproportionately large margins:
image

here you have differences between fixed and responsive layout

so your layout should be responsive with breakpoints where you change things like font-size, margins and grid
example

Copy link

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

  • brakuje odstępów między tekstami:
    image
  • obrazki powinny być sobie równe w wersji mobile
    image
  • button powinien rozciągać się aż do marginesów
    image
  • inputy i button w sekcji contact us też powinny być bardziej rozciągnięte
    image
    image

Copy link

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

  • wyrównaj button w formularzu (wersja na telefon)
    image
  • dla wersji tabletowej wymiary zdjęć zależą od wysokości okna przez co czasami są zbyt przybliżone lub poucinane. marginesy boczne nadal są 20px a nie 72px, nazwy kategorii mają zbyt mały font oraz odległości między elementami są zaburzone bo zdjęcia mają własne marginesy
    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