Closed
Bug 978118
Opened 10 years ago
Closed 10 years ago
Thimble needs a "Run" or manual preview button
Categories
(Webmaker Graveyard :: Thimble, defect)
Webmaker Graveyard
Thimble
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: gerv, Assigned: michiel)
References
Details
Attachments
(1 file)
If you are writing JS code in Thimble, the error console will (ideally) show you any errors in it when you run it. Except that it will also show you lots of syntax errors caused by the fact that the Thimble preview keeps trying to run your code as you are writing it! I haven't tried it, but I'm willing to bet this would be really confusing for users, and make it much harder to use Thimble to teach JS. Can we please have a "manual preview" or "Run" mode, which you can switch on, such that the user has to manually click "Preview" or "Run" in order for the code to be copied to the preview and run? This could be switched on by users when writing JS. Gerv
Comment 1•10 years ago
|
||
Neat idea. I think in the simple case just having a checkbox in the "Preview" pane that is default on, and can be turned off to stop the updates, would be pretty simple.
Assignee: nobody → pomax
Status: NEW → ASSIGNED
this adds a "reload" button to force reload the preview based on the sourcecode in the editor, without needing to type something in the editor to force a reparse. It also adds the "run/do not run JS" checkbox so that you can turn off automatically running any and all code inside <script> tags.
Attachment #8386458 -
Flags: review?(scott)
Comment 4•10 years ago
|
||
Comment on attachment 8386458 [details] [review] https://github.com/mozilla/thimble.webmaker.org/pull/381 Some pretty trivial comments, easy fixes though.
Attachment #8386458 -
Flags: review?(scott) → review-
Comment on attachment 8386458 [details] [review] https://github.com/mozilla/thimble.webmaker.org/pull/381 fixed the .overwrite issue. Also keeping it "reload" because unlike jsbin/fiddle we *alway* run, just either with or without JS interpreted.
Attachment #8386458 -
Flags: review- → review?
Attachment #8386458 -
Flags: review? → review?(scott)
Updated•10 years ago
|
Attachment #8386458 -
Flags: review?(scott) → review+
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•