Kritika webu cvcrn.8u.cz

HTML a CSS Kritika webů - ukažte se! cvcrn.8u.cz American English version English version

Jak se ti líbí web?
Celkové hodnocení (6 hlasů):
11111
Přejít na web:
http://cvcrn.8u.cz

Při vložení tohoto webu do kritiky web získal následující hodnocení:

Kladných hodnocení: 12; Varování a poznámky: 2; Kritické chyby: 0.
Stránka je velice dobře optimalizována dle nejnovějších webových technologií. Je rychlá a dobře dostupná pro většinu uživatelů. Jen tak dál!

Český neoficiální validátor HTML stránek

Avatar
Pavelka Jiří:

Zhodnoťte prosím můj první web. Spíš by mě zajímal názor na kod (jak by se dal zjednodusit, optimalizovat, zvysit rychlost nacitani) než na obsah. Díky :)

 
Odpovědět  -2 4.9.2015 10:30
Avatar
Jiří Gracík
Redaktor
Avatar
Nahoru Odpovědět  +4 4.9.2015 10:33
Creating websites is awesome till you see the result in another browser ...
Avatar
Jiří Gracík
Redaktor
Avatar
Odpovídá na Pavelka Jiří
Jiří Gracík:

No dobře, na těch 34 řádcích html a 63 css kódu se pár chyb najde. Syntaxe trochu pokulhává, ale s tím se dá žít. Menu může být v navu místo divu. Favicon se nechává skoro zásadně v root složce webu. Znak copyrightu by měl být escapovaný. Po čárce i tečce se píše mezera. Co se týče css - nulovat margin a padding u všech elementů je čuňárna, není to vůbec potřeba. Proč je 3x za sebou definovaný clearfix, a co to vlastně má proboha dělat :D Ať to má dělat cokoliv, tak je celý zápis clearfixu nesmyslný. Proč by body měl mít position relative? Proč by měl mít oba atributy html a body nadefinovanou font-size, a ještě k tomu každý jinou? Proč by měl mít menu-box pozadí z gif obrázku? Menu se nezarovnává na střed, musel bys mu nastavit pevnou šířku. Atribut a je defaultně inline element. Proč používáš zápis velikosti fontu v procentech, em a px najednou? Ne že by zrovna tohle byla chyba, ale zajímal by mě důvod. Tučné písmo není úplně dobré používat na běžný text. Footer padding-top 430px? Tohle není asi úplně správně řešení.
To co ti řeknu dál k css je můj styl psaní css, který jsem si časem vytvořil z několika guideline a v praxi se osvědčuje, ale vůbec se tím nemusíš řídit - nauč se stylovat pomocí tříd, nepoužívej na stylování selektory s identifikátory nebo elementy (natož pak hvězdičku). Každá css vlastnost na jeden řádek (až na případy, kdy styluješ podobné věci - například ikonky a měníš různým ikonkám jen pozici pozadí, pak může být každý zápis na řádku). Jestli se tímhle zápisem snažíš zmenšit velikost souboru, tak si v budoucím vývoji budeš jedině škodit. Css k nasazení na ostrém webu bys stejně měl správně minifikovat. Tohle je za mě asi skoro všechno ke kódu.
Na závěr otázka ze zvědavosti. Jaký je tvůj cíl? Čeho chceš touto stránkou dosáhnout?

Nahoru Odpovědět 4.9.2015 17:36
Creating websites is awesome till you see the result in another browser ...
Avatar
Odpovídá na Jiří Gracík
Pavelka Jiří:

Včera když jsem se učil pracovat s photoshopem jsem narazil na fotku kluka s uhrama. Nějak sem se rozdováděl a potom ten kluk vypadal popáleně.. V tom jsem si vzpomněl na Natálku a tak to vlastně vzniklo ..

 
Nahoru Odpovědět  -4 4.9.2015 19:17
Avatar
Jiří Gracík
Redaktor
Avatar
Nahoru Odpovědět 4.9.2015 19:23
Creating websites is awesome till you see the result in another browser ...
Avatar
Jurajs
Člen
Avatar
Odpovídá na Jiří Gracík
Jurajs:

Mám dotaz, proč máš na svém webu v kódu CSS všechno třídy? Proč nepoužíváš i ID ? Z jakého důvodu?

Editováno 4.9.2015 19:38
 
Nahoru Odpovědět  ±0 4.9.2015 19:36
Avatar
Odpovídá na Jurajs
Pavelka Jiří:

Jakej je v tom rozdíl?

 
Nahoru Odpovědět  -4 4.9.2015 20:16
Avatar
Pavelka Jiří:

Vy jste všichni takoví sluníčkáři že nedáte like? Ten kdo nedá like má rád cikány. TAK. A BASTA!

 
Nahoru Odpovědět  -14 4.9.2015 20:20
Avatar
Jiří Gracík
Redaktor
Avatar
Odpovídá na Jurajs
Jiří Gracík:

Má to hned několik důvodů:

  • Máš element s ID a potřeboval by jsi mu přidat ještě další styl - občas mají oba styly stejnou váhu, takže bys musel náhodně jeden určit jako třídu a druhý jako id - je mnohem jednodušší je mít obě jako třídu.
  • ID nepoužiješ na víc elementů. Můžeš si říct, že tenhle element bude na stránce pouze jednou, ale pak se rozhodneš jinak a budeš to muset celé předělat
  • ID má vyšší váhu než třída. Pokud budeš chtít na nějaké stránce mít element z jakéhokoliv důvodu jiný, tak budeš muset použít important. Pokud bude třída deklarovaná nad id, opět musíš použít important. A takhle se to může nabalovat, až tvoje css nebude nic jiného než pseudonáhodná směs tříd, id a important. Kdykoliv budeš chtít něco změnit nebo přepsat, tak nikdy nebudeš vědět, co kde to udělá.
  • ID se běžně používá v javascriptu. Nikdy nechceš mít na identifikátoru závislý javascript i css, vždy je dobré to mít oddělené (budeš potřebovat použít funkcionalitu z js i jinde, nebo naopak ji nebudeš chtít ve finále použít vůbec a opět budeš přepisovat). Tenhle argument je trochu výmluva, protože na javascript používám třídy s předponou js- :D
  • třída jako klíčový selektor je na tom mnohem výkonostně lépe než selektor, kde vybíráš potomky elementu s identifikátorem ( například ".contact-list-item-link" vs "#contact ul li a" )

Dost možná jsem na něco zapoměl, ale tohle by asi mohlo stačit.

Nahoru Odpovědět  +2 4.9.2015 20:39
Creating websites is awesome till you see the result in another browser ...
Avatar
Pavelka Jiří:

Buď si rasista nebo cikán.. nic víc není

 
Nahoru Odpovědět  -7 4.9.2015 20:49
Avatar
Jiří Gracík
Redaktor
Avatar
Odpovídá na Pavelka Jiří
Jiří Gracík:

Výborně, dobrá práce.

Zavřeno

Nahoru Odpovědět  +3 4.9.2015 20:50
Creating websites is awesome till you see the result in another browser ...
Děláme co je v našich silách, aby byly zdejší diskuze co nejkvalitnější. Tato diskuze byla označena moderátorem jako nekonstruktivní a myslíme si, že nemá pro veřejnost příliš vysokou hodnotu. Členy můžete samozřejmě stále oslovit soukromě a to formou zpráv nebo chatu.

Zobrazeno 13 zpráv z 13.