-
Notifications
You must be signed in to change notification settings - Fork 4
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
Disable run button when running #38
Disable run button when running #38
Conversation
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.
I believe this is the correct way to use props in solid-js according to their docs.
src/frontend/src/app.tsx
Outdated
@@ -75,7 +78,7 @@ export default function () { | |||
</aside> | |||
</header> | |||
<section> | |||
<Editor runCode={runCode} /> | |||
<Editor runCode={runCode} running={running} /> |
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.
<Editor runCode={runCode} running={running} /> | |
<Editor runCode={runCode} running={running()} /> |
Ah I see what you mean now, thanks! |
thanks so much. |
Big fan of your project!
Noticed that you can spam click run and that'll run the code multiple times concurrently.
So I've opportunistically made a small change to disable the run button while pyodide worker is running.