Skip to content

Conversation

@chenglou
Copy link
Member

RFC. Feedback welcome. cc @bobzhang

Huge quality improvement for the online experience. This activates
color (needed, otherwise it's a wall of white text). I use a small lib
to parse the raw ANSI color escape chars into html styles. It'd not look
good in the BS playground currently.

This doesn't activate Reason syntax for outcome printer, since it's
compiling ocaml.

RFC. Feedback welcome. cc @bobzhang

Huge quality improvement for the online experience. This activates
color (needed, otherwise it's a wall of white text). I use a small lib
to parse the raw ANSI color escape chars into html styles. It'd not look
good in the BS playground currently.

This doesn't activate Reason syntax for outcome printer, since it's
compiling ocaml.
@chenglou
Copy link
Member Author

Before:
screenshot 2018-03-30 16 55 04

After:
screenshot 2018-03-30 16 54 29

@bsansouci
Copy link
Contributor

Could we make it such that Line 1, 20 is clickable and sets the cursor at that position?

@chenglou
Copy link
Member Author

That’ll be done on the playground side

@bobzhang
Copy link
Member

bobzhang commented Apr 2, 2018

can we make it configurable so that it would be less intrusive?

chenglou added 2 commits April 2, 2018 18:26
jsoo doesn't support variadic functions, so instead of exposing an
optional toggle on the function, I'm exposing two new functions
@chenglou
Copy link
Member Author

chenglou commented Apr 3, 2018

Done. jsoo doesn't support variadic functions, so instead of exposing an optional toggle on the function, I'm exposing two new functions

@bobzhang
Copy link
Member

bobzhang commented Apr 4, 2018

lgtm

@chenglou chenglou merged commit d2e3864 into rescript-lang:master Apr 5, 2018
@chenglou chenglou deleted the se branch April 5, 2018 03:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants