Ocena Strony

Question

Witajcie
Chciałbym Was poprosić o ocenę kolejnej strony, którą napisałem w ramach nauki webdevu.
https://kacperszo.github.io/cukiernia/
Starałem się zastosować do wcześniej udzielonych mi rad i tym razem nie użyłem bootstrapa.
Dzięki ;)

Answer 1

Trochę 'ciężkie' te obrazki. Ponad 2 mb.

Ja bym dodał jakiś button, który przenosi z samego dołu do menu, bo po zjechaniu, strasznie dużo trzeba scrolloować.

Co do kodu, to na szybko rzuciło mi się w oczy, że stylujesz po ID .

Polecam - https://kot-zrodlowy.pl/goscinne/programowanie/2017/09/11/najczeste-bledy-poczatkujacych-i-nie-tylko-webdeveloperow.html#stylowanie-po-id

Jeżeli używasz scss to czemu nie używasz zmiennych ?

Comments

Pierwszy raz go użyłem(sass-a) i głównie po to, żeby sobie ułatwić zagnieżdżanie, nawet niespecjalnie wiem co miałbym trzymać w takiej zmiennej.
A co do stylowanie to rozumiem, że nigdy nie powinno się stylować po id?

---

Nigdy nie stylujemy po ID.  :)

Co do zmiennych :ustawiasz padding dla nagłówków 

    padding-top: 3em;

to jeżeli teraz sobie wymyslisz, że jednak wolisz 4em, to dla każdego nagłówka musisz zmieniać osobno, a mógłbyś ustawić zmienna 

$padding-top-for-heading: 3em;

h1 {
padding-top: $padding-top-for-heading;
}
h2 {
padding-top: $padding-top-for-heading;
}