NOVINKA - Online rekvalifikační kurz Python programátor. Oblíbená a studenty ověřená rekvalifikace - nyní i online.
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
Lukáš Krbec
Člen
Avatar
Lukáš Krbec:18.5.2021 18:48

Ahojte, budu rád za kritiku k mému portfoliu. Právě začínám kurz web developmentu a jedná se o web pro školní projekty. Díky Lukáš

 
Odpovědět
18.5.2021 18:48
Avatar
Lukáš Krbec
Člen
Avatar
Odpovídá na Lukáš Krbec
Lukáš Krbec:18.5.2021 19:05

www.lukaskrbec.com

 
Nahoru Odpovědět
18.5.2021 19:05
Avatar
Veil
Člen
Avatar
Veil:18.5.2021 20:27

Ahoj, zkoušel jsi ty stránky zobrazit na mobilu? Postranní menu je takové divné

 
Nahoru Odpovědět
18.5.2021 20:27
Avatar
Lukáš Krbec
Člen
Avatar
Odpovídá na Veil
Lukáš Krbec:18.5.2021 20:28

vyjizdi zprava? nebo jiny problem?

 
Nahoru Odpovědět
18.5.2021 20:28
Avatar
Veil
Člen
Avatar
Odpovídá na Lukáš Krbec
Veil:18.5.2021 20:31

Zobrazuje se

Ab
out

S
kills - koho chceš zabít? :-)

Pro
jects

atd...

 
Nahoru Odpovědět
18.5.2021 20:31
Avatar
Lukáš Krbec
Člen
Avatar
Odpovídá na Veil
Lukáš Krbec:18.5.2021 20:33

jo tak to je zamer :) i pro desktop to tak mam

 
Nahoru Odpovědět
18.5.2021 20:33
Avatar
Veil
Člen
Avatar
Veil:18.5.2021 20:34

Tak to pak jo, na tom mobilu to vypadalo jako když se to tam nevešlo :-)

 
Nahoru Odpovědět
18.5.2021 20:34
Avatar
Ondřej Šrytr
Člen
Avatar
Ondřej Šrytr:19.5.2021 15:03

Ahoj,

co jsem to tak narychlo prošel, první věc, že script.js by měl být před </body>, nikoliv za. Další, že pokud na mobilu otevřu menu, zbytek stránky není nijak ztmavený, takže jsem měl tendenci hledat křížek na zavření menu. :)

 
Nahoru Odpovědět
19.5.2021 15:03
Avatar
Lukáš Krbec
Člen
Avatar
Odpovídá na Ondřej Šrytr
Lukáš Krbec:19.5.2021 21:54

Diky, opravim

 
Nahoru Odpovědět
19.5.2021 21:54
Avatar
Vakos
Tvůrce
Avatar
Vakos:20.5.2021 12:27

Ahoj, na první pohled web vypadá zajímavě, dost věcí bych udělal popravdě jinak.

Design

  • Menu - za mě divný odskok, blbě se to čte. Není to ani moc konzistentní vzhledem k ostatním tlačítkům, kde vždy v červeném obdélníku máš text bílou barvou. V mobilní verzi při otevření menu chybí tlačítko pro zavření menu.
  • Srdíčko - Zajímavá věc, ale zpracování mě přijde spíš jako, podivte se, že umím implementovat carusell, kde se točí nějaké fotky ...
  • Skils - Natáčení elementu s jeho obsahem mi taky nepřijde moc super. Když už, tak bych asi tam dal menší úhel natočení.
  • Slider - Tlačítka pro posun bych dal asi doprostřed, ale zas proč je nemít dole no ...
  • Formulář - Přidal bych Popisky k jednotlivým inputům a asi i zmenšil mezery mezi inputem a textboxem

Design je subjektivní a jsou to věci, které bych udělal jinak, ale třeba tak jak jsou, tak mohou být správně.

Kód

  • styly - v hlavičce vkládáš 2 css soubory, v all je fontawesome a ve style jsou tvoje styly. Je lepší první vkládat cizí a pak ty své, abys případně mohl nějakou vlastnost přepsat
  • Nevím jestli je úplně správné tvé zpracování CZ a EN verze. Je to asi jednoduché, ale nevím jak je to z hlediska SEO.
  • Menu - V menu pro každou část, která má být nějak na řádku, tak vytváříš vlastní třídu. Je to zbytečné, jde to udělat jednou. S použitím třídy new-line, která bude nastavená na display block a tak to budeš mít na novém řádku, tak jak potřebuješ.
pr<span class="new-line">jects</span>
  • Výčet u skills - nedělat to pomocí h2, to je pouze na nadpisy. Udělat to normálně pomocí seznamu, od toho jsou
  • Script hodit do body - to už ti psali

Kód jsem tak proběhl, třeba tam bude ještě něco, ale toto bych řekl, že jsou takové nejviditelnější věci.

Doufám, že ti můj náhled pomůže. Hlavně u designu je to hodně subjektivní a třeba s některýma věcma nemusíš souhlasit, což je v pořádku.

Nahoru Odpovědět
20.5.2021 12:27
"Jediný způsob, jak dělat skvělou práci, je milovat to, co děláte. Pokud jste to ještě nenašli, hledejte dál. Ne...
Avatar
Lukáš Krbec
Člen
Avatar
Odpovídá na Vakos
Lukáš Krbec:20.5.2021 22:06

Diky, za vecne pripominky a postrehy, s vetsinou zpetne souhlasim :)

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