Cheatsheet visibility code refactored.

cms/static/js/views/modals/edit_xblock.js

@@ -132,17 +132,11 @@ define(["jquery", "underscore", "gettext", "js/views/modals/base_modal",
             },
 
             changeMode: function(event) {
+                this.removeCheatsheetVisibility();
                 var parent = $(event.target.parentElement),
                     mode = parent.data('mode');
                 event.preventDefault();
                 this.selectMode(mode);
-                var $cheatsheet = $('.simple-editor-cheatsheet');
-                if ($cheatsheet.length == 0){
-                    $cheatsheet = $('.simple-editor-open-ended-cheatsheet');
-                }
-                $(".CodeMirror").css({"overflow": "none"});
-                $(".modal-content").removeAttr("style");
-                $cheatsheet.removeClass('shown');
             },
 
             selectMode: function(mode) {
@@ -204,6 +198,17 @@ define(["jquery", "underscore", "gettext", "js/views/modals/base_modal",
                     mode: mode,
                     displayName: displayName
                 }));
+            },
+
+            removeCheatsheetVisibility: function() {
+                var cheatsheet = $('article.simple-editor-open-ended-cheatsheet');
+                if (cheatsheet.length === 0) {
+                    cheatsheet = $('article.simple-editor-cheatsheet');
+                }
+                if (cheatsheet.hasClass('shown')) {
+                    cheatsheet.removeClass('shown');
+                    $('.modal-content').removeClass('cheatsheet-is-shown');
+                }
             }
         });
 

cms/static/sass/_shame.scss

@@ -74,3 +74,11 @@ body b {
 * use the @include placeholder Bourbon mixin (http://bourbon.io/docs/#placeholder) for any placeholder styling
 
 */
+
+.modal-content.cheatsheet-is-shown {
+  overflow: visible !important;
+
+  .CodeMirror {
+    overflow: visible !important;
+  }
+}