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
milossweb
Člen
Avatar
milossweb:22.9.2012 23:08

Zdravím,
prosím o kritiku webu http://miloss.cz/ konkrétně sekci Android http://miloss.cz/….php/android
Pište co se vám líbí/nelíbí, co byste změnili ...
Díky

 
Odpovědět
22.9.2012 23:08
Avatar
matesax
Tvůrce
Avatar
Odpovídá na milossweb
matesax:23.9.2012 6:53

Vypadá to velice profesionálně, ale:
JS do HTML souboru nepatří... (Měl by být sólo)
Proč nepoužíváš HTML5?
V CSS máš pár zbytečností.
Na mě je to předivované - uvítal bych HTML5 tagy - header, nav, article...

Ale jinak - hustodémonsky krutopřísné! :)

Editováno 23.9.2012 6:56
 
Nahoru Odpovědět
23.9.2012 6:53
Avatar
Kit
Tvůrce
Avatar
Odpovídá na matesax
Kit:23.9.2012 7:47
  • JS a CSS můžou v HTML být, zejména pokud se dynamicky generují pro každou stránku jinak. Společné části je dobré includovat.
  • Naopak se mi nelíbí, že těch includů JS a CSS je příliš mnoho. Ta stránka je velmi pomalá a to ji degraduje.
  • Mnoho webmasterů deklaruje XHTML a přitom používá HTML. Je to běžná chyba.
  • Ta stránka by si HTML5 zasloužila.

Ta stránka je napsána v RS Joomla, proto je dál zbytečné se pitvat v kódu. Pozitivní je, že ta stránka má obsah. Design se mi sice moc nelíbí, ale to je záležitostí osobního vkusu.

Editováno 23.9.2012 7:50
Nahoru Odpovědět
23.9.2012 7:47
Vlastnosti objektů by neměly být veřejné. A to ani prostřednictvím getterů/setterů.
Avatar
David Hartinger
Vlastník
Avatar
Odpovídá na milossweb
David Hartinger:23.9.2012 9:30

Také chválím obsah, web vypadá užitečně. Design se ale vůbec k androidu nehodí, nepoznal bych z toho, o čem web je. Hodilo by se něco více technického, pokud používáš RS, stačilo by jen upravit nějakou vhodnou šablonu.

Nahoru Odpovědět
23.9.2012 9:30
New kid back on the block with a R.I.P
Avatar
milossweb
Člen
Avatar
milossweb:24.9.2012 22:05

Děkuji za odpovědi. S šablonou si někdy určitě trochu pohraju, aby se to více hodilo k androidu. Ještě jednou děkuji za odpovědi. :)

 
Nahoru Odpovědět
24.9.2012 22:05
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.