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 #2682

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

Conversation

katarzynamilewska
Copy link

@katarzynamilewska
Copy link
Author

Potrzebuję pomocy w wykończeniu strony. Byłam dziś na Q&A o 16 ale nikt nie przyszedł więc wysyłam tutaj, żeby poprosić o pomoc :)

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.

Niestety demo link nie działa i dostaje bład 404 :(
Jak cos to pisz na czacie do mentorów a jezeli problem jest bardziej skomplikowany to zapraszamy na QnA

@katarzynamilewska
Copy link
Author

a czym to moze byc spowodowane? wydawaloby sie, ze zrobilam wszystko tak jak zawsze by utworzyc Pull Request?

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.

Hej. DEMO LINK nadal nie działa. Czy próbowałaś uruchomić komendę npm run deploy?

@katarzynamilewska
Copy link
Author

DEMO LINK

chyba dalej nie działa mimo, że wykonałam kroki 13-18 ponownie. Jak jeszcze mogę to sprobowac naprawic?

@katarzynamilewska
Copy link
Author

teraz link powinien dzialac

https://katarzynamilewska.github.io/layout_miami/

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.

layout is not responsive, it's fixed with breakpoints which results in too large margins between them:
image

Here you can find a comparison between fixed and responsive layouts and here you can see example of responsive layout with breakpoints

@natalia-klonowska
Copy link

Potrzebuję pomocy w wykończeniu strony. Byłam dziś na Q&A o 16 ale nikt nie przyszedł więc wysyłam tutaj, żeby poprosić o pomoc :)

Przepraszam za tą sytuację :( Zgłosiłam już ten problem dotyczący nie odbycia się Q&A. Daj znać z czym dokładnie potrzebujesz pomocy :)

@katarzynamilewska
Copy link
Author

No właśnie chciałabym żeby ktoś powiedział co mam źle i jak to naprawić bo walczę z tą stroną i nie udaje mi się jej wykończyć. Normalnie to zaczęłabym od nowa ale szkoda mi pracy jaką w to włożyłam :P więc tu potrzebuję pomocy a przy drugiej stronie już będę mądrzejsza :P

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.

  1. Tutaj przykład jak header wyglada na figmie przy rozdzielczosci 1260:
    image

a tu przykład jak to wyglada u ciebie:
image

  1. Gdzieś ci uciekła jedna kreseczka w burger menu
  2. Sprawdz prosze dokładnie wszystkie czcionki i odstepy miedzy elementami. Porównaj je z figma i zobacz czy maja odniesienie w twoim projekcie:
    image

image

@danon321
Copy link

danon321 commented Mar 4, 2025

Jezeli potrzebujesz z kims usiasc na chwile tak zeby ci powiedział co gdzie moze byc zle zapraszamy na QnA :) Ich godziny mozesz znalezc w swoim kalendarzu na platformie mate

@katarzynamilewska
Copy link
Author

@danon321 coś się chyba nie zgadza bo jak otwieram swoją stronę to wygląda to tak:
Zrzut ekranu 2025-03-04 115114

@katarzynamilewska
Copy link
Author

w sensie, ze sa trzy kreseczki w burgerze i obrazek jest lepiej dopasowany

@katarzynamilewska
Copy link
Author

Czy ktoś widzi moje komentarze tutaj czy muszę wysłać ponownie pull request za kazdym razem jak chcę by to było zobaczone?

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.

niestety nie dostajemy powiadomień o komentarzach więc jeśli chcesz abyśmy na nie odpowiedzieli musisz poprosić o review albo zadać pytanie przez chat na platformie mate.

CHECKLIST:

  • Change text color on hover for phone, email and address
  • When you click on phone icon or phone number in contacts section, make sure that there is no 404 error, make it a real link to start a call on device
  • When clicking on any location / address - prevent errors and make it to open location in Google Maps
  • Pictures in Gallery should increase on hover
  • Form shouldn't be submitted if some of the fields are not filled
  • Page shouldn't be reloaded on form submit (https://developer.mozilla.org/en-US/docs/Web/API/Event/preventDefault)
  • disable page scrolling under the menu:
    image

font-size: 16px;
line-height: 22px;
letter-spacing: 0;
width: 411px;

Choose a reason for hiding this comment

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

przez ustawienie tej szerokości pojawia ci się scroll poziomy jeśli szerokość okna jest mniejsza niż 411px

line-height: 27px;
border-radius: 40px;
border: 3px solid;
width: 324px;

Choose a reason for hiding this comment

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

ustawienie na sztywno szerokości powoduje pojawienie się scrolla poziomego. zamiast togo postaraj się wykorzystać jednostki procentowe aby element był responsywny

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.

checklist:

Comment on lines 50 to 52
&__item:hover {
width: 110%;
}

Choose a reason for hiding this comment

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

instead of increasing width use scale with smooth transition:
https://css-tricks.com/almanac/properties/s/scale/#aa-transitions-and-animations

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.

On mobile footer image is broken

image

And after that finito 🎉🎉

@katarzynamilewska katarzynamilewska requested a review from Zibi95 March 8, 2025 19:03
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.

4 participants