Strona do oceny - flex box

pytanie zadane 16 czerwca 2015 w HTML i CSS przez tarnasm Gaduła (3,030 p.)
edycja 16 czerwca 2015 przez tarnasm

Hej.

Wrzucam stronkę, ktorą aktualnie robię na zaliczenie. Troche nadziabany, kod (sorry). Ale to wynika z faktu iż pierwszy raz robię stronę opartą na flex-boxach...i trochę kombinowalem. Ocencie ją pod względem wizualnym (chociaż to w połowie szkic). I zerknijnie na kod.

http://codepen.io/tarnasm/pen/dovOVR

Pozdrawiam

Grzegorz.

8 odpowiedzi

odpowiedź 16 czerwca 2015 przez Mizukage Pasjonat (21,750 p.)

odpowiedź 16 czerwca 2015 przez chris Gaduła (3,680 p.)

odpowiedź 16 czerwca 2015 przez testerius Pasjonat (23,960 p.)

Mógłbyś zmienić trochę typografię, tzn. ustalić line-height, font-size, color, jakieś marginy itd. Co do reszty no to widać, że wymaga jeszcze jakiegoś dopracowania, ale myślę, że jako prosta strona jest ok - to co bym poprawił to przede wszystkim pozbycie się id z arkusza stylów i zapewne jeszcze jakoś to wszystko inaczej pozapisywał, ale dobra.

Kod w codepen możesz "przeczyścić" za pomocą buttona tidy - jeśli miałeś na myśli to, że nie jest sformatowany. :P

komentarz 16 czerwca 2015 przez tarnasm Gaduła (3,030 p.)

Mógłbyś zmienić trochę typografię, tzn. ustalić line-height, font-size, color, jakieś marginy itd.

Masz na myśli zdefiniowanie tych atrybutów dla całej sekcji body na pozątku kodu?

o co bym poprawił to przede wszystkim pozbycie się id z arkusza stylów

Możesz rozwinąć myśl? Chodzi Ci o to aby stosowac same klasy?

komentarz 16 czerwca 2015 przez eliano Gaduła (3,640 p.)

Masz na myśli zdefiniowanie tych atrybutów dla całej sekcji body na pozątku kodu?

tak

Chodzi Ci o to aby stosowac same klasy?

tak

komentarz 16 czerwca 2015 przez testerius Pasjonat (23,960 p.)

Ktoś mnie ubiegł, ale tak - zamiast id stosuj klasy, id powinno się używać po to aby skrypty mogły odnieść się do konkretnego elementu.

Nom, w body sobie ustal krój czcionki, wielkość i inne style - tak jak będzie ci pasować.

komentarz 16 czerwca 2015 przez tarnasm Gaduła (3,030 p.)

A nie jest czasem tak, że w ramach dobrej praktyki powinno się wszędzie zerowac marginesy, padding do 0?

EDIT:

zamiast id stosuj klasy, id powinno się używać po to aby skrypty mogły odnieść się do konkretnego elementu.

Nie bardzo rozumiem co masz na myśli. Nie jest tak, że id stosujemy przy unikalnych divach a klasy przy tych, ktore użyjemy więcej niż 1 raz?

komentarz 16 czerwca 2015 przez eliano Gaduła (3,640 p.)

napisałem to za ciebie, bo byłem absolutnie pewny co masz na myśli:

ustalanie tych rzeczy w body to absolutny standard - wszyscy tak robią.
używanie samych klas zamiast id to natomiast jedno z założeń BEM

komentarz 16 czerwca 2015 przez tarnasm Gaduła (3,030 p.)

Dziękuję, to wiele mi wyjaśnia. Sprawdzę w necie jak przedstawiają się założenia o ktorych mówiliście:)

Pozdrawiam.

komentarz 17 czerwca 2015 przez testerius Pasjonat (23,960 p.)

SMACSS, OOCSS i jeszcze DRY - o tym możesz poczytać.

komentarz 17 czerwca 2015 przez tarnasm Gaduła (3,030 p.)

Uff dużo do nauki :D Te wszystkie metody faktycznie są stosowane jednakowo? Czy zamiennie?

odpowiedź 16 czerwca 2015 przez HaKIM Szeryf (87,590 p.)

Ładnie.

Mam takie pytanko, strona jest responsywna? Niestety na codeopen.io nie jest to widoczne. ;c

komentarz 16 czerwca 2015 przez tarnasm Gaduła (3,030 p.)

Tak, jest. Własnie zauważyłem, że na niektórych przeglądarkach CodePen nie układa strony właściwie, albo w ogóle...Nie wiem od czego to zależy przyznam szczerze.