Fixes #1125: change defaults to insertSpaces: true, tabSize: 4

src/vs/editor/common/config/defaultConfig.ts

@@ -63,8 +63,8 @@ class ConfigClass implements IConfiguration {
 			referenceInfos: true,
 			renderWhitespace: false,
 
-			tabSize: 'auto',
-			insertSpaces: 'auto',
+			tabSize: 4,
+			insertSpaces: true,
 			fontFamily: '',
 			fontSize: 0,
 			lineHeight: 0

src/vs/editor/common/editorCommon.ts

@@ -490,12 +490,14 @@ export interface ICommonEditorOptions {
 export interface IEditorOptions extends ICommonEditorOptions {
 	/**
 	 * Tab size in spaces. This is used for rendering and for editing.
-	 * Defaults to 'auto', meaning the model attached to the editor will be scanned and this property will be guessed.
+	 * 'auto' means the model attached to the editor will be scanned and this property will be guessed.
+	 * Defaults to 4.
 	 */
 	tabSize?:any;
 	/**
 	 * Insert spaces instead of tabs when indenting or when auto-indenting.
-	 * Defaults to 'auto', meaning the model attached to the editor will be scanned and this property will be guessed.
+	 * 'auto' means the model attached to the editor will be scanned and this property will be guessed.
+	 * Defaults to true.
 	 */
 	insertSpaces?:any;
 	/**

src/vs/editor/test/common/controller/cursor.test.ts

@@ -756,7 +756,10 @@ suite('Editor Controller - Cursor', () => {
 
 	test('Bug 9121: Auto indent + undo + redo is funky', () => {
 		var model = new Model.Model('', thisHighlighter);
-		var cursor = new Cursor.Cursor(1, new MockConfiguration(null), model, null, false);
+		var cursor = new Cursor.Cursor(1, new MockConfiguration({
+			tabSize: 4,
+			insertSpaces: false
+		}), model, null, false);
 		cursorCommand(cursor, H.Type, { text: '\n' }, null, 'keyboard');
 		assert.equal(model.getValue(EditorCommon.EndOfLinePreference.LF), '\n', 'assert1');