Code review strony internetowej

Question

Witam, prosiłbym o ocenę kodu napisanej jakiś czas przeze mnie strony internetowej: co jest dobrze napisane, co poprawić oraz ewentualnie jakie nawyki byłoby dobrze wprowadzić :)

Miłego dnia,
Pozdrawiam

Comments

No nie żeby coś, ale code-review ... to robią ludzie odpłatnie.  To jest poświęcanie swojego czasu, na rzecz czyjejś korzyści.

Rozumiem jakiś fragment kodu, problemu, optymalizacje, ale nie projektu strony.
Zresztą sory, ale ciężko coś tu ocenić skoro to jest strona napisana w HTML, CSS tylko i wyłącznie, generalnie to jest nie strona a utworzenie szablonu... graficznego.

---

Rozumiem, zależałoby mi na w miarę ogólnym spojrzeniu na całość i ocenieniu czy nie pojawiają się jakieś większe błędy

---

tylko ze dla projektu w którym piszesz tylko w SCSS, CSS, HTML i importujesz zdjęcia, bardzo trudno mówić o błędach, bo tu wartości twórczej "twojej" jest bardzo mało.

---

No nie żeby coś, ale code-review ... to robią ludzie odpłatnie.  To jest poświęcanie swojego czasu, na rzecz czyjejś korzyści.

@Wiciorny, a czy pomaganie odpowiadając na pytania nie jest poświęcaniem swojego czasu na rzecz czyjejś korzyści? Jaka jest różnica (oprócz ilości poświęconego czasu) czy pomożesz komuś w formie zrobienia code review, czy odpowiesz na pytanie lub podasz wskazówki do rozwiązania problemu? Poza tym, na forum pomoc jest dobrowolna i jeśli nie chcesz poświęcać komuś czasu, to nie musisz.

Zresztą sory, ale ciężko coś tu ocenić skoro to jest strona napisana w HTML, CSS tylko i wyłącznie, generalnie to jest nie strona a utworzenie szablonu... graficznego.

Przecież strona internetowa nie musi zawierać nie wiadomo jak skomplikowanego kodu, czy nawet JavaScriptu. I to, że strona jest prosta, nie znaczy że nie ma tam czego oceniać.

---

@Layoutowiec, czy mógłbyś tą stronę pokazać w formie live? Choćby na GitHub Pages, jeśli nie masz własnego hostingu.

---

@Layoutowiec, czy mógłbyś tą stronę pokazać w formie live? Choćby na GitHub Pages, jeśli nie masz własnego hostingu.

Jasne, dodałem do treści posta 

---

@Layoutowiec, strona z poziomym paskiem przewijania (na ekranie o 1366 px szerokości) to nie jest to, co tygrysy lubią najbardziej.

---

Racja, dziękuję za czujność :)

Answer 1

Przede wszystkim masz tylko 1 commit, musisz zacząć lepiej używać git'a. Po drugie pierwszy plik scss'owy do którego zajrzałem ma prawie 1000 lini kodu. To za dużo. Masz tam dużo powtarzających się fragmentów kodu, co scss powinien eliminować

Comments

po trzecie, niepotrzebnie trzymasz pliki .map i main.css w repo... używaj .gitignore.

Po czwarte, zerknij na narzędzia takie jak prettier i styleline

---

Dzięki za pomoc, na pewno poczytam i zastosuję :)

Answer 2

• brakuje sekcji: header i main
• brakuje nagłówka <h1>
• obrazki o klasach v-line i h-line nie wydają się istotne dla zrozumienia treści strony, więc można je wstawić jako background w CSS zamiast jako <img>
• linki w nawigacji nie prowadzą do odpowiadających im sekcji (w href jest sam #, bez podanego identyfikatora sekcji)