Oprava ukol 1

[MaggieFoxx]

Ahoj, v prvom rade ma mrzí, že som sa k tomu nedostala skôr. Ja som si to spustila už minulý týždeň a videla, že je to dosť dobré. Chcela som si ale nechať viac času na kvalitnú spätnú väzbu a nie to iba v rýchlosti odkliknúť, že super hotovo.

Veľmi pekná práca, netreba určite nič opravovať, je to ďaleko za očakávaniami pre túto úlohu. (responzívne, dvojjazyčné, rôzne fonty písma... super!!) Píšem iba zopár vecí, ktoré mi napadli, pri prechádzaní kódu. Určite to ale nie sú chyby.

[MaggieFoxx]

Pre takúto jednuduchú úlohu, to nie je vôbec potrebné (ani možné) a nikto to od vás neočakával, toto píšem úplne zo zaujímavosti pretože si si vyžiadala podrobnejšie spätnú väzbu. :smile: :smile:

Keby niekedy v budúcnosti robíš niečo väčšie (viacstránkové, reaktívne atď.) zvykne sa používať javascript a framework-y postavené na ňom, nie čisté HTML. To možno vieš, chcem iba poukázať na jednu vec. Jeden z hlavných dôvodov prečo sa to tak robí je práve znovupoužiteľnosť - raz si napíšem napríklad header/footer (alebo aj iné opakujúce sa komponenty) a potom to už iba použijem, nepotrebujem to opakovane písať stále dokola, tak ako teraz tu. Ale je to naozaj už na väčšie projekty, na osobné stránky a podobné menšie veci by to bolo zbytočná komplikácia.

[MaggieFoxx]

kontakt - znovu veľmi pokročilá vec a je to viac frontend poznámka (takže irelevantné do tohoto kurzu), poziciovanie pomocou tabuľky je efektívne a v podstate fajn, používalo sa to ako bežný postup v minulosti. Teraz sa používa skôr flexbox. Pozerám, že si ho použila v projekte, takže asi vieš. Dajú sa s tým oveľa jednoduchšie robiť pokročilé grafické veci. Ale zas, na dva obrázky a popisok k tomu nám stačí tabuľka, takže pekná práca.

[MaggieFoxx]

Ja určite nie som graficky veľmi nadaná a je to veľmi subjektívne, čo sa komu páči. Mne by sa tam ale hodil padding textu z ľava viac (alebo možno nejakému nadradenému kontaineru), proste viac centrovať text a netlačiť to nakraj. Hlavné miesto kam sa prirodzene pozeráme je stred obrazovky a v tvojom pripade je práve ten prázdny.

A tiež tým že je tam menej textu tak kľudne zväčšiť písmo a výšku riadkov (letter-spacing,line-height ), jednoduché a nebude to vyzerať tak prázdno :sweat_smile:

Vidím, že CSS zvládaš, ak by si sa chcela naučiť niečo nové, štýlovať sa dá aj pomocou bootstrap alebo tailwind. Keď sa to človek naučí, je to podstatne jednoduchšie, miesto vypisovania všetkých tried, môžeš použiť už dopredu zadefinované a máš menej práce.

[MaggieFoxx]

Ak by si niekomu chcela ukazovať aj kód nejakého tvojho projektu, tak potom je fajn použiť na formátovanie kódu prettier. Je to štandard, používa sa to práve na prehľadnosť a upratanie kódu. Neviem, či sa to na nejakom inom kurze spomínalo, ak nie, môžem ukázať na cvičení, stačí pripomenúť.

Pozerala som, že chceš stránky publikovať. Vieš už ako na to?? Je asi veľa spôsobov, na inom czechitas kurze sa používali github pages, môžeme sa o tom pobaviť keď to bude aktuálne a budeš chcieť.

[MaggieFoxx]

Veľmi pekná práca, ak je niečo nejasné, kľudne sa pýtaj. Schválne som to rozdelila do viacerých komentárov, aby si prípadne mohla na jednotlivé konkrétne veci reagovať. :nerd_face: :100: