Ocena strony HTML+CSS

pytanie zadane 26 listopada 2017 w Nasze projekty przez marynovski Początkujący (250 p.)

Jeśli ktoś mógłby ocenić moją stronę pod względem poprawności kodu - byłbym bardzo wdzięczny.

Oto link do strony na darmowym hostingu: Strona

A tu link do repozytorium na Github: Kod na github

komentarz 26 listopada 2017 przez imklau Nałogowiec (42,090 p.)

Standardowe błędy, które możesz sam wyłapać przy przeglądaniu kodu innych osób tutaj na forum. Wystarczy wpisać w wyszukiwarkę "ocena strony" ;)

PS. jak masz stronkę na Githubie to zainteresuj się czymś takim jak Github Pages. Dzięki temu za darmo sobie zrobisz podgląd strony i nie będzie reklam.

komentarz 26 listopada 2017 przez DragonCoder Nałogowiec (36,500 p.)

1. Logo na gorze, jest cudowne (sarkazm), nie uzywa sie png, tylko svg do logo lub co kolwiek do grafiki wektorowej

2, SameLowPrice - tutaj bym popracowal nad farbami, podstawy plastyki gdy mieszamy kolory cieple i zimne

3. Gdy najedziesz na firma, o zaslania filmik, obnizylbym wszystko lekko i tyle

4. Zddjecie na gorze,, jest takie sobie robione pod swiatlo :D

5. Kontakt - pola edycji sprawdz je jeszcze

6. Filmik fajny, muzyka tez ale nie widze polaczenia z tematyka strony

komentarz 26 listopada 2017 przez marynovski Początkujący (250 p.)

Ok, dzięki za polecenie tego Github Pages - ogarne :)

komentarz 26 listopada 2017 przez marynovski Początkujący (250 p.)

1,2,4 - No grafikiem to ja jestem ŻADNYM, ale na pewno wezmę sobie te rady do serca i spojrzę na to chwilę
6. To prawda... wstawione tylko na próbę xd
Dziękuję za poświęcenie nawet chwilki :)

1	komentarz 26 listopada 2017 przez radek024 Szeryf (77,160 p.) Odnośnie Github Pages: https://radek024.github.io/blog/strony-www/2017/07/14/github-pages/. Na stronę zerknę jutro, o ile nie zapomnę :)

1 odpowiedź

odpowiedź 26 listopada 2017 przez JayJay Mądrala (6,020 p.)
wybrane 27 listopada 2017 przez marynovski

Najlepsza

Kilka elementów, które warto byłoby poprawić:

Postaraj się nadawać (wszystkim) elementom style w CSS, nie stosować styli inline.
Lepiej ze względów "dostępności" będzie zastosować "span" (z atrybutem aria-hidden="true") zamiast "i" w przypadku ikon.
Obrazy powinny mieć właściwe dobrany "alt" (czasami też puste, jeżeli jest to konieczne).
Zamiast "br'ów", stwórz potrzebny efekt z pomocą CSS.
"Clear:both" możesz stworzyć w CSS, bez dodatkowego div'a w html lub od razu przejść na flexbox'a ;)
Brakuje "title" strony (w sekcji head).
Możesz sprawdzać stronę pod kątem (prostych) błędów w html z pomocą walidatora (https://validator.w3.org/).
Niewłaściwie korzystasz z "aside". Służy on do tworzenia sekcji luźno powiązanej z główną treścią, mogącej być przedstawioną samodzielnie, np. cytaty, wyjaśnienia pojęć użytych w artykule itp.

Kilka (według mnie) wartych uwagi materiałów:

Comandeer: Dostępność - po co? Na co? Dlaczego?

Semantyczny blog w HTML5

The Clearfix: Force an Element To Self-Clear its Children

Understanding aside

Flexbox Froggy