Remove edX branding from default Open edX LMS

OPEN-307

lms/djangoapps/branding/__init__.py

@@ -63,11 +63,15 @@ def get_logo_url():
     # otherwise, use the legacy means to configure this
     university = microsite.get_value('university')
 
-    if university is None:
-        return '{static_url}images/logo-edX-77x36.png'.format(
+    if university is None and settings.FEATURES.get('IS_EDX_DOMAIN', False):
+        return '{static_url}images/edx-logo-77x36.png'.format(
+            static_url=settings.STATIC_URL
+        )
+    elif university:
+        return '{static_url}images/{uni}-on-edx-logo.png'.format(
+            static_url=settings.STATIC_URL, uni=university
+        )
+    else:
+        return '{static_url}images/logo-placeholder.png'.format(
             static_url=settings.STATIC_URL
         )
-
-    return '{static_url}images/{uni}-on-edx-logo.png'.format(
-        static_url=settings.STATIC_URL, uni=university
-    )

lms/envs/common.py

@@ -186,7 +186,8 @@ FEATURES = {
 
     # Enable legacy instructor dashboard
     'ENABLE_INSTRUCTOR_LEGACY_DASHBOARD': True,
-    # Is this an edX-owned domain? (used on instructor dashboard)
+
+    # Is this an edX-owned domain? (used for edX specific messaging and images)
     'IS_EDX_DOMAIN': False,
 
     # Toggle to enable certificates of courses on dashboard

lms/templates/courseware/courses.html

@@ -17,12 +17,17 @@
 
   else:
     course_index_overlay_text = microsite.get_value('course_index_overlay_text', _("Explore courses from {platform_name}.").format(platform_name=platform_name))
-    # not sure why this is, but if I use static.url('images/edx-logo-bw.png') then the HTML rendering
-    # of this template goes wonky
+
+    if settings.FEATURES.get('IS_EDX_DOMAIN', False):
+      # For some reason, `static.url('images/edx-logo-bw.png')` breaks template rendering.
+      default_image_url = settings.STATIC_URL + 'images/edx-logo-bw.png'
+    else:
+      default_image_url = settings.STATIC_URL + 'images/logo-placeholder-large.png'
 
     logo_file = microsite.get_value(
-      'course_index_overlay_logo_file', settings.STATIC_URL + 'images/edx-logo-bw.png'
+      'course_index_overlay_logo_file', default_image_url
     )
+
     logo_alt_text = _("{platform_name} Logo").format(platform_name=platform_name)
 %>
 

lms/templates/footer-edx-new.html

@@ -11,7 +11,7 @@
       <h2 class="sr footer-about-title">${_("About edX")}</h2>
 
       <div class="footer-about-logo">
-        <img alt="edX logo" src="${static.url('images/header-logo.png')}">
+        <img alt="edX logo" src="${static.url('images/edx-header-logo.png')}">
       </div>
 
       <div class="footer-about-copy">

lms/templates/main.html

@@ -153,7 +153,7 @@
       <%
         if theme_enabled() and not is_microsite():
           footer_file = 'theme-footer.html'
-        elif settings.FEATURES['IS_EDX_DOMAIN'] and not is_microsite():
+        elif settings.FEATURES.get('IS_EDX_DOMAIN', False) and not is_microsite():
           footer_file = microsite.get_template_path('footer-edx-new.html')
         else:
           footer_file = microsite.get_template_path('footer.html')