IT rekvalifikace s garancí práce. Seniorní programátoři vydělávají až 160 000 Kč/měsíc a rekvalifikace je prvním krokem. Zjisti, jak na to!
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í.
Avatar
Miloš Havlíček
Neregistrovaný
Avatar
Miloš Havlíček:28.12.2012 9:07

Ahoj, můžu poprosit o kritiku webu RestauraceCesko.cz? Předem díky! :-)

 
Odpovědět
28.12.2012 9:07
Avatar
Luboš Běhounek Satik
Autotvůrce
Avatar
Luboš Běhounek Satik:28.12.2012 9:45

Snizil bych trochu kvalitu obrazku na pozadi (to drevo), takhle to ma 600 KB a pri prvnim zobrazeni se to nacita docela dlouho

Nahoru Odpovědět
28.12.2012 9:45
https://www.facebook.com/peasantsandcastles/
Avatar
David Čápka Hartinger
Tým ITnetwork
Avatar
Odpovídá na Miloš Havlíček
David Čápka Hartinger:28.12.2012 10:00

Web je docela dobře zvládnutý, jen v kódu je hodně obtruzivního JavaScriptu, hlavně mám na mysli zastaralé události onclick, které patří do DOMReady.

Nahoru Odpovědět
28.12.2012 10:00
You are the greatest project you will ever work on.
Avatar
matesax
Tvůrce
Avatar
Odpovídá na Miloš Havlíček
matesax:28.12.2012 10:02 
xmlns="http://www.w3.org/1999/xhtml" xml:lang="en"

Proč mi to děláte? :)

S tím JS to beru jako předčasný apríl... Maximálně tak 4 soubory - více opravdu ne... :) (Také by to chtělo trošku zmodernizovat. :) )

CSS - totéž - smysl má dělit CSS pro různé šablony - pro jednu v tom smysl nevidím. Leda tak CSS s PHP a bez - ale i to...

JS na stránce být nemusí - alespoň se netváří, jako generovaný PHP, či tak...

Tabulky jsou jedny z nejnáročnějších elementů - když se jim můžeš vyhnout - vyhni se... (Zde to jde.)

Hezčí by bylo při deklaraci HTML5 dělat HTML5 web:

header
nav
article
footer

Zde bezesporu chybí významy slovům a slovním spojením - nulová potrava pro vyhledávače atd...

Je zde na mne mnoho zbytečných konstrukcí atp. - dalo by se to dost osekat a zrychlit...

Editováno 28.12.2012 10:03
 
Nahoru Odpovědět
28.12.2012 10:02
Avatar
Kit
Tvůrce
Avatar
Odpovídá na Miloš Havlíček
Kit:28.12.2012 10:29

Deklarováno XHTML, ale web je v HTML. Doporučuji změnit deklaraci na HTML5, pak to bude OK.

Děsné kvantum externích javascriptů zdržuje načítání webu. Doporučuji sloučit do 1-2 souborů.

Doporučuji v kódu nepoužívat absolutní URL pro interní navigaci.

Nevidím žádný nadpis h1.

Nahoru Odpovědět
28.12.2012 10:29
Vlastnosti objektů by neměly být veřejné. A to ani prostřednictvím getterů/setterů.
Děláme co je v našich silách, aby byly zdejší diskuze co nejkvalitnější. Proto do nich také mohou přispívat pouze registrovaní členové. Pro zapojení do diskuze se přihlas. Pokud ještě nemáš účet, zaregistruj se, je to zdarma.

Zobrazeno 5 zpráv z 5.