Skip to content
This repository has been archived by the owner on Aug 10, 2023. It is now read-only.

Commit

Permalink
Disable editor autocompletion for didactic reasons
Browse files Browse the repository at this point in the history
In our educational environment, autocompletion is generally not
desirable:

* Beginners might be confused by the suggestions.
* If we want students to memoize the keywords, suggesting them is
  counterproductive.

Of course, autocompletion also has its benefits. If necessary, we can
revisit this decision at a later time. For example, we could have
different settings for beginner and complex tasks.
  • Loading branch information
martinmo committed Jan 12, 2020
1 parent 8ec1930 commit c8ff40c
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions assets/js/editor.js
Original file line number Diff line number Diff line change
Expand Up @@ -861,8 +861,8 @@ class Editor {
mode: "ace/mode/java",
newLineMode: "auto",
tabSize: 4,
enableBasicAutocompletion: true,
enableLiveAutocompletion: true,
enableBasicAutocompletion: false,
enableLiveAutocompletion: false,
maxLines: Infinity,
fontFamily: "Menlo, Monaco, Consolas, \"Courier New\", monospace",
fontSize: "10.5pt",
Expand Down

0 comments on commit c8ff40c

Please sign in to comment.