Kritika webu http://www.autopujcovnafides.cz/
http://www.autopujcovnafides.cz/
V předchozím kvízu, Online test znalostí HTML a CSS, jsme si ověřili nabyté zkušenosti z kurzu.

Člen

Zobrazeno 12 zpráv z 12.
//= Settings::TRACKING_CODE_B ?> //= Settings::TRACKING_CODE ?>
V předchozím kvízu, Online test znalostí HTML a CSS, jsme si ověřili nabyté zkušenosti z kurzu.
Určitě líbí, jen tím logem Fides v pozadí bych trochu šetřil,
působí to divně
Nevím, čím to je, ale ten pravý pruh s rychlou kalkulačkou mám plácnutý vlevo přes text. Tím se web pro mne stává nepoužitelným.
Jaký prosím používáte prohlížeč? IE6? IE7?
Myslel jsem Internet Explorer. S tím vaším prohlížečem máte pravdu, chyba kodéra. Díky moc za přínosnou kritiku.
Internet Explorer se na mém počítači nevyskytuje v žádné verzi.
Ve stránce je příliš mnoho javascriptových includů. Bylo by vhodné je všechny spojit do jednoho nebo několika málo souborů. Výrazně to urychlí stahování aplikace a sníží zátěž serveru. Totéž platí pro soubory s CSS.
Deklarace XHTML 1.0 je chybná, protože stránka není v XHTML, ale v HTML neurčené verze. Doporučuji raději použít doctype HTML5.
V kódu vidím příliš mnoho nesémantických značek, nevhodným způsobem stylované nadpisy, odstavce apod. Když už includuje tolik souborů CSS, proč je nepoužívá? K čemu je tam tolik divů a spanů, když se dají přímo nastylovat značky?
dobra stranka, ale "menu:hover" neviem preco su 2 obrazky, ale mohlo by to byt nakodovane v css a aj ta kalkulacka by sa dala tiez a nemuseli by sa tie zaoblene rohy davat ako obrazky, a cim menej obrazkov, tim sa to rychlejsie nacita si myslim
co sa da napriklad spravit v css:
namiesto
div#h_menu a:hover, div#h_menu a.actual{
background: url(img/menu_hover_l.png) no-repeat left;
color: #ffffff;
text-decoration: none;
}
div#h_menu a:hover span, div#h_menu a.actual span{
background: url(img/menu_hover_r.png) no-repeat right;
color: #ffffff;
}
by to mohlo byt, priklad
div#h_menu a:hover, div#h_menu a.actual{
background: rgb(75,71,70);
border-radius: 10px;
color: #ffffff;
text-decoration: none;
}
namiesto
<div class="box_head"></div>
<div class="box"></div>
<div class="box_foot"></div>
iba
<div class="box"></div>
a upravit css na to
mozno by sa dalo aj nakodovat ten "indexBox" ale to by uz bolo asi viacej prace s tim
ak mam nieco zle, alebo ak som nieco zle pochopil, tak ma pls opravte
"Deklarace XHTML 1.0 je chybná, protože stránka není v XHTML, ale v HTML neurčené verze. Doporučuji raději použít doctype HTML5."
Tak nevím jak si to tohle přišel? To asi W3C kecá http://validator.w3.org/check?…
Na svém webu máš v HTTP hlavičkách:
Content-Type: text/html; charset=UTF-8
Pokud by to mělo být XHTML, musí to vypadat takto:
Content-Type: application/xhtml+xml
Prohlížeč to pak zpracovává úplně jiným způsobem.
Validátor W3C kecá. To už bylo potvrzeno dávno.
Zobrazeno 12 zpráv z 12.