Kritika webu cvcrn.8u.cz
http://cvcrn.8u.cz
Pri vložení tohto webu do kritiky, web získal od nášho robota nasledujúce hodnotenie:
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!
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 
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?
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-
- 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.
Zobrazené 13 správy z 13.