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
Jan Barvíř
Neregistrovaný
Avatar
Jan Barvíř:15.1.2013 19:16

Zdravím všechny,

poprosil bych o kritiku k filmovému webu http://NaFilmu.cz. Fungujeme zhruba rok, celkem se daří, ale určitě by to mohlo být lepší. Konkurence je obrovská, rozhodně to ale nevzdáváme, naopak. :)

 
Odpovědět
15.1.2013 19:16
Avatar
Luboš Běhounek Satik:15.1.2013 19:22

Mě se to líbí, žádná výtka mě nenapadá :)

Nahoru Odpovědět
15.1.2013 19:22
https://www.facebook.com/peasantsandcastles/
Avatar
matesax
Tvůrce
Avatar
Odpovídá na
matesax:15.1.2013 20:10

Proč je deklarováno XHTML? Deklarujte normálně HTML5. Přijde mi to spíše České - než Anglické:

<!DOCTYPE html>
<html lang="cs-cz">

Proč je tolik JS na stránce? Proč je tam tolik divů a spanů? Proč je tam tolik JS a CSS souborů? CSS má smysl dělit pro 2 diametrálně odlišné stránky/controly - kdy pak chceš určité části kombinovat... A JS stačí ve 4 souborech...

"menu-item menu-item-type-post_type menu-item-object-page menu-item-13010"

To jako vážně? Já nevím, ale přijde mi, že kratší by to mohlo být...

Některé komentáře jsou absolutně zbytečné - jako třeba konec html - to snad vidím - ne? :)

Takto kontrukce:

<div class="info">
                                                        Leden 15, 2013                                                                                                                  <a href="http://nafilmu.cz/2013/01/ben-hur-bude-opet-zremakovan/#comments" class="comments">
                                                                        0                                                               </a>
                                                                                                        </div>

Mi přijde poněkud rozbitá - proč to není nějak pokupě?

Kód by se dal zkrátit možná až na polovinu - a tím i zrychlit načítání stránky a celkově ji učinit svižnější. Protože načítání této stránky je poněkud ďábelské - to snížení použitých souborů tomu rozhodně prospěje - jakož i snížení JS na stránce.

Projeď to validátorem - to je také krapet násilnické... :)

Design by ušel - ale jen tím, jak to vypadá - ne uskupením a obsahem. První co mne uvítá je nějaký velký banner + ty zprávy nahoře také nepřidají na uvítání.

Takže to co první uvidí uživatel by mělo být to, co je primárním účelem webu. Tedy něco, co mne vtáhne dovnitř a zbytek okolo mne má jen udržovat - popř. informovat i o něčem trochu jiném. Ale hlavní část by měl zabírat primární obsah - a nějaké přehledné uskupení - toho, co tu vlastně mám dělat. Chtělo by to udělat jasné uskupení - aby byly od sebe odděleny části, co spolu nesouvisí... Zde je to přesně naopak - většinu stránky zabírá nějaký ten banner + reklamy - a k obsahu musím scrollovat...

 
Nahoru Odpovědět
15.1.2013 20:10
Avatar
David Hartinger
Vlastník
Avatar
Odpovídá na
David Hartinger:15.1.2013 20:46

Až na JS v kódu hezký web :)

Nahoru Odpovědět
15.1.2013 20:46
New kid back on the block with a R.I.P
Avatar
Ex_0
Člen
Avatar
Ex_0:16.1.2013 22:16

v kalendari by sa mozno zislo nejako farebne odlisit dni, ktore uz boli, alebo na ktorych je nejake info aby to bolo prehladnejsie :)

a este v menu by sa mozno dalo pouzit namiesto obrazku css3 v ul li ul li

background: -webkit-gradient(linear, left top, left bottom, color-stop(0%,#f0f9ff), color-stop(100%,#e9e9e9));
background-size: 169px;
border-bottom: 1px solid #D2D2D2;

skusal som to narychlo len upravit si v chrome a vyzeralo to tak isto ako obrazok

na hover mozno dojdes ak nie zajtra ti ho napisem, zatial mozes skusit :)

Editováno 16.1.2013 22:19
 
Nahoru Odpovědět
16.1.2013 22:16
Avatar
matesax
Tvůrce
Avatar
Odpovídá na
matesax:16.1.2013 22:34

Uspořádání již celkem ujde. Pořád je tam ale nějak moc souborů - a JS na stránce minimalizujte. Jde o to, že to pak zdržuje a zatěžuje...

Neuškodilo by použít HTML5 layout:

header
nav
aside
article
footer

Editováno 16.1.2013 22:37
 
Nahoru Odpovědět
16.1.2013 22:34
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 6 zpráv z 6.