Skip to content

fix(sidePanel): reset dirty stav už v before fázi odeslání panel form…#62

Merged
thorewi merged 1 commit into
mainfrom
fix/sidepanel-dirty-close-confirm
Jun 30, 2026
Merged

fix(sidePanel): reset dirty stav už v before fázi odeslání panel form…#62
thorewi merged 1 commit into
mainfrom
fix/sidepanel-dirty-close-confirm

Conversation

@tkorcina

Copy link
Copy Markdown
Contributor

…uláře

Potvrzovací dialog "Opravdu zavřít bez uložení?" vyskakoval i po úspěšném odeslání formuláře v side panelu (např. export se stažením souboru nebo přes background queue), protože dirty stav se resetoval až ve "success" fázi sidePanelDirty – jenže submitForm.success stihne panel zavřít dřív (zavolá downloadFile → klik na .btn-close → tryClose) a navíc odpověď nemusí vracet snippet panelu (JSON se souborem / redirect).

Reset proto děláme už v "before" fázi pro formuláře uvnitř panelu; při validační chybě (hasErrors) ho v success vrátíme zpět.

…uláře

Potvrzovací dialog "Opravdu zavřít bez uložení?" vyskakoval i po úspěšném
odeslání formuláře v side panelu (např. export se stažením souboru nebo
přes background queue), protože dirty stav se resetoval až ve "success"
fázi sidePanelDirty – jenže submitForm.success stihne panel zavřít dřív
(zavolá downloadFile → klik na .btn-close → tryClose) a navíc odpověď
nemusí vracet snippet panelu (JSON se souborem / redirect).

Reset proto děláme už v "before" fázi pro formuláře uvnitř panelu;
při validační chybě (hasErrors) ho v success vrátíme zpět.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@tkorcina tkorcina requested a review from thorewi June 30, 2026 14:26
@tkorcina tkorcina self-assigned this Jun 30, 2026
@thorewi thorewi merged commit 5ebd9c3 into main Jun 30, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants