Code review pierwszej responsywnej strony

Odpowiedz Nowy wątek
2018-07-08 22:50

Rejestracja: 2 lata temu

Ostatnio: 1 miesiąc temu

Lokalizacja: Poznań

0

https://github.com/Mateuszto/Pierwsza-strona-responsywna-2018
Jeżeli ktoś miałby czas na zrobienie code review mojej pierwszej responsywnej strony byłbym bardzo wdzięczny.

Pozostało 580 znaków

2018-07-08 23:07

Rejestracja: 3 lata temu

Ostatnio: 1 miesiąc temu

Lokalizacja: Wrocław

2

Nazewnictwo klas i ID do poprawy. Najlepiej klasy pisać po angielsku, zgodnie z ktorakolwiek konwencją nazewniczą, np. Block Element Modifier. Żeby to miało jakiś ład i płaską strukturę.
Unikalbym też nadmiernego stylowania po selektorach i ID.
Reszta wydaje się ok.

edytowany 2x, ostatnio: tubaris, 2018-07-08 23:12
Dzięki :) No, rzeczywiście z nazwami jest trochę ciężko. Nawet z ich wymyślaniem hah. Mam nadzieję, że się w tym poprawie, bo jakby projekt był większy to trudno się w tym póżniej połapać :D. Jeszcze raz dzięki. - Mateuszto 2018-07-08 23:21
Też miałem z tym problem na początku, myślałem kilka minut jaką wymyślić nazwę dla klasy. Ale z czasem zobaczysz, że będzie Ci to przychodzić coraz łatwiej. Ja czytałem różne tutoriale na ten temat. Podrzucam 2 arty które mi najlepiej pomogły. http://javascript-html5-tutor[...]ass-bem-rscss-smacss-wtf.html https://seesparkbox.com/foundry/bem_by_example - tubaris 2018-07-09 06:20

Pozostało 580 znaków

2018-07-09 09:41
Moderator Kariera

Rejestracja: 2 lata temu

Ostatnio: 6 godzin temu

Lokalizacja: Poznań

1

To RWD nie do końca działa tak, jak powinno - pomniejszając okno, trzecie kółko całkowicie znika, a dopiero po chwili następuje przełączenie na widok pionowy. Powinno się przełączyć już w chwili, gdy skrajne kółko mocno zbliży się do krawędzi okna. Obecne zachowanie jest niedopuszczalne.

Ewentualnie mógłbyś dodać jeszcze z jakiś 1 czy 2 breakpointy, które podczas skalowania okna będą zbliżać kółka do siebie, albo w ogóle je od siebie odsunąć nie na sztywno, ale w jakiś płynny/procentowy sposób. Ale poza tym nie widzę większych problemów, strona raczej OK, zwłaszcza, jak na pierwszą - po kilku kolejnych nabędziesz biegłości :)

.

screenshot-20180709093544.png


Naczelny forumowy hejter Apple

That game of life is hard to play, I'm gonna lose it anyway
The losing card I'll someday lay, So this is all I have to say
edytowany 1x, ostatnio: cerrato, 2018-07-09 09:47
niekoniecznie, przecież użytkownik nie zmniejsza i powiększa okna dla zabawy tylko ogląda w danej rozdzielczości, więc jeśli te bazowe są ok, to nie widze większego problemu - czysteskarpety 2018-07-09 10:58
Na komórce - OK. Ale na desktopie może mieć dowolny rozmiar, także "nietypowy". I nawet nie bawiąc się w zmienianie - może mieć przeglądarkę odpaloną w rozmiarze z mojego screena lub podobnym, w związku z czym straci część treści. Pomijając kwestię estetyki, może w ten sposób coś przeoczyć i nawet nie mieć świadomości tego. - cerrato 2018-07-09 11:45

Pozostało 580 znaków

2018-07-09 11:03

Rejestracja: 5 lat temu

Ostatnio: 6 godzin temu

Lokalizacja: Piwnica

0

prócz powyższego, mała podpowiedź, odtwórz źródło strony w firefox i tam też często pokazuje co do poprawki
ponadto możesz też sprawdzić walidatorem: https://validator.w3.org/nu/?[...]za-strona-responsywna-2018%2F

screenshot-20180709110238.png


Pozostało 580 znaków

2018-07-09 11:30

Rejestracja: 3 lata temu

Ostatnio: 1 tydzień temu

1

Możesz dodać swój telefon w postaci linku a nie surowego tekstu. Będzie łatwiej dzwonić skajpajem :-)

Pozostało 580 znaków

Odpowiedz

1 użytkowników online, w tym zalogowanych: 0, gości: 1, botów: 0