Set tab size in Ace editor to 2 (rstudio#761)

Co-authored-by: Garrick Aden-Buie <[email protected]>
xavidp · Feb 21, 2023 · b6669f9 · b6669f9
1 parent e0a42bd
commit b6669f9
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 3 deletions.
diff --git a/NEWS.md b/NEWS.md
@@ -2,6 +2,8 @@
 
 -   Fixed an issue that prevented authors from using symbols, such as `T` or a variable, as the value of the `exercise` chunk option, which caused tutorials with chunks with `exercise = T` to fail to render (thanks @cknotz #757, #758).
 
+-   The embedded Ace editor used in learnr exercises now defaults to a tab width of 2, aligning with the Tidyverse style guide (#761).
+
 # learnr 0.11.2
 
 -   Fixed an issue that prevented htmlwidgets from working in exercise code unless similar widgets were added to the tutorial prose (thanks @munoztd0 #744, #745).

diff --git a/inst/lib/tutorial/tutorial.js b/inst/lib/tutorial/tutorial.js
diff --git a/inst/lib/tutorial/tutorial.js.map b/inst/lib/tutorial/tutorial.js.map
diff --git a/learnr-js/tutorial/tutorial.js b/learnr-js/tutorial/tutorial.js
@@ -864,6 +864,7 @@ Tutorial.prototype.$attachAceEditor = function (target, code, options) {
   editor.session.setMode(`ace/mode/${options.engine}`)
   editor.session.getSelection().clearSelection()
   editor.session.setNewLineMode('unix')
+  editor.session.setTabSize(2)
   editor.setValue(code, -1)
   editor.setOptions({
     enableBasicAutocompletion: true