Clarify JSX/React WhiteSpace rules

[facebook]

Clarify JSX/React WhiteSpace rules

[syranide]

facebook/react#480 (comment), there is also the full technical reasoning facebook/react#480, but it may be a bit hard to digest.

The idea being that indentation is not intended for output and trailing whitespace is a bad idea all-around. 2 is added because that is almost universally what you want and not having it is really really cumbersome and mistake-prone.

[basarat]

@syranide reading that I can only see that there is one more rule regarding tab handling. I've updated the original question with point 3, and also see demo

Cheers 🌹

[RReverser]

This question is being raised rather often, I'm thinking maybe we should add some kind of react.md in this repo?

[basarat]

I'm thinking maybe we should add some kind of react.md in this repo?

I would prefer whitespace.md. Even though the thing I am discussing feels react emit specific, but I feel that not having whitespace rules specified in jsx decreases its portability.

Consider:

<div> 
test
</div>

Would be weird if something things see it as test (like react) and other's see it as \ntest\n. But react.md is fine as well :)

[syranide]

@basarat "other's see it as \ntest\n", which ones are that?

[basarat]

"other's see it as \ntest\n", which ones are that?

Other implementations that might use JSX for an HTML emit