nikihlawiczka / final_bodynamic Goto Github PK

• prvni UX - ty jo nez jsem pochopil, ze to jsou 3 ruzne casti webu (osobni, skupinove, poradenstvi), tak mi to chvilku trvalo. Imho na to mel byt nejakej kus velkeho nadpisu, breadcrumbs nebo jednotne menu... proste neco, toto je imho hodne matouci
• telefon v hlavicce by mel byt odkaz, adresa v hlavicce by taky mohla byt link na mapy nebo do kontaktu
• navzdory slozitejsi strukture musim pochvalit, ze to mas hezky rozhazene v templatach

• Final_Bodynamic/personal_training/personal_main.html

  Line 31 in e79089a

  <a class="a_link" href="/group_workout/group_program/"

  toto by validator nemel pustit

• Final_Bodynamic/personal_training/personal_contact.html

  Line 66 in e79089a

  <img class="tab_photo" src="/images/map.jpg" alt="Mapa Bodycentrum" />

  tady je prostor pro vlozenou mapu nebo aspon odkaz do mapy.cz/google
• treneri v osobnich treninzich by sli hezky generovat pres eleventy . neco takoveho https://piccalil.li/quick-tip/create-json-feed-eleventy/

• Final_Bodynamic/group_workout/group_price.html

  Line 29 in e79089a

  <tr class="envelope_row_c">

  toto by slo dobre resit pres odd-even pseudo selectory https://www.w3schools.com/cssref/sel_nth-child.asp

• Final_Bodynamic/group_workout/group_price.html

  Line 25 in e79089a

  <th colspan="2" class="envelope_main">

  jak jsme si rikali - myslim, ze by ten nadpis by mel byt mimo tabulku a hX

• Final_Bodynamic/group_workout/group_program.html

  Line 28 in e79089a

  <a class="a_link" href="/images/program.jpg" download="program.jpg"

  tady by se siknul komentar proc to tam je. Pro budouci vyvojare/ky
• toto je divny

  Final_Bodynamic/food/food_main.html

  Line 62 in e79089a

  <h4 class="tab_main">První konzultace</h4>

  . mas tam .tab_content a pod tim .tab__main . pritom content ridi _food_main.scss a main jde z contact. Je v tom proste gulas
• moc velka skoda, ze sis nevyzkousela v tom scss ty & operatory, moc by ti to pomohlo v prehlednosti scss soubru
• myslim, ze na pouziti BEMu by ses mela jeste mrknout. Ta semantika "kde a jak to pouzivat" je tak na pul
• chvalim rozbiti scss do souboru

Obecne se mi ten projekt libi - snazila ses najit si svoji cestu a napad, je to plne obsahu. Trosku to kazi to UX. Technicky to neni spatne, jen si myslim, ze SASS i BEM jsi pouzila spise papirove nez ze bys zkusila vyuzit maximalne jeho vyznam (nez bys to chtela nekam posilat, tak to by se melo zlepsit). Ale myslim, ze mas na to se posunout dale a ve vysledku zmenit karieru.