Vydělávej až 160.000 Kč měsíčně! Akreditované rekvalifikační kurzy s garancí práce od 0 Kč. Více informací.
Hledáme nové posily do ITnetwork týmu. Podívej se na volné pozice a přidej se do nejagilnější firmy na trhu - Více informací.

Kritika webu cvcrn.8u.cz

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

Při vložení tohoto webu do kritiky web získal od našeho robota 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

Aktivity
Avatar
Pavelka Jiří:4.9.2015 10:30

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
4.9.2015 10:30
Avatar
Odpovídá na Pavelka Jiří
Neaktivní uživatel:4.9.2015 10:33

Není co hodnotit

Nahoru Odpovědět
4.9.2015 10:33
Neaktivní uživatelský účet
Avatar
Odpovídá na Pavelka Jiří
Neaktivní uživatel:4.9.2015 17:36

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
Neaktivní uživatelský účet
Avatar
Odpovídá na Neaktivní uživatel
Pavelka Jiří:4.9.2015 19:17

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.9.2015 19:17
Avatar
Nahoru Odpovědět
4.9.2015 19:23
Neaktivní uživatelský účet
Avatar
Jurajs
Člen
Avatar
Odpovídá na Neaktivní uživatel
Jurajs:4.9.2015 19:36

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
4.9.2015 19:36
Avatar
Odpovídá na Jurajs
Pavelka Jiří:4.9.2015 20:16

Jakej je v tom rozdíl?

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

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
4.9.2015 20:20
Avatar
Odpovídá na Jurajs
Neaktivní uživatel:4.9.2015 20:39

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
4.9.2015 20:39
Neaktivní uživatelský účet
Avatar
Pavelka Jiří:4.9.2015 20:49

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

 
Nahoru Odpovědět
4.9.2015 20:49
Avatar
Odpovídá na Pavelka Jiří
Neaktivní uživatel:4.9.2015 20:50

Výborně, dobrá práce.

Zavřeno

Nahoru Odpovědět
4.9.2015 20:50
Neaktivní uživatelský účet
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.