zoul
commented
1 month ago Supr! ❤️ Moje postřehy:
Ta sekce Kontaktní informace mně teď přijde trochu divná, když je v ní jediná věc. Je to tak aktuálně na produkci, tak nevím, proč mě to najednou trklo tady :) Nechceme ten kontaktní mail přesunout do sekce Základní informace?
Tohle ve mně triggeruje nějaké spider senses:
const [pendingChangesBio, setPendingChangesBio] = useState(false);
const [pendingChangesName, setPendingChangesName] = useState(false);
const canSubmitBio = pendingChangesBio && !updating;
const canSubmitName = pendingChangesName && !updating;Konkrétně je mně podezřelá ta duplikace canSubmitFoo, canSubmitBar. Klidně bychom to mohli takhle nechat, ale jak se ta duplikace množí, je to myslím dobrá známka toho, že bychom měli nějak zobecnit a extrahovat kus kódu, abychom se furt neopakovali ručně. Tady by se velmi nabízelo extrahovat komponentu „vstupní textové pole s ukládacím čudlíkem“? Mimo jiné bychom tím sjednotili i vertikální mezery mezi inputem a čudlíkem, které teď hodně lítají.
Chceš se nad tím zamyslet? Nedělal bych z toho zatím úplně obecnou komponentu do components/, ale nechal bych ju v rámci tohoto souboru. Pokud na to teď nemáš kapacitu nebo se ti do toho nechce, můžem mergnout a udělám já ex post. Nebo to můžu přihodit jako commit sem a mrkneš se, jak by to fungovalo.
laurabaluchova
Nechala som inputy s tlačítkom na uloženie, myslím že keď sa to spojí s Toast notifikacemi, bude to spolu dobre fungovať. Robila som aj prieskum ako to je inde, zmeny sa bežne ukladajú osobitne, je to o dosť iný koncept ako registračný formulár kde všetko vyplním a odošlem
Tlačítka som pretextovala nech je jasné čo ukladám, uloženie bio som dala ako Uložit text, nenapadlo mi tam veľmi niečo iné vhodné.. neviem či by bolo Uložit bio jasnejšie či naopak menej
Pridala som sekciu Práce, mne to takto zoskupené dáva pekne zmysel, ale zaujíma ma názor iných očí, keď tak viem prerobiť