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.

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

Oprava inicializace formulářů #57

Merged
merged 2 commits into from
Sep 5, 2024
Merged

Oprava inicializace formulářů #57

merged 2 commits into from
Sep 5, 2024

Conversation

zipper
Copy link
Contributor

@zipper zipper commented Sep 4, 2024

Původní metodu Nette.initForm chceme volat vždy. Sama o sobě obsahuje kontrolu na opětovnou inicializaci, na začátku ale ještě spouští nastavení, které je nutné udělat i po ajaxu (jmenovitě inicializace toggle elementů). Metoda Nette.toggleControl je pak v Nette ošetřená proti vícenásobnému navázání handleru pomocí weakMap.

@zipper
Copy link
Contributor Author

zipper commented Sep 4, 2024

Děkuji za approve, nicméně to bohužel jednu chybu opravuje a druhou zase přidává, takže merge se odkládá :trollface: Ale cením si toho, kluci! ❤️

… volat vždy. Sama o sobě obsahuje kontrolu na opětovnou inicializaci, na začátku ale ještě spouští nastavení, které je nutné udělat i po ajaxu (jmenovitě inicializace toggle elementů). Metoda `Nette.toggleControl` je pak v Nette ošetřená proti vícenásobnému navázání handleru pomocí weakMap.
@zipper zipper merged commit db69596 into master Sep 5, 2024
3 checks passed
@zipper zipper deleted the initForm branch September 5, 2024 08:55
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.

5 participants