Skip to content
This repository was archived by the owner on Sep 6, 2021. It is now read-only.

Commit 62aef70

Browse files
committed
Merge pull request #7355 from adobe/rlim/tab-size-issue
Use the correct name of CodeMirror option for tab size.
2 parents 3286e7e + 2b58e60 commit 62aef70

1 file changed

Lines changed: 1 addition & 5 deletions

File tree

src/editor/Editor.js

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -109,7 +109,7 @@ define(function (require, exports, module) {
109109
cmOptions[SMART_INDENT] = "smartIndent";
110110
cmOptions[SPACE_UNITS] = "indentUnit";
111111
cmOptions[STYLE_ACTIVE_LINE] = "styleActiveLine";
112-
cmOptions[TAB_SIZE] = "indentUnit";
112+
cmOptions[TAB_SIZE] = "tabSize";
113113
cmOptions[USE_TAB_CHAR] = "indentWithTabs";
114114
cmOptions[WORD_WRAP] = "lineWrapping";
115115

@@ -2018,7 +2018,6 @@ define(function (require, exports, module) {
20182018

20192019
if (oldValue !== newValue) {
20202020
this._currentOptions[prefName] = newValue;
2021-
var useTabChar = this._currentOptions[USE_TAB_CHAR];
20222021

20232022
if (prefName === USE_TAB_CHAR) {
20242023
this._codeMirror.setOption(cmOptions[prefName], newValue);
@@ -2031,9 +2030,6 @@ define(function (require, exports, module) {
20312030
} else if (prefName === SCROLL_PAST_END && this._visibleRange) {
20322031
// Do not apply this option to inline editors
20332032
return;
2034-
} else if ((useTabChar && prefName === SPACE_UNITS) || (!useTabChar && prefName === TAB_SIZE)) {
2035-
// This change conflicts with the useTabChar setting, so do not change the CodeMirror option
2036-
return;
20372033
} else {
20382034
this._codeMirror.setOption(cmOptions[prefName], newValue);
20392035
}

0 commit comments

Comments
 (0)