Code-Review (HTML,CSS,JS) - Strona na turniej oraz "Twoja Półka"

Question

Witam

Proszę was o ocenę moich ostatnich projektów. Jeden przygotowałem niedawno,a dokładnie jest to strona na turniej e-sportowy. Starałem się,aby wygląd strony był jak najprostszy,aby uzyskać efekt flat design. Drugi natomiast stworzyłem kilka miesięcy temu i nie jest on responsywny,ale z okazji tego,że nauczyłem się korzystać z git postanowiłem też go wrzucić i poznać waszą opinie. Jak zawszę zależy mi na ocenie doświadczonych programistów oraz innych użytkowników  Dziękuje bardzo za pomoc 

Turniej: https://github.com/Cyborek/ZSEII-LANOWO    https://cyborek.github.io/ZSEII-LANOWO/

Twoja Półka: https://github.com/Cyborek/TwojaPolka   https://cyborek.github.io/TwojaPolka/

Comments

Opublikuj to na GitHub Pages – będzie łatwiej (w skrócie: git push master:gh-pages ;))

---

Postaram się to zrobić jak najszybciej. Jutro powinno być już wszystko ok.

---

Po prostu wejdź w ustawienia repozytorium i przejdź do Options -> GitHub Pages. Tam ustaw Source: master branch i podlinkuj :)

---

https://cyborek.github.io/TwojaPolka/ https://cyborek.github.io/ZSEII-LANOWO/

---

Ja to tylko tutaj zostawię:

http://tutorials.comandeer.pl/html5-blog.html

---

Muszę przyłożyć się bardziej do semantyki,bo dalej u mnie ona leży ;/

Answer 1

Moja opinia - stronki całkiem niezłe! Analizę kodu zostawię kolegom, a ja wypowiem się na temat wyglądu, co mi się nie podoba i co bym zmienił.

ZSEII Lanowo:

1) Tło strony jednolite, lepiej wyglądałoby przyciemnione zdjęcie na około 70-80% z jasnym logo,
2) Menu - pod podstrony statycznie dałbym wszędzie backgroundy ze zdjęciem (strona główna może zostać z zasadami, Facebooka dałbym po prostu w jego kolorach) i powiedzmy, że jak najedziesz myszką np. na Counter Strike - background zmieni się na kolorowy (gdzie statycznie ustawiasz je na szare),

3) Czcionki i ich krój - zmieniaj je, nie używaj jednolitej czcionki, np. w tabelach konkretnych grup ustaw inną czcionkę. Według mnie ogólna czcionka jest za duża, za mało konkretnej treści.

4) Co do reszty - jest okej, kolorystykę dobrałeś dobrze. Stwórz może jakieś logo zawodów? ;)

Twoja Półka:

1) W sumie większość problemów opisałem wyżej, jedynie czego można się doczepić to "skaczące" przywitanie, kiedy scrollujesz w dół,

2) Używaj większych odstępów pomiędzy blokami - wygląda to estetyczniej,

3) W początku, trzy kolumny z tekstami - spróbuj je wyjustować i zobacz efekt. :)

Ogólnie - pozytywnie!

Comments

Jestem pewny, ze Twoja odpowiedz bardzo pomoże w dalszym doskonaleniu zaprezentowanych stron oraz da cenne wskazówki innym użytkownikom tego forum.

Chciałbym tez zadać pewne pytanie (Oczywiście jeśli nie masz ochoty nie musisz na nie odpowiadać): a czy potrafisz napisać co Ci się podoba? Większość ludzi według mnie czasami do ocen podchodzi jak do "znajdź błąd" - sam czasami wpadam w ta, wydaje mi się, pułapkę. Popatrz często jest tak ze ktoś pisze: "Strona/rzecz/film jest bardzo dobra, jednak ja bym zmienił..." (pewnie ja tez tak pisałem i to wielokrotnie) i tutaj kilka zdań na temat tego co się nie podoba i dlaczego. Wydaje mi się że często brakuje tego co się właściwie tak podobało skoro jedno zdanie mówi o tym że się ogólnie podoba, a kilkanaście na temat tego ze coś bym zmienił bo się nie podoba i przez to nie wiemy tak na prawdę co się komuś w tym czymś podobało (a zwłaszcza jest to wydaje mi się widziane w szkole bo nauczyciel przeważnie sprawdzając sprawdzian szuka błędów (czyli szuka negatywów) i o tym co uczeń zrobił dobrze właściwie się milczy (a czasem powiedzenie takiego prostego zdania w gronie klasy: "świetnie zrobiłeś to czy tamto tak trzymaj" mogłoby umocnić ucznia i dać mu do zrozumienia ze coś tam potrafi zrobić), a o tych błędach się mówi i mówi).  Wielokrotnie tez widziałem takie zdanie "Ogólnie nie mam się do czego przyczepić" - i mam czasem wrażenie jakby ktoś próbował wyrazić w ten sposób ze ma pewna pretensje o to ze nie może znaleźć błędu, czy czegoś co się nie podoba.

Co o tym sądzisz?

---

Wszystkim nigdy nie da się dogodzić. Kiedy skończyłem tworzyć obie strony zapytałem się moich znajomych o opinie i jak to w życiu bywa ile ludzi tyle opinii :) Jednym się podobała,a inni w niektórych momentach kręcili głową. Sam swoje projekty pod względem wyglądu uważam za dość "znośne" jak na mój poziom.

---

To jest oczywiste ze się nie da dogodzić wszystkim, jednak przy tworzeniu stron trzeba mieć na uwadze że nie robi jej się wyłącznie dla siebie (no chyba że w ramach jakiś konkretnych ćwiczeń).

Answer 2

A no naprzyklad container moglby byc znacznikiem main.. ten tile w csgo mozna zamienic na section.. I imo nie wiem czemu uzywasz do tak banalnych rzeczy JQuery postaraj sie je wyeliminowac

Comments

JQuery użyłem tylko i wyłączenie do tego,a by uzyskać efekt płynnego rozwijania się nawigacji. Dzięki za rady ;)

Answer 3

TwojaPółka - 404

Turniej - po wejściu np. w zasady po lewej stronie na miejscu rozwijanego menu jest wydzielony margines, akurat pasujący na rozwijane menu.

 

Comments

Link powinien już działać. Jak nie to spróbuj drugim w komentarzach.