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
Marián Ligocký:2.1.2015 12:00

Ahoj, práve som dokončil moje portfólio a bol by som rád keby ste mi ho zhodnotili. :)

Chcem ešte pripomenúť že logo ktoré je v hlavičke je iba dočasné, teda ešte ho budem meniť.

 
Odpovědět
2.1.2015 12:00
Avatar
Ondrca
Tvůrce
Avatar
Odpovídá na Marián Ligocký
Ondrca:2.1.2015 12:05

Na mě je úvodní stránka moc bílá, kdybys tam dal nějakou texturu zšedivělou, tak by to bylo lepší :)

Nahoru Odpovědět
2.1.2015 12:05
Zase jsem o něco chytřejší
Avatar
Odpovídá na Marián Ligocký
Neaktivní uživatel:2.1.2015 12:37
<br />

by se už používat nemělo, jinak pěkné :)

Editováno 2.1.2015 12:38
Nahoru Odpovědět
2.1.2015 12:37
Neaktivní uživatelský účet
Avatar
Tonda Kozák
Člen
Avatar
Tonda Kozák:2.1.2015 12:43

Není to v zásadě špatné, jen efektnost je na úkor efektivnosti.

  1. Úvodní stránka je na můj vkus až moc jednoduchá, zkoušel jsem to Fpětnout, jestli se mi náhodou nedonačtou styly.
  2. Skryté menu je sice ukázka toho, jak umíš používat animace, ale nemá to pro návštěvníka žádnou přidanou hodnotu.
  3. Na jednotlivé části stránky se nedá odkazovat, protože neměníš adresu.
  4. Text O mne je na tom obrázku špatně čitelný (čti nečitelný).
  5. Reference budou (hádám, nezkoušel jsem) na dotykových zařízeních nepoužitelné.
  6. Nemáš podporu pro starší IE, ani pro IE9.
  7. Nemáš favicon ico.
  8. Po slovensky veľmi neviem, ale jsem si jistý, že za čiarkou sa robí medzera.
  9. Pokud má ta šipka (třeba pod tlačítkem Referencie) něco dělat, tak mi nefunguje, jestli ne, tak je to matoucí.
  10. Kód je víceméně v pořádku, přizpůsobivost šířkám okna funkční.
 
Nahoru Odpovědět
2.1.2015 12:43
Avatar
Odpovídá na Neaktivní uživatel
Neaktivní uživatel:2.1.2015 12:57

Mělo by se používat tam, kde se hodí. Nehodí se tehdy, když bys pomocí <br> tagu pozicoval nějaký element ve stránce, pokud však potřebuješ zalomit text v odstavci, je to správně.

Editováno 2.1.2015 12:59
Nahoru Odpovědět
2.1.2015 12:57
Neaktivní uživatelský účet
Avatar
Odpovídá na Neaktivní uživatel
Neaktivní uživatel:2.1.2015 13:38

Naučený jsem jinak, ale budiž :)

Nahoru Odpovědět
2.1.2015 13:38
Neaktivní uživatelský účet
Avatar
Odpovídá na Tonda Kozák
Marián Ligocký:2.1.2015 16:51

Ďakujem za kritiku, trošku to teda pomením ale trošku som nepochopil bod 5 (nepoužiteľné referencie na mobile), mohol by si mi ho vysvetliť? :)

Trošku som nepochopil prečo som mal pred chvíľou hodnotenie 1/5? Nemyslím si že je to až také zlé...

 
Nahoru Odpovědět
2.1.2015 16:51
Avatar
Pavel Mareš
Tvůrce
Avatar
Odpovídá na Neaktivní uživatel
Pavel Mareš:2.1.2015 17:02

Jen z mojí zvědavosti, to vše odděluješ tagem p? Takže co odstavec to p? Jo to je fajn, ale je miliarda možností, kdy chceš zalomit text třeba v buňce a nechápu proč tam cpát zbytečné styly nebo tagy navíc. Prostě to zalomíš a konec. Jinak kde ses to naučil?

Nahoru Odpovědět
2.1.2015 17:02
Však ono půjde ...
Avatar
Tonda Kozák
Člen
Avatar
Odpovídá na Marián Ligocký
Tonda Kozák:2.1.2015 17:05

O tom hodnocení nevím. Já tady nehvězdičkuji. Být tebou bych se spíš zaměřil na konkrétně vypsané rady a na hvězdičky se vybodl. :-)

ad 5)
Pokud jsem na mobilu/tabletu, tak tam není hover, takže se nedostanu k té informaci, která se mi zobrazí, když přes to přejedu myší.

 
Nahoru Odpovědět
2.1.2015 17:05
Avatar
Odpovídá na Pavel Mareš
Neaktivní uživatel:2.1.2015 17:07

V textu to je ok, šlo mi o to

<br>

v formě dole.

Odnaučil jsem se to jelikož jsem byl vždy na to upozorňován, a i tady :)

A nechápu jak se s

<br>

dá pozicovát :D

Nahoru Odpovědět
2.1.2015 17:07
Neaktivní uživatelský účet
Avatar
TomasGlawaty
Člen
Avatar
TomasGlawaty:2.1.2015 19:08

Ještě dodám, že když zmenším okno prohlížeče, tak mi ty reference přetečou do kontaktu :)

 
Nahoru Odpovědět
2.1.2015 19:08
Avatar
Odpovídá na Neaktivní uživatel
Neaktivní uživatel:3.1.2015 13:47

Ten form je právě špatně, protože díky <br> pozicuješ elementy ve stránce. Tys to napsal tak, že to vypadalo, že se to nemá používat nikdy a nikde. S tím jsem nesouhlasil. :)

Nahoru Odpovědět
3.1.2015 13:47
Neaktivní uživatelský účet
Avatar
Odpovídá na Neaktivní uživatel
Neaktivní uživatel:3.1.2015 14:11

Je to možné, protože v textu jsem viděl pčka a jen proscrolloval, tak jsem si ho tam ani nevšiml :D

Nahoru Odpovědět
3.1.2015 14:11
Neaktivní uživatelský účet
Avatar
Crrack
Člen
Avatar
Crrack:3.1.2015 18:00

Za mě to vypadá to hodně dobře :-)

 
Nahoru Odpovědět
3.1.2015 18:00
Avatar
Bl4ckout
Tvůrce
Avatar
Bl4ckout:4.1.2015 12:55

Nic bych neměnil... vypadá to dobře.

 
Nahoru Odpovědět
4.1.2015 12:55
Avatar
Odpovídá na Marián Ligocký
Neaktivní uživatel:4.1.2015 13:16

Ahoj,

  1. Nevidím důvod, proč scrollovat 2 prvky na úvodní stránce, to má smysl až u 3-4 prvků, můžeš to napsat bez efektu.
  2. Na titulní stránce mě zmátla šipka dolů, mělo by na ni jít kliknout, aby se stránka posunula.
  3. O mne - opět šipka
  4. Když scrolluji dolů, je dobré, aby se reference zobrazily hned potom, co je uvidím, a ne až po najetí myši.
  5. Placeholdery u polí ve formuláři by měly mít stejnou velikost fontu.

V kódu mi jen nesedí odsazení. Ještě si vyescapuj ten script tag co načítá recaptchu (druhý), validátor kvůli tomu řve (http://validator.w3.org/check?…).
Jinak dávám 4*, protože toto jsou vážně jen malé vady na kráse na jinak bezvadně nakódovaném a navrhnutém webu.

Nahoru Odpovědět
4.1.2015 13:16
Neaktivní uživatelský účet
Avatar
Marián Ligocký:4.1.2015 14:19

Ďakujem a kritiku, budem sa snažiť opraviť čo najviac vecí :)

 
Nahoru Odpovědět
4.1.2015 14:19
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 17 zpráv z 17.