Avatar
Inoue Yūki
Redaktor
Avatar
Inoue Yūki:

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

Odpovědět 16.12.2013 14:22
Avatar
Honza Bittner
Redaktor
Avatar
Odpovídá na Inoue Yūki
Honza Bittner:

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
Ptejte se mě na cokoli na https://github.com/HoBi/ama a followujte mě na Twitteru https://twitter.com/tenhobi. :-)
Avatar
Jan Vargovský
Redaktor
Avatar
Odpovídá na Inoue Yūki
Jan Vargovský:

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
Inoue Yūki
Redaktor
Avatar
Odpovídá na Jan Vargovský
Inoue Yūki:

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
Avatar
Jan Vargovský
Redaktor
Avatar
Odpovídá na Inoue Yūki
Jan Vargovský:

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
Inoue Yūki
Redaktor
Avatar
Odpovídá na Honza Bittner
Inoue Yūki:
  • 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
Avatar
Honza Bittner
Redaktor
Avatar
Odpovídá na Inoue Yūki
Honza Bittner:

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

Nahoru Odpovědět 16.12.2013 15:20
Ptejte se mě na cokoli na https://github.com/HoBi/ama a followujte mě na Twitteru https://twitter.com/tenhobi. :-)
Avatar
FastNode
Redaktor
Avatar
FastNode:

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
Inoue Yūki
Redaktor
Avatar
Odpovídá na FastNode
Inoue Yūki:

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
Avatar
vajgl97
Člen
Avatar
vajgl97:

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.