jc-orozco / blockside Goto Github PK

There's a block for creating a class, but no way to add a constructor for the class.

If you try to add it manually, like this:
class Name {
constructor(name) {
}
}

It creates a 'method constructor' block and round trips to:
class Name{
methodconstructor(name){
}
}

Be able to preserve comments on the js to blocks conversion and back.

Expressions must also be allowed as property name:

var expr = 'foo';

var obj = {
  baz: 'bar',
  set [expr](v) { this.baz = v; }
};

Creating a map with quoted elements like the following

var i={
'foo': 'baz'
}

generates the attached blocks.

Neil Fraser commented on a Blockly Google Group thread that this did not work:
for (var i = 0, j = 0; i < 10; i++) {
}

or

for (var i = 0; i < 10; i++, j++) {
}

https://groups.google.com/forum/#!searchin/blockly/blockly$20round$20tripping%7Csort:relevance/blockly/R3mkfYkVQQM/lU4AolOEBwAJ

There are some cases were the system gets blocked and no more conversion from js to blocks is generated.
TODO: Search for a specific and simple case were this happens

Neil Fraser commented on a Blockly Google Group thread that this did not work:
var i;
for (i = 0; i < 5; i++) {
}

Don't forget that thanks to variable hoisting, this is equally valid JS:
for (i = 0; i < 5; i++) {
}
var i;

Or for that matter, so is dropping the var statement completely, creating a global.

Also, even if variable definitions are inside the init part, the demo still fails with this:

for (var i = 0, j = 0; i < 10; i++) {
}

https://groups.google.com/forum/#!searchin/blockly/blockly$20round$20tripping%7Csort:relevance/blockly/R3mkfYkVQQM/lU4AolOEBwAJ

var o = {
get property() {return 4},
set property(value) {alert(value)}
};

How did you do Resizable Workspace?

void literal should be preserved and not get converted to an empty string

Tested on version 0.3.3
The simplest form to reproduce this problem is typing numbers from 1 to 11 (each on one line). Up until 10 the number blocks get created in order but when typing 11 the first block is 6:
1
2
3
...
10
11

var i = 0;
window.alert(i++);
window.alert(++i);

This does not work:
var f = () => true

This does not work:
var str = 'block'
if(str === 'block'){
}

Neil Fraser pointed out on a Blockly google group thread that this was not working:
var s = 'abc' + 'def'

https://groups.google.com/forum/#!searchin/blockly/blockly$20round$20tripping%7Csort:relevance/blockly/R3mkfYkVQQM/lU4AolOEBwAJ

try {
} catch (e) {
}

arr = arr || []

function x() {
var a = 1, b = 1;
}

Maybe avoid translating on each character input when a js file is larger than a certain threshold, or use some heuristics to change the code to blocks translation on large code files.

This does not work:
(function c(msg){console.log(msg)})("Hello")

This works (fixed bug #9):
(function(msg){console.log(msg)})("Hello")

This does not convert to valid blocks: (function (msg){console.log(msg)})("hello")

The regular expression is quoted here:

foo.split(/\d/);

This fails to render anything.

'a1b2c'.split(/\d/);