Autor Zpráva
Gwidion
Profil
Prosím o zhodnocení kódu...

http://www.dieselpoint.cz/
Měsíček
Profil
1.) Je nějáký důvod zalamovat <title>?
2.) Je nějáký důvod pro použití META tagu "robots", nic nezakazuješ, takže si odpovím sám - není.
3.) Pro vytváření menu máme v HTML 4.01 tagy <ul> | <menu> a <li> ušetříš si tak zbytečnou třídu "menu".
4.) Proč používáš NET zápisy (<br />)? Víš k čemu slouží?
5.) Tato stránka není natolik složitá a zákeřná, aby jsi musel použít podmíněný komentář.
6.) Nauč se zkracovat CSS vlastnosti z background-color, na background atd ..
7.) To samé jako u bodu 6 aplikuj na vlastnosti margin a padding.
8.) Používáš zbytečně mnoho divů, jen nevím proč celé by se to dalo s trochou sebezapření a pár křápanců hlavou do klávesnice zmenšit.

Toť mé rychlostní postřehy, snad ještě někdo něco přihodí.
Nox
Profil
- description bych nedělal takové povídání ("je od svého vzniku (1992)","V důsledku zavedení elektronicky řízených systémů vstřikování paliv") a napsat to stručněji a rázněji (to je možná spíš k obsahu, ale dejme tomu)
- místo <div id="menu"> dát <menu> nebo <ul>
- <img src="liskrdol.png" alt="dolní lišta" id="lista2"> ...udělat přes css
- urchin je starší skript GA, mohlo by se updatnout
- proč jsou všechny adresy absolutní cestou? pokud to takto bude všude a bude třeba stránky vyvíjet / upravovat mimo ostrou verzi,
tak se vše bude muset přepisovat...

- docela přehledně psané (občas by mohl být volný řádek), alty
gyoja
Profil
1) V logu máš jen obrázek, s vypnutými obrázky nikdo nepozná čí, že to web vlastně je.
2) Přejmenoval bych obrázek banner.png, třeba ve Firefoxu mi ho zachytil Adblock a nezobrazil ho.
3) Obrázky sloužící jako lišty je lepší strkat do souboru *.css a ne do vlastního html
Měsíček
Profil
Ještě pár mých perliček ..


9.) Proč u některých odkazů píšeš title a u některých ne? Máš v tom systém (to samé s vyplňováním altů)?
10.) Nebyl by lepší pro "patku" obyčejný odstavec (<p>)?
11.) Na jedné straně definuješ klidně 6 barev, proč to? Stačí bohatě dvě barvy (případně tři).
12.) Logo by mohlo být klikací.
13.) Možná by se dole někde šikla kotva, která by vytáhla stránku nahoru.
14.) Zase další perlička, která ukazuje, že si píšeš jak tě napadne .. proč některé odkazy mají text-decoration:none; a některé ne?
A přitom jsou na stejné úrovni.
15.) Nějáké odsazení prvního řádku v odstavci by nebylo?
16.) Pro adresu máme HTML tag <address>.
Gwidion
Profil
Měsíček
ad 1) to se asi udělalo samo při formátování kódu, vadí to?
ad 2) na seo-servis.cz to chtěj, tak jsem to tam dal, smysl toho také nevidím...
ad 3) díky, zařídím se podle toho
ad 4) to je XHTML zápis, nemám pravdu? Vadí to nějak tomu kódu?
ad 5) myslíš v HTML, nebo v CSS (nevím, co je podmíněný komentář)
ad 6 a 7) to je jenom kvůli velikosti přenesených dat? jinak díky, zařídím se
ad 8) dobrá, zkusím s tím něco udělat...

Nox
description se pokusím zkrátit... menu upravím... s tím obrázkem jako lištou jsem si nevěděl rady (abych tam nedával zbytečně moc divů, ale abych měl lištu nahoře i dole a čáry (border) po stranách).. díky, updatnu skript GA... adresy opravím...

gyoja
ad 1) nenapadá mě asi nic chytrého (lepšího)
ad 2) ADP používám taky, nikdy mi to nebloknul... Budu přemýšlet, jak ho přejmenovat...
ad 3) psal jsem už jako reakci na Noxe
Gwidion
Profil
Měsíček
ad 9) když to je odkaz na interní stránku, tak tam nemam title, když ven, tak tam je... Jinak alty u obrázků mám doufám všude.
ad 10) na tom tolik nesejde, ne?
ad 11) s barvama pořád bojuju, pokusím se s tím něco udělat...
ad 12) logo je klikací (pokud tím myslíš odkaz)
ad 13) ty stránky nejsou moc vysoké, nikdy jsem nad tím neuvažoval, jaká je v tom výhoda?
ad 14) pokusím se to sjednotit...
ad 15) je to pouze na dojem, nebo to má nějaký účel? Mně se to moc nelíbí...
ad 16) vždyť to tam mám!
Měsíček
Profil
"ad 1) to se asi udělalo samo při formátování kódu, vadí to? "

Vadit čemu? Jestli funkčnosti stránky - tak ne.

"ad 2) na seo-servis.cz to chtěj, tak jsem to tam dal, smysl toho také nevidím... "

SEO-servis ví o SEO asi tolik jako já o kvantové fyzice. Řiď se podle toho co víš, ne podle toho co jsi "někde viděl".

"ad 4) to je XHTML zápis, nemám pravdu? Vadí to nějak tomu kódu? "

Přímo XHTML bych to nepřisuzoval. Kódu to vadí, prohlížeči ne, .. zatím. V každém případně zbytečné dva znaky navíc.

"ad 6 a 7) to je jenom kvůli velikosti přenesených dat? jinak díky, zařídím se "

Možná taky kvůli přehlednosti a rychlosti případné opravy.
AlešD
Profil
Když hodnotit kód, tak tedy kód:
- přijde mi to nějaké předivované:
 <body>
        <div id="vse">
        ....
            <div id="patka"> © 2007 Gwidion
            </div>
        ....
        </div>
 </body>


to by nestačilo nastylovat rovnou BODY?
- logo: použil bych text skrytý obrázkem
- a tohle je vysloveně fuj, od čeho je CSS?
<img src="./delphi.gif" alt="logo firmy DELPHI" class="logo_hlavni_stranka"><br /><br />

- patří do CSS ne do HTML (oddělení vzhledu od obsahu)
<img src="lisdldol.png" alt="dolní lišta" id="listadol">

- ostatní už předřečníci uvedli, doufám ,že se neopakuji (moc)
Gwidion
Profil
AlešD
to by nestačilo nastylovat rovnou BODY?

nestačilo, protože by se mi rozhodilo v IE floatování, který je takhle uzavřený v kontejneru...
Gwidion
Profil
Upravil jsem svůj kód podle vašich rad, děkuji vám všem. Jediné, s čím si nevím rady, jsou ty dolní lišty, jak je do CSS nasáčkovat spolu s horníma lištama a orámováním. Založil jsem proto téma v sekci CSS: http://diskuse.jakpsatweb.cz/index.php?action=vthread&forum=7&topic=87 072

Kdybyste někdo v upraveném kódě viděl někdo něco, co by šlo zlepšit, dejte prosím vědět.

Vaše odpověď

Mohlo by se hodit


Prosím používejte diakritiku a interpunkci.

Ochrana proti spamu. Napište prosím číslo dvě-sta čtyřicet-sedm:

0