4 ms·
Try removing the indentation guides or decreasing their contrast so that they are just barely visible. They appear to cause strong visual grouping between code
by suchow 13y ago
Try removing the indentation guides or decreasing their contrast so that they are just barely visible. They appear to cause strong visual grouping between code blocks and make it harder to see what's what.
- city41 13y agoThat's a fair point. I will try that and maybe some more tweaks too.
- ilaksh 13y agoSeems like you really have a lot of contrast in that theme. Also I hope you will consider dropping the extra parentheses. And instead of dropping CoffeeScript I hope you will try out ToffeeScript. But just use the basic ! thing, you don't need the other stuff like auto callback. e, data1 = fs.readFile! 'foo' e, data2 = redisclient.get! "thekey" console.log data1 console.log data2
- city41 13y agoWhen I first started using CoffeeScript I dropped just about every optional character you could. Lately I've been going in the opposite direction and adding most of them back in. I find this is also a matter of taste and experience. Whether I add the parens or not depends a lot on what's going on at that point. I do appreciate how flexible CS can be here. I'm interested in things like ToffeeScript and JS generators. I do like what they do for code readability.
- ricardobeat 13y agoI'd add using four spaces and a slightly increased line-height has major benefits to readability.
- dham 13y agoI'll go ahead and back this point up also. The visual white space is actually making it harder to read the code.