-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
Add Shiny Event classes for custom events #3815
base: main
Are you sure you want to change the base?
Changes from all commits
01fd8fa
90c0523
8d12ae7
0e2d040
e2d19b9
60f074d
54fa857
63427a7
651b768
922d16d
2740be1
b5c6561
a66ef6d
08ced6b
4f50796
ce4961f
0adeb4b
4648423
eddb0d9
064da3a
10c2398
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Large diffs are not rendered by default.
Large diffs are not rendered by default.
Large diffs are not rendered by default.
Large diffs are not rendered by default.
This file was deleted.
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -54,3 +54,5 @@ declare global { | |
): this; | ||
} | ||
} | ||
|
||
export type { EvtFn }; | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Assuming this stands for "Event Function"? If so Is there a reason to keep it so abbreviated? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I think mostly because it was reused often in this file and because |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See comments below on this. I'd imagine we can unset this again. Might require some assertions but that's probably safer anyways.