Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Problem with indentation #19

Open
nyorem opened this issue Mar 1, 2015 · 7 comments
Open

Problem with indentation #19

nyorem opened this issue Mar 1, 2015 · 7 comments

Comments

@nyorem
Copy link

nyorem commented Mar 1, 2015

Hello,
It seems that there is an issue regarding the indentation.

Consider the following snippet, the cursor is represented by '^':

foo =
    bar
    ^

Then if I press backspace, I get the following:

foo =
bar
   ^

The position of the cursor seems to be wrong.

@hoosierEE
Copy link
Contributor

tl;dr I hear you, and am also currently working on this problem

I've also been dissatisfied with the indent file in this plugin (even though I'm the one who sent the PR for it in the first place!), and have been trying various other Haskell indent plugins. So far they all suck for Elm (auto-indenting will break Elm programs), and are proving difficult to adapt.

So... I'm starting to think that it might be easier to adapt an existing Python indent plugin to work for Elm. Python suffers for the same reasons (contextually significant whitespace) as Haskell, but since there are more Python users in the world their tools might be a bit better.

@spinningarrow
Copy link

Also since Elm demands 4-space indentation, maybe the plugin should make vim do that automatically?

@hoosierEE
Copy link
Contributor

Is 4-space indentation an official thing? I've been away from Elm (and this plugin) for a while now; sorry to have dropped the ball. But if 4-space indentation is not semantic, then the user's shiftwidth setting should be used. 4-spaces as a default should be fine though.

@spinningarrow
Copy link

I realised I was referring to this gist which is not
official. On the other hand, I was originally using tabs for indentation
which were definitely giving me whitespace errors from the compiler.

I guess it's not an official thing then.

—Sahil

On Sat, Aug 29, 2015 at 7:21 AM, Alex Shroyer notifications@github.com
wrote:

Is 4-space indentation an official thing? I've been away from Elm (and
this plugin) for a while now; sorry to have dropped the ball. But if
4-space indentation is not semantic, then the user's shiftwidth setting
should be used. 4-spaces as a default should be fine though.


Reply to this email directly or view it on GitHub
#19 (comment).

@talon
Copy link

talon commented Nov 24, 2015

Until this is fixed, deleting the indentation folder works for me.

@gilesbowkett
Copy link

4 spaces is idiomatic (and enforced by elm-format) but not semantic.

@quangv
Copy link

quangv commented Oct 4, 2017

creact-elm-app starting file Main.elm is 4 spaces too btw.

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

No branches or pull requests

6 participants