Commit 01ff8396 by cahrens

Fix CodeMirror initialization problem.

parent 235ecff2
...@@ -113,14 +113,16 @@ class CMS.Views.ModuleEdit extends Backbone.View ...@@ -113,14 +113,16 @@ class CMS.Views.ModuleEdit extends Backbone.View
settingsModeButton = @$el.find('#settings-mode').find("a") settingsModeButton = @$el.find('#settings-mode').find("a")
if mode == @editorMode if mode == @editorMode
dataEditor.addClass('is-active') # Because of CodeMirror editor, cannot hide the data editor when it is first loaded. Therefore
settingsEditor.removeClass('is-active') # we have to use a class of is-inactive instead of is-active.
dataEditor.removeClass('is-inactive')
editorModeButton.addClass('is-set') editorModeButton.addClass('is-set')
settingsEditor.removeClass('is-active')
settingsModeButton.removeClass('is-set') settingsModeButton.removeClass('is-set')
else else
dataEditor.removeClass('is-active') dataEditor.addClass('is-inactive')
settingsEditor.addClass('is-active')
editorModeButton.removeClass('is-set') editorModeButton.removeClass('is-set')
settingsEditor.addClass('is-active')
settingsModeButton.addClass('is-set') settingsModeButton.addClass('is-set')
hideDataEditor: => hideDataEditor: =>
......
...@@ -535,12 +535,14 @@ body.course.unit { ...@@ -535,12 +535,14 @@ body.course.unit {
// Editor Wrapper // Editor Wrapper
.wrapper-comp-editor { .wrapper-comp-editor {
display: none; display: block;
&.is-active { // Because the editor may be a CodeMirror editor (which must be visible at the time it is created
display: block; // in order for it to properly initialize), we must toggle "is-inactive" instead of the more common "is-active".
&.is-inactive {
display: none;
} }
} }
// Settings Wrapper // Settings Wrapper
.wrapper-comp-settings { .wrapper-comp-settings {
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment