Kritika webu https//www.lukaskrbec.com

Naučíme tě pracovat na home-office.

Zjistit více...

Jak se ti líbí web?

Ještě nikdo nehodnotil, buď první!

Přejít na web:
http://https//www.lukaskrbec.com

Aktivity

Lukáš Krbec

Člen

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

Lukáš Krbec

Člen

Lukáš Krbec:18.5.2021 19:05

www.lukaskrbec.com

Nahoru Odpovědět

18.5.2021 19:05

Veil

Člen

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

Lukáš Krbec

Člen

Lukáš Krbec:18.5.2021 20:28

vyjizdi zprava? nebo jiny problem?

Nahoru Odpovědět

18.5.2021 20:28

Veil

Člen

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

Lukáš Krbec

Člen

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

Veil

Člen

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

Ondřej Šrytr

Člen

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

Lukáš Krbec

Člen

Lukáš Krbec:19.5.2021 21:54

Diky, opravim

Nahoru Odpovědět

19.5.2021 21:54

Vakos

Tvůrce

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...

Lukáš Krbec

Člen

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.