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

Fix #3100: New lines always create white spaces #3209

Merged
merged 2 commits into from
Mar 22, 2013

Conversation

TomMalbran
Copy link
Contributor

This is a possible fix for #3100. The idea is that now changing Spaces and Tabs both also change the indent unit option of all the open editors, since Indent Unit is used for both and not just for Spaces.

editor._codeMirror.setOption(cmOption, value);
});

_setEditorOption(value, cmOption);
_prefs.setValue(prefName, (typeof value === "boolean") ? Boolean(value) : value);
}

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Line 1339 has 2 tab chars. Remove or convert to spaces.

@redmunds
Copy link
Contributor

Done with initial code review.

@ghost ghost assigned redmunds Mar 22, 2013
@TomMalbran
Copy link
Contributor Author

Fixed pushed.

@redmunds
Copy link
Contributor

This looks good. Thanks, @TomMalbran . Merging.

redmunds added a commit that referenced this pull request Mar 22, 2013
Fix #3100: New lines always create white spaces
@redmunds redmunds merged commit 051f2da into adobe:master Mar 22, 2013
@TomMalbran TomMalbran deleted the tom/fix-issue-3100 branch July 23, 2013 08:56
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants