Oki, to parę takich drobnych kwestii do rozważenia:
- dlaczego 1.jpg nie znajduje się w folderze img?
- dlaczego w folderze img masz różne standardy nazewnictwa plików, raz z podkreśleniami, za chwilę myślniki, a jeszcze potem spacje... jeśli przy tak małym projekcie nie panujesz nad tym to przy dużym na pewno się rozłożysz :) Nie chodzi tu stricte o obrazki bo akurat to może najmniej trafny przykład, ale np. już w plikach js, css itp. nazewnictwo jest bardzo ważne, co więcej niektóre frameworki wręcz wymagają pewnych konwencji. Warto zawsze myśleć o strukturze plików nawet jeśli to mikro projekt, chodzi o nawyki.
- dlaczego nazwaleś plik body-bg.jpg? Pytanie podchwytliwe :) Teraz oki, jest to tło bodziaka i wsio pasuje. Ale co jeśli apka by się rozwinęła to z czasem ten sam plik zacząłbyś używać jako tło nie wiem, buttona czy czegokolwiek? Nie chodzi mi, że jest to błąd, chcę Ci tylko zaznaczyć, że zawsze warto myśleć perspektywicznie o nazwach plików, funkcji, zmiennych itp. Kiedyś może zechcesz zmienić tło na inne, podmienisz więc ten plik i bum... nagle zmieniło się tło jakiegoś diva, czego nie otestowałeś bo zapomniałeś, i sugerowałeś się nazwą pliku :)
- Dlaczego wszystkich elementom nadajesz padding i margin zerowy? Nie jest to błąd, ale jest to pewne stare myślenie w kategoriach strony jako całości. Kiedyś nawet nagminnie stosowało się tzw. resety css (sam tego kiedyś używałem), ale znacznie lepiej jest myśleć o stronach jak o zestawach małych, reużywalnych komponentów i gdy wejdziesz w to myślenie to nagle żadne style globalne nie są potrzebne. To taka rada na przyszłość, z czasem sam się przekonasz.
-
.visually-hidden {
left: -999em;
position: absolute;
}
hmm, a dlaczego nie po prostu display: none? Być może jest jakiś sens w tych Twoich stylach, ale czy za pół roku na pewno będziesz wiedział o co kaman? Może błąd tkwi gdzieś w założeniach, przemyśl to.
-
Jeśli pracujesz na globalnych stylach dla całej apki to unikałbym styli typu .form > div itp. Jest to zbyt ryzykowne pod kątem przyszłych zmian, modyfikacji, rozbudowy itp. Wyjątkiem od tej zasady jest stylowanie komponentów, gdzie mamy pełną enkapsulację, w takich przypadkach pozwalam sobie na stylowanie nawet po tagach ale generalnie nie jest to dość częste i zależy np. od stopnia rozbudowania danego komponentu.
-
.form .username-input, .form .password-input, .form .confirm-password-input, .form .email-input, .form .location-input, .form .phone-input{
a dlaczego nie stworzyłeś po prostu osobnej klasy z tymi stylami, i nie przypisałeś jej do tych wszystkich elementów?
-
input[type=text], input[type=email], input[type=password], input[type=tel], input[type=number]
j.w.
-
o html się nie wypowiem bo tu nie jestem mistrzem i niechętnie wchodzę w pliki .html :)
Parę jeszcze uwag ogólnych:
- dlaczego w readme nie ma linku do dema skoro istnieje?
- masz na GH też swoje CV, nie przeglądalem go, ale pierwsze co widzę to plik reset.css. Gdybym miał Cię rekrutować to moje pierwsze pytanko po co ten reset, co Ci daje, dlaczego go używasz, jakie są alternatywy bez niego itp. Nie odpowiadaj, to takie pytanko dla Ciebie do przemyślenia tylko :)
- Kolejne pytanko, skoro krozystasz z zewnętrznej libki co nie jest złe, ale pytanko - czy umiesz i czy wiesz czy się w ogóle da (dać się da zawsze, ale chodzi o możliwość z poziomu libki bez ingerencji w jej core) np. zmienić ikonki na tych inputach itp.? Chodzi o to, aby sprawdzić, czy faktycznie czytałeś dokumentację libki i sam to robiłeś czy po prostu przekopiowałeś kod z jakiegoś innego miejsca :)
