Closed Bug 978118 Opened 10 years ago Closed 10 years ago

Thimble needs a "Run" or manual preview button

Categories

(Webmaker Graveyard :: Thimble, defect)

defect
Not set
normal

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
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 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)
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.

Attachment

General

Creator:
Created:
Updated:
Size: