Gdyby miał ktoś czas: https://mateuszto.github.io/Portfolio/index.html
https://github.com/Mateuszto/Portfolio
Wiem, że nie za dużo do oceny, ale ważne to dla mnie
POMIJAJĄC NAZWY KLAS XD
0
0
I nawet nie chodzi o te tekstu bo przed chwila na grupce mnie ktoś wyśmiał odnoście tego tekstu ze współpraca myśląc, ze to poważnie. Chodzi mi o zwykły coder i czy strona estetycznie ładnie wyglada. Dziękuje
1
To zielone tło jest zbyt intensywne / oczojebne.
https://github.com/Mateuszto/Portfolio/blob/master/style.css#L7
#landingpage{
Staraj się nie używać id do stylowania elementów. Owszem, nadanie id się przydaje do robienia kotwic na stronie (tego po #, np. #about https://mateuszto.github.io/Portfolio/index.html#about ) i to jest okej, albo do tego, żeby jednoznacznie zidentyfikować elementy za pomocą JavaScriptu.
Natomiast id mają też taką cechę, że tylko jeden element na stronie może mieć dane id.
Więc jeśli stylujesz po id w CSS to zakładasz, że będziesz miał tylko jeden element #landingpage, jeden element #about itp. Więc to cię od razu ogranicza, bo jeśli będziesz chciał mieć np. about w dwóch miejscach (albo ogólnie dwa elementy stylowane podobnie) to się robi galimatias w CSS. Stylowanie po klasach pozwoliłoby ci po prostu na przypisanie do wielu elementów podobnej klasy (a do jednego elementu można przypisać nawet kilka klas). Ze stylowaniem id jest też taki ambaras, że ma większą specifity, o czym trzeba pamiętać https://developer.mozilla.org/en-US/docs/Web/CSS/Specificity
(specifity odpowiada za to, która deklaracja w CSS będzie "ważniejsza").
POMIJAJĄC NAZWY KLAS XD
nazewnictwo akurat jest bardzo ważne (bo to wpływa na utrzymanie kodu później), ale już mi się nie chciało i tak w to wnikać. Poza tym, że twarztekst brzmi jak WTF.