Merge pull request #1079 from MITx/feature/cdodge/tabs-show-only-name-on-preview

made a special 'module preview' view for Static Tabs in the CMS. Per Tom...

cms/djangoapps/contentstore/views.py

@@ -496,11 +496,20 @@ def load_preview_module(request, preview_id, descriptor, instance_state, shared_
             error_msg=exc_info_to_str(sys.exc_info())
         ).xmodule_constructor(system)(None, None)
 
-    module.get_html = wrap_xmodule(
-        module.get_html,
-        module,
-        "xmodule_display.html",
-    )
+    # cdodge: Special case 
+    if module.location.category == 'static_tab':
+        module.get_html = wrap_xmodule(
+            module.get_html,
+            module,
+            "xmodule_tab_display.html",
+        )
+    else:        
+        module.get_html = wrap_xmodule(
+            module.get_html,
+            module,
+            "xmodule_display.html",
+        )
+
     module.get_html = replace_static_urls(
         module.get_html,
         module.metadata.get('data_dir', module.location.course),

cms/templates/xmodule_tab_display.html

+<section class="xmodule_display xmodule_${class_}" data-type="${module_name}">
+    ${display_name}
+</section>

common/djangoapps/xmodule_modifiers.py

@@ -21,6 +21,7 @@ def wrap_xmodule(get_html, module, template, context=None):
     module: An XModule
     template: A template that takes the variables:
         content: the results of get_html,
+        display_name: the display name of the xmodule, if available (None otherwise)
         class_: the module class name
         module_name: the js_module_name of the module
     """
@@ -31,6 +32,7 @@ def wrap_xmodule(get_html, module, template, context=None):
     def _get_html():
         context.update({
             'content': get_html(),
+            'display_name' : module.metadata.get('display_name') if module.metadata is not None else None,
             'class_': module.__class__.__name__,
             'module_name': module.js_module_name
         })