search

yosssi / ace

HTML template engine for Go
MIT License
836 stars 89 forks source link

Option for number of spaces as indentation #47

Open arvinsim opened 9 years ago

arvinsim commented 9 years ago

Current indentation rule is 2 spaces.

It would be better if there is an option to set this to to other values like 4 spaces. It is cumbersome to change the space tabbing everytime I edit an Ace file since almost everything else I work on use 4 spaces.

yosssi commented 9 years ago

Thanks for your comment. I'll think of adding this option.

TheHippo commented 9 years ago

It looks like I am in the wrong place, but since we are at this topic: How about allowing TAB for indention?

yosssi commented 9 years ago

Thanks for your comment. I think it would be nice if we could add an option that specifies the indent rule (2 spaces, 4 spaces or tab) to the Options struct.

TheHippo commented 9 years ago

I'll try to look into this.

The fix should be applied somewhere around here in line.go

jf commented 9 years ago

YES. Tabs, pls!!! :} Just like go, and just like a lot of sane code... :}