Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

chyby při práci s modálními okny #95

Closed
rodlukas opened this issue Feb 4, 2020 · 3 comments
Closed

chyby při práci s modálními okny #95

rodlukas opened this issue Feb 4, 2020 · 3 comments
Assignees
@rodlukas
Labels
🐜 bug něco nefunguje

Comments

@rodlukas
Copy link
Owner

rodlukas commented Feb 4, 2020 •
edited
Loading

  1. reactstrap >= 8.3.0 (konkretne reactstrap/reactstrap@310b061) zpusobi, ze po ulozeni formulare v modalnim okne zustane na body class pro otevrene okno, tedy nefunguje scrollovani v ramci stranky
  2. pri aktualizaci skupiny, kdy chceme pridat noveho klienta se vsechny formulare po jeho pridani zavrou, je sice pridany ale skupina se samozrejme vubec nevytvori
@rodlukas rodlukas added the 🐜 bug něco nefunguje label Feb 4, 2020
@rodlukas rodlukas self-assigned this Feb 4, 2020
rodlukas added a commit that referenced this issue Feb 4, 2020
@rodlukas
docasna oprava problemu s reactstrap #95
5f88ad1 
navrat na verzi, kde vse fungovalo
rodlukas added a commit that referenced this issue Feb 4, 2020
@rodlukas
docasna oprava problemu s reactstrap #95
8d7f065 
navrat na verzi, kde vse fungovalo
rodlukas added a commit that referenced this issue Feb 4, 2020
@rodlukas
docasna oprava problemu s reactstrap #95
917032c 
navrat na verzi, kde vse fungovalo
@rodlukas rodlukas changed the title chyba pri kladani v modalnich oknech chyba pri ukladani v modalnich oknech Feb 5, 2020
@rodlukas rodlukas changed the title chyba pri ukladani v modalnich oknech chyby při práci s modálními okny Feb 5, 2020
@rodlukas
Copy link
Owner Author

rodlukas commented Feb 5, 2020

  1. zpusobuje toto:

    UP-admin/frontend/src/forms/FormClients.js

    Line 71 in 341ef1a

    this.props.groupsActiveContext.funcHardRefresh()

@rodlukas
Copy link
Owner Author

rodlukas commented Feb 5, 2020

  • Toto zpusobuje fakt, ze po ulozeni dat z formulare krome zavirani formulare (setModal) zaroven obvykle refreshujem nadrazenou komponentu.
  • Modalu nastavime isOpen na false, ale mezitim uz probihaji asynchronne take refreshe, kvuli kterym se cely modal unmountne z DOMu.
  • A vzhledem k tomu, ze nezvladl korektne dokoncit svou operaci zavreni (onClosed), ale uz se zacina hned zpracovavat jeho metoda componentWillUnmount, tak tato metoda uz modalni okno uplne nedoodstrani (predevsim modal-open atribut na body), protoze by ho odstranila v pripade, ze this.props.isOpen === true, coz jsme ale pomoci setModal nastavili na false.
  • V predchozich verzich vse fungovalo, protoze this.state.isOpen === true vskutku platilo diky tomu, ze se zmeny modalu nestihly zpropagovat z props do stavu (v metode onClosed) a tedy ackoliv neprobehla korektne onClosed, zachranila to alespon componentWillUnmount.
  • Fakt, ze onClosed neprobehne korektne take zpusobi to, ze formular neni zavreny prechodovym efektem, ale okamzikem unmountu zmici kompletne.
  • Pro kompletni opravu je tedy potreba zmenit zpusob prace s modaly napric celou aplikaci.

rodlukas added a commit that referenced this issue Feb 7, 2020
@rodlukas
[js+testy] opravy chyb pri praci s modalnimi okny (nektere viz #95)
7f3ac92 
- vystiznejsi nazvy komponent ModalLecturesCore.js, ModalLecturesWizard.js
- oprava zbytecneho druheho pozadavku na stazeni cerstvych aktivnich skupin/klientu po aktualizaci, ackoliv uz jednou pozadavek probehl (pokud uzivatel byl zrovna v zalozce aktivnich skupin/klientu)
- pri uprave/pridani skupiny/klienta dojde automaticky k prepnuti na zalozku aktivni/neaktivni podle toho, do ceho jsme pridavali
- oprava chyby, kdy pri smazani klienta/skupiny z karty doslo k presmerovani na stranku Nenalezeno a nahlaseni chyby, nyni dojde k presmerovani na seznam skupin/klientu
- oprava chyby, kdy pri aktualizaci skupiny, kdyz chceme pridat noveho klienta se vsechny formulare po jeho pridani zavrou, je sice pridany ale skupina se samozrejme vubec nevytvorila #95
- oprava chyby, ktera se objevila pri pouziti reactstrap >= 8.3.0 #95: po ulozeni formu v modal okne zustal na body class atribut znacici otevrene okno, kvuli tomu se nedalo v aplikci posouvat (overflow: hidden), soucasti opravy chyby je i oprava animace modalniho okna pri ulozeni, ktera se z duvodu prilis brzkeho unmountu modalu nezobrazovala
- rozsireni UI testu pro zachyceni chyb se spatne zavrenym modalnim oknem
- oprava chyby, kdy pri pridavani/uprave skupiny/zajemce se pridal novy klient a formular nebyl nastaven na dirty, tedy pri zavreni neupozornil na neulozene zmeny
- DRY: refresh contextu nemaji uz na starosti komponenty vyuzivajici Modal komponenty, ale primo Modal komponenty
- aktualizace na reactstrap 8.4.1, ktery pred opravou nekterych techto chyb zpusobil castecne nefunkcni aplikaci
@rodlukas
Copy link
Owner Author

rodlukas commented Feb 7, 2020

vse opraveno, viz description commitu vyse

@rodlukas rodlukas closed this as completed Feb 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@rodlukas rodlukas

Labels
🐜 bug něco nefunguje
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@rodlukas