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

Send + Sync Signals #1568

Closed
ealmloff opened this issue Oct 23, 2023 · 1 comment
Closed

Send + Sync Signals #1568

ealmloff opened this issue Oct 23, 2023 · 1 comment
Labels
desktop Suggestions related to the desktop renderer enhancement New feature or request signals Related to the signals crate

Comments

@ealmloff
Copy link
Member

Specific Demand

Signals can currently only be used in the thread they were created in. This works great within the virtual dom, but it makes using signals within threads or tokio futures difficult.

Implement Suggestion

We can add a generic to signals to make them automatically sync + send when you use them in a context like a new thread that requires them to be sync + send. Something like this playground
@ealmloff ealmloff added enhancement New feature or request desktop Suggestions related to the desktop renderer signals Related to the signals crate labels Oct 23, 2023
@ealmloff ealmloff mentioned this issue Dec 6, 2023
Closed
@jkelleyrtp
Copy link
Member

Solved by #1791

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
desktop Suggestions related to the desktop renderer enhancement New feature or request signals Related to the signals crate
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Make signals optionally Send+Sync ealmloff/dioxus
2 participants
@jkelleyrtp @ealmloff