Pierwszy represywny projekt z flexbox

Question

Witam i zarazem dziękuje dla każdej osoby, która poświecą swój cenny czas dla pomoc mniej doświadczonym użytkownikom.  Wciągu ostatnich tygodni sporo poświęcam czasu na naukę i udało mi się stworzyć proste portfolio.

Prosił bym o sprawdzenie jak bardzo z moim kodem jest źle i czy są jakieś karygodne błędy  .Pozdrawiam

ps. (Projekt nie jest w 100% skończony ,ma to być baza do nauki podstaw java czyli scrolowanie do sekcji i zmiana bg-color dla headera, dlatego też nie robiłem nawigacji )

Link

 

Answer 1

Za dużo do oceny tam nie ma. Jak skończysz projekt to można oceniać. Póki co buttony nie działają, social media się podświetlają, ale nie mają już cursor pointer ?

Czemu 

<textarea id="subject" name="subject" placeholder="Write something.." style="height:200px"></textarea>

Nie jest zrobione w css ?

Sam plik css wygląda ok, tylko brakuje formatowania.

Comments

Dziękuję, skończę i wtedy wrzucę całość.

Answer 2

Moim zdaniem:

cursor: pointer powinien być już ustawiony dla całego div.scroll , a nie tylko dla div.scroll a.

Ogólnie co do pisania styli to bym trzymał się zasady piszemy od najważniejszych właściwości (czyli width, height, position itp) dopiero potem kolory, background itp.

Strona wygląda solidnie, wygląda na responsywną. Tak trzymaj.

Answer 3

W wersji mobilnej średnio podoba mi się menu. Jest moim zdaniem trochę niewygodne. Zrób albo normalne takie menu mobile rozwijane z ikonką hamburgera, albo jeśli już chcesz się tego trzymać to daj jakiś padding bo opcje menu są za bardzo zlane z logo.

A tak to strona bardzo ładna i responsywna.