smartIndent and indentWithTabs not working

[samuela]

smartIndent and indentWithTabs are not working as expected. See http://jsbin.com/ihunin/34/edit#preview for full explanations and examples. In the first CodeMirror instance, you can see that tabs are inserted instead of spaces and in the second, you can see that smart indentation does not follow the indentation settings and instead just inserts two spaces.

[marijnh]

There are two sources of confusion here:

• You have to set indentUnit and tabSize to the same number to get the behaviour you expected for the second editor
• You have to bind the "Tab" key to something other than "defaultTab" ("indentAuto") is probably what you want in extraKeys.

[samuela]

Thanks! Sorry for the confusion.. These properties do behave a little different than expected though. I'm not sure if this is just me but maybe some mention in the documentation would prevent this question from coming up again

[samuela]

I'm still running into some problems with indentAuto(). See http://jsbin.com/ihunin/49/edit#preview. Attaching it in extraKeys as a string seems to work but calling it by itself doesn't. Am I doing something wrong here?

[marijnh]

indentAuto is a 'command', which are functions stored in CodeMirror.commands. You can call it through CodeMirror.commands.indentAuto(cm), wherecm` is your CodeMirror instance.

[samuela]

Right, and in the jsbin I called CodeMirror.commands.indentAuto(editor). Really though what I'd like to do is something like this::

CodeMirror.fromTextArea(document.getElementById("code"), {
    extraKeys: {
        "Tab": function(cm) {
            if(<blah blah blah>) {
                CodeMirror.commands.indentAuto(editor);
            }
        }
   }
});

But I can't get that to work.. The weird thing is that setting "Tab": "indentAuto" in extraKeys does work. I'll post another jsbin when I get a chance.

[samuela]

Ok I put together a jsbin here http://jsbin.com/ihunin/54/edit#preview. I'm not writing this on my usual computer so if I recall correctly, setting "Tab": "indentAuto" worked on Chrome but now it doesn't look like it's working on Safari 5. Setting "Tab": function(cm) { CodeMirror.commands.indentAuto(cm); } has never worked for me though.

[marijnh]

Describe 'work'. The problem here is likely to be that your expectations of features don't agree to what the manual says.

When a keymap points at a string, it will interpret it as a property of CodeMirror.commands. Thus, the two invocations you gave are equivalent (and the second works just fine when I try it).

[samuela]

Right, and I understand that. Neither of them work for me on Safari. In the second one when I hit tab, I get the alert but no indent. Am I misunderstanding what indentAuto does?

[samuela]

Also, when indentAuto is called in an indented line, all of the indentation is deleted.. I expect indentAuto to insert either indentUnit spaces or one real tab depending on the options. Is this correct?

[marijnh]

What indentAuto does is reset the line's indentation to the 'correct' indentation that the mode reported. You probably want the indentMore command instead.

[samuela]

Got it, that makes sense now. Thanks!