-
-
Notifications
You must be signed in to change notification settings - Fork 299
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
Remove JavaScript fireEvent() #431
Labels
Comments
asankov
added a commit
to asankov/Kiwi
that referenced
this issue
Feb 16, 2020
because it is no longer used. Refs kiwitcms#431
asankov
added a commit
to asankov/Kiwi
that referenced
this issue
Feb 18, 2020
because it is no longer used. Refs kiwitcms#431
atodorov
pushed a commit
that referenced
this issue
Feb 18, 2020
because it is no longer used. Refs #431
asankov
pushed a commit
to asankov/Kiwi
that referenced
this issue
Dec 2, 2020
- after v8.8 these are now unused - Closes kiwitcms#212 b/c there are no more legacy `jQ` references - Closes kiwitcms#1382 b/c there are no more legacy `jQ.bind` references - Closes kiwitcms#431 b/c there are no more references to `fireEvent()`
asankov
pushed a commit
to asankov/Kiwi
that referenced
this issue
Dec 2, 2020
- after v8.8 these are now unused - Closes kiwitcms#212 b/c there are no more legacy `jQ` references - Closes kiwitcms#1382 b/c there are no more legacy `jQ.bind` references - Closes kiwitcms#431 b/c there are no more references to `fireEvent()`
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
From what I've seen so far this is like an intermediate layer which isn't useful. For example click on Change priority menu item -> sets a hidden form field, then calls fireEvent() to trigger the onChange event handler.
The handling function can be executed directly when clicking the UI element.
The text was updated successfully, but these errors were encountered: