KatTrnk
commented
1 year ago Děkuju za review :-)
Jsem v tomhle nemožná, takže jsem si nějakých zaoblených rohů vůbec nevšimla, ale už jsem to upravila. Co se týče té jiné barvy pozadí, tak to jsem taky uplný niemand (při googlení jak na ten gradient, tak jsem si tam dala jen nějaké ukázkové barvy a jak byly plus mínus podobné těm z předlohy, tak jsem je pak uplně zapomněla změnit), nicméně už také opraveno. Padding jsem nastavovala, ale máš pravdu, že by to zasloužilo větší odsazení od stran.
BEM mi dával smysl jen u tlačítek, ale je asi pravda, že by to šlo dát na celé html. Co se týče toho stylování nadpisu až na konci, tak to je proto, že jsem si nejdřív udělala hrubou strukturu a až pak začala upravovat barvy, font apod. Jednou jsem se nějakého lektora na to ptala, zda to vadí, a ten mi řekl, že ne, nicméně chápu tvůj point a pro orientaci je to určitě lepší, aby to bylo nahoře.
Ahojky, koukám na na stránku a chybí ti nahoře border radius pro zaoblení horních rohů, zároveň bych nastavila margin na ten text o piškvorkach, ať to más dále od stran a dále od spodu od toho prvního buttonu. Myslím, že jsi použila špatné barvy na colorgradient na pozadí.
Když koukám na Html, tak super ty zakomentované poznámky pro lepší orientaci. Použila bych více "class" ulehčilo by ti to velmi to css potom. Zkus příště využít ten BEM je to fakt fajn na tu přehlednost, ale to je samozřejmě na tobě (koukám že BEM jsi použila jen u tlačítek, zkus to dát na celé html) :)
Možná bych ještě zkusila v CSS držet vizualně nějak tu stránku, h1 styluješ až na konci po tlačítkách, což je trošku matoucí. Jinak to vypadá hezky <3