Vydělávej až 160.000 Kč měsíčně! Akreditované rekvalifikační kurzy s garancí práce od 0 Kč. Více informací.
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
Neaktivní uživatel:16.12.2013 14:22

Tak, vše nakódováno. :)

Odpovědět
16.12.2013 14:22
Neaktivní uživatelský účet
Avatar
Honza Bittner
Tvůrce
Avatar
Odpovídá na Neaktivní uživatel
Honza Bittner:16.12.2013 14:34

Vzhled vypadá dobře ...

jQuery kód pro posun je špatný, proč třeba $target? ...

Rozvržení elementů v HTML také nic moc, proč například máš hlavičku na 2 části - header a div#home?

Zjistil jsem že element <main> modernizr nepodporuje, tzn. třeba v IE mi nefungoval, na to bacha, kdyžtak vyzkoušej. :)

Proč dáváš ve formuláři do odstavce odstavec a input?

Nahoru Odpovědět
16.12.2013 14:34
FIT ČVUT alumnus :-) Sleduj mě na https://twitter.com/tenhobi a ptej se na cokoli na https://github.com/tenhobi/ama.
Avatar
Jan Vargovský
Tvůrce
Avatar
Odpovídá na Neaktivní uživatel
Jan Vargovský:16.12.2013 14:43

Když už se prezentuješ na webu a nejde ti moc čeština, nech si to raději od někoho zkontrolovat.

 
Nahoru Odpovědět
16.12.2013 14:43
Avatar
Odpovídá na Jan Vargovský
Neaktivní uživatel:16.12.2013 14:46

Text jsem psal opravdu narychlo a stejně se bude přepisovat, takže za chyby se omlouvám. :)

Nahoru Odpovědět
16.12.2013 14:46
Neaktivní uživatelský účet
Avatar
Jan Vargovský
Tvůrce
Avatar
Odpovídá na Neaktivní uživatel
Jan Vargovský:16.12.2013 14:48

Mi se neomlouvej, jen je to první věc co tě trkne do oka :) každopádně ty statistiky co jak umíš bych dal více k sobě. Vždyť na širokoúhlém monitoru jezdím zleva doprava a už ani nevím jaký to byl řádek.

 
Nahoru Odpovědět
16.12.2013 14:48
Avatar
Odpovídá na Honza Bittner
Neaktivní uživatel:16.12.2013 14:57
  • Ten JS kód není můj, javascript neumím. :D
  • To rozvržení elementů je kvůli CSS, ale asi by se to dalo udělat lépe.
  • I můj editor mi <main> neoznačuje a když jsem to testoval na IE, <main> nedostal pozadí. Asi to bude tím. Nevíš, čím bych to mohl nahradit?
  • Ježiš, to jsem zapomněl opravit. :D Ten vnořený odstavec byl původně <label>. Ani bych si nevšiml.
Nahoru Odpovědět
16.12.2013 14:57
Neaktivní uživatelský účet
Avatar
Honza Bittner
Tvůrce
Avatar
Odpovídá na Neaktivní uživatel
Honza Bittner:16.12.2013 15:20

Nahraď ho třeba <div id="main">, to je jedno... :)

Nahoru Odpovědět
16.12.2013 15:20
FIT ČVUT alumnus :-) Sleduj mě na https://twitter.com/tenhobi a ptej se na cokoli na https://github.com/tenhobi/ama.
Avatar
Nahoru Odpovědět
16.12.2013 15:46
Neaktivní uživatelský účet
Avatar
FastNode
Tvůrce
Avatar
FastNode:22.12.2013 1:58

Design je super, ale stejně si neodpustím pár poznámek:

Na podnadpisy "O mně", "Reference" a "Kontakt" bych radši použil <h2>. Tím by odpadla i nutnost použít class u h1, ten by tam zbyl pouze jeden, právě na hlavní nadpis. Navíc by se nebořila i hierarchie nadpisů (teď máš nejdřív h1, pak rovnou h3).

Nějak se ti tam objevil překlep u sekce kontakt. Stejně tak "Obsah zpráva" ve formu nezní zrovna dobře :D

K těm CSS, všechny by šly sloučit do jednoho CSS souboru. Nevím, nakolik by to ušetřilo nebo neušetřilo traffic, ale ta možnost tu je :)

Když uživatel klikne na odkaz, tak URL adresa vypadá pěkně, ale nezobrazí požadovaný obsah. Tím myslím, že když přejdu na adresu třeba /#contact, stránka se na tuto sekci při načtení neposune. Zkus buď upravit ten javascript (tohle by mělo fungovat: http://www.itnetwork.cz/dev-lighter/268) nebo přidej k těm podnadpisům stejné id, pak stránku bude posouvat přímo prohlížeč.

BTW, Angel Beats bych sakra hodně zvýraznil, nejlépe ještě s odkazem na akihabaru ;) :D

Editováno 22.12.2013 2:00
 
Nahoru Odpovědět
22.12.2013 1:58
Avatar
Odpovídá na FastNode
Neaktivní uživatel:22.12.2013 8:01

Díky, zkusím to upravit. Obsah zpráva měl být obsah zprávy. :D Díky, nevšiml jsem si.
S media pracuju poprvé, takže až se to naučím lépe, zřejmě to i nějak splácám do sebe.
Javascript už by měl být přepsaný a funguje, takže díky. :) Spíš jsem chtěl ten plynulý JS efekt.

Ó, někdo kdo zná Angel Beats? :D

Nahoru Odpovědět
22.12.2013 8:01
Neaktivní uživatelský účet
Avatar
vajgl97
Člen
Avatar
vajgl97:5.1.2014 18:06

vzhled se mi opravdu líbí. :)

Nahoru Odpovědět
5.1.2014 18:06
Kdo není líný nezná limit.
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 11 zpráv z 11.