In the Javascript interpreter, 1/0 does not return Infinity about klipse HOT 12 CLOSED

Nice catch @Lucas-C !

Did you find other inconsistencies?

On Sun, 2 Oct 2016 at 17:15 Lucas Cimon [email protected] wrote:

I used a very basic setup, and tested with Chrome:

1/0

<script> window.klipse_settings = {selector_eval_js: '.klipse'}; </script> <script src="/libs/js/klipse_plugin.min.js"></script>

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
#122, or mute the thread
https://github.com/notifications/unsubscribe-auth/AA6VPvXP9fxqJrZf48RDJTrA78qaDkTEks5qv7yVgaJpZM4KMBRT
.

from klipse.

@Lucas-C the root cause is that Infinity is not part of JSON and currently KLIPSE uses JSON.stringify to stringify the result of eval.

I found a library that pretty print javascript without using JSON.stringify:
https://github.com/cvadillo/js-object-pretty-print

Integrating this lib into KLIPSE will solve the issue.

from klipse.

Thank you !

from klipse.

This issue is going to be fixed by #149
By the way the same problem occured with ES6 Symbols

from klipse.

It is fixed! Can you please double check @Lucas-C ?

from klipse.

The original bug seems fixed, but now there is a "klipse-container-0" string by default in the 2nd .CodeMirror block :

from klipse.

Any idea on where that is coming from ??

from klipse.

Yeah. It's the id of the container associated with the the snippet. Try to type

document.getElementById("klipse-container-0").innerHTML = 
 "<div style='color:blue;'> Hello <b>World</b>!</div>"

You can prevent it by setting data-async-code="true"

from klipse.

I could not make it work properly:

• data-async-code simply broke the code evaluation
• there is and HTML element with ID "klipse-container-0" but it is not the one that contains the text I see

Hence I decided to revert using v0.7.0 : https://lucas-c.github.io/progWorkshop-bootstrapPage/workshop_material/challenges/3.html

from klipse.

I see. Indeed it's a bug inside KLIPSE.
Could you open a separate issue please?
(Très sympa ton site!!!)

from klipse.

from klipse.

Thanks
(merci!)

from klipse.