BOSE#2737
Conversation
Zibi95
left a comment
Zibi95
left a comment
There was a problem hiding this comment.
DEMO LINK is on some old version.
Have you removed .gitignore file? node_modules should not be saved in remote repo
Zibi95
left a comment
Zibi95
left a comment
There was a problem hiding this comment.
No sure where to start.
Firstly remove node_modules from the remote repo and add .gitignore to your project so you won;'t push node_modules again.
I can't see the navbar, Image is out of bounds. It looks like its not styled yet. So I'm not sure why did you request review?
If you have some problems with deployment, feel free to ask anything on the Mate platform for help.
natalia-klonowska
left a comment
natalia-klonowska
left a comment
There was a problem hiding this comment.
z tego co widzę to commity widać
ale jeśli jakichś brakuje to sprawdź czy użyłaś komendy push oraz czy commity na pewno były robione na odpowiedni branch
deploy nie działa poprawie ponieważ jest robiony z brancha develop zamiast gh-pages:
tak to powinno wyglądać w ustawieniach:
danon321
left a comment
danon321
left a comment
There was a problem hiding this comment.
Desktop
Tutaj jak widzisz zdjecie jest w bardzo słabej jakosci oraz po lewej i prawej stronie mam białe przerwy na desktopie.
Czcionka również sie nie zgadza. Menu i ikonka telefonu w prawym gorym rogu jest niewidoczna
Tutaj tez produkty sa jeden pod drugim zamiast obok siebie.
Mobile
Na telefonie moge przeskrolowac w bok (co na tym dizajnie nie powinno byc mozliwe). Widze wtedy te same białe paski co na desktopie.
Czcionka tutaj rowniez sie nie zgadza
natalia-klonowska
left a comment
natalia-klonowska
left a comment
There was a problem hiding this comment.
nadal występują te same problemy co opisane wyżej
-
pojawiają ci się białe paski po bokach ponieważ twój layout nie jest responsywny. grid ma ustawione stałe rozmiary kolumn więc szerokość jest stała zamiast się dopasowywać.
tutaj możesz zobaczyć porównanie różnego typu layoutów
https://medium.com/design-bootcamp/fixed-width-responsive-and-adaptive-layout-what-to-chose-a29c4f93537b -
nadal masz scroll poziomy. jest on spowodowany elementami article co możesz zauważyć ponieważ wychodzą one poza obszar section
- nie masz ustawionego grida dla elementu nadrzędnego dla sekcji więc grid-column nie działa
danon321
left a comment
danon321
left a comment
There was a problem hiding this comment.
Widze ze juz kilka razy odbiłas sie od mentorów. Jak cos zapraszamy na QnA w celu wyjasnienia co i jak :))
Tutaj jeszcze moje sugestie poprawek.
Desktop
Tutaj obrazek powinien byc na cała szerokosc ekranu
Ponizej przykład z figmy
Tutaj przycisk ci ucieka
Grafika w footerze rowniez powinna byc na cała szerokosc ekranu.
4 participants
https://agacia1515w-ops.github.io/layout_miami/)