[BD-10] Remove uses of variable uses_pattern_library (#24628)

cms/templates/widgets/user_dropdown.html

@@ -8,61 +8,27 @@
   from student.roles import GlobalStaff
 %>
 
-% if uses_pattern_library:
-  <%!
-    ## TODO: Use metric to see if CMS ever uses pattern library or if this case can be deleted.
-    ## NOTE: When removing, remove all references to `set_custom_metric`.
-    set_custom_metric('uses_pattern_library', True)
-  %>
-  <div class="wrapper-user-menu dropdown-menu-container logged-in js-header-user-menu">
-      <h3 class="title menu-title">
-          <span class="sr-only">${_("Currently signed in as:")}</span>
-          <span class="account-username" title="${ user.username }">${ user.username }</span>
-      </h3>
-      <button type="button" class="menu-button button-more has-dropdown js-dropdown-button" aria-haspopup="true" aria-expanded="false" aria-controls="${_("Usermenu")}">
-          <span class="icon-fallback icon-fallback-img">
-            <span class="icon icon-angle-down" aria-hidden="true"></span>
-            <span class="sr-only">${_("Usermenu dropdown")}</span>
-          </span>
-      </button>
-      <ul class="dropdown-menu list-divided is-hidden" id="${_("Usermenu")}" tabindex="-1">
-          <%block name="navigation_dropdown_menu_links" >
-              <li class="dropdown-item item has-block-link">
-                <a href="/">${_("{studio_name} Home").format(studio_name=settings.STUDIO_SHORT_NAME)}</a>
-              </li>
-          </%block>
-          <li class="dropdown-item item has-block-link">
-            <a class="action action-signout" href="${settings.FRONTEND_LOGOUT_URL}">${_("Sign Out")}</a>
-          </li>
-      </ul>
+<h3 class="title">
+  <span class="label">
+    <span class="label-prefix sr-only">${_("Currently signed in as:")}</span>
+    <span class="account-username" title="${ user.username }">${ user.username }</span>
+  </span>
+  <span class="icon fa fa-caret-down ui-toggle-dd" aria-hidden="true"></span>
+</h3>
+<div class="wrapper wrapper-nav-sub">
+  <div class="nav-sub">
+    <ul>
+      <li class="nav-item nav-account-dashboard">
+        <a href="/">${_("{studio_name} Home").format(studio_name=settings.STUDIO_SHORT_NAME)}</a>
+      </li>
+      % if GlobalStaff().has_user(user):
+      <li class="nav-item">
+        <a href="${reverse('maintenance:maintenance_index')}">${_("Maintenance")}</a>
+      </li>
+      % endif
+      <li class="nav-item nav-account-signout">
+        <a class="action action-signout" href="${settings.FRONTEND_LOGOUT_URL}">${_("Sign Out")}</a>
+      </li>
+    </ul>
   </div>
-
-% else:
-  <%!
-    set_custom_metric('uses_pattern_library', False)
-  %>
-  <h3 class="title">
-    <span class="label">
-      <span class="label-prefix sr-only">${_("Currently signed in as:")}</span>
-      <span class="account-username" title="${ user.username }">${ user.username }</span>
-    </span>
-    <span class="icon fa fa-caret-down ui-toggle-dd" aria-hidden="true"></span>
-  </h3>
-  <div class="wrapper wrapper-nav-sub">
-    <div class="nav-sub">
-      <ul>
-        <li class="nav-item nav-account-dashboard">
-          <a href="/">${_("{studio_name} Home").format(studio_name=settings.STUDIO_SHORT_NAME)}</a>
-        </li>
-        % if GlobalStaff().has_user(user):
-        <li class="nav-item">
-          <a href="${reverse('maintenance:maintenance_index')}">${_("Maintenance")}</a>
-        </li>
-        % endif
-        <li class="nav-item nav-account-signout">
-          <a class="action action-signout" href="${settings.FRONTEND_LOGOUT_URL}">${_("Sign Out")}</a>
-        </li>
-      </ul>
-    </div>
-  </div>
-% endif
+</div>

lms/djangoapps/courseware/views/views.py

@@ -584,7 +584,6 @@ class StaticCourseTabView(EdxFragmentView):
             'active_page': 'static_tab_{0}'.format(tab['url_slug']),
             'tab': tab,
             'fragment': fragment,
-            'uses_pattern_library': False,
             'disable_courseware_js': True,
         })
 

lms/templates/header/header.html

@@ -22,15 +22,6 @@ from openedx.core.djangoapps.site_configuration import helpers as configuration_
 ## Provide a hook for themes to inject branding on top.
 <%block name="navigation_top" />
 
-## Add UI Toolkit components if using the Pattern Library
-% if uses_pattern_library:
-    <%block name="js_extra">
-    <%static:require_module module_name="js/header_factory" class_name="HeaderFactory">
-    HeaderFactory();
-    </%static:require_module>
-    </%block>
-% endif
-
 <%
   unsupported_browser_alert_versions = configuration_helpers.get_value('UNSUPPORTED_BROWSER_ALERT_VERSIONS', settings.FEATURES.get('UNSUPPORTED_BROWSER_ALERT_VERSIONS'))
 %>

lms/templates/main.html

@@ -99,7 +99,7 @@ from pipeline_mako import render_require_js_path_overrides
     <%static:css group='${self.attr.main_css}'/>
   % endif
 
-  % if disable_courseware_js or uses_pattern_library:
+  % if disable_courseware_js:
     <%static:js group='base_vendor'/>
     <%static:js group='base_application'/>
   % else:

lms/templates/navigation/navigation.html

@@ -25,15 +25,6 @@ from openedx.core.djangoapps.lang_pref.api import header_language_selector_is_en
 ## Provide a hook for themes to inject branding on top.
 <%block name="navigation_top" />
 
-## Add UI Toolkit components if using the Pattern Library
-% if uses_pattern_library:
-    <%block name="js_extra">
-    <%static:require_module module_name="js/header_factory" class_name="HeaderFactory">
-    HeaderFactory();
-    </%static:require_module>
-    </%block>
-% endif
-
 % if uses_bootstrap:
   <header class="navigation-container header-global ${'slim' if course else ''}">
     <nav class="navbar navbar-expand-lg">

lms/templates/user_dropdown.html

@@ -39,27 +39,6 @@ profile_image_url = get_profile_image_urls_for_user(self.real_user)['medium']
         <li role="presentation" class="nav-item nav-item-open-collapsed-only"><a role="menuitem" href="${reverse('account_settings')}">${_("Account")}</a></li>
         <li role="presentation" class="nav-item nav-item-open-collapsed-only"><a role="menuitem" href="${reverse('logout')}">${_("Sign Out")}</a></li>
     </ul>
-% elif uses_pattern_library:
-    <div class="wrapper-user-menu dropdown-menu-container logged-in js-header-user-menu">
-        <a href="${reverse('dashboard')}" class="menu-title">
-            <span class="sr-only">${_("Dashboard for:")}</span>
-            <img class="menu-image" src="${profile_image_url}" alt="${username}">
-            ${username}
-        </a>
-        <div role="group" aria-label="User menu" class="user-menu">
-            <button type="button" class="menu-button button-more has-dropdown js-dropdown-button" aria-expanded="false" aria-controls="${_("Usermenu")}">
-                  <span class="icon fa fa-caret-down" aria-hidden="true"></span>
-                  <span class="sr-only">${_("Usermenu dropdown")}</span>
-            </button>
-            <ul class="dropdown-menu list-divided is-hidden" id="${_("Usermenu")}" tabindex="-1">
-                <%block name="navigation_dropdown_menu_links" >
-                    <li class="dropdown-item item has-block-link"><a href="${reverse('dashboard')}" role="menuitem" class="action dropdown-menuitem">${_("Dashboard")}</a></li>
-                    <li class="dropdown-item item has-block-link"><a href="${reverse('account_settings')}" role="menuitem" class="action dropdown-menuitem">${_("Account")}</a></li>
-                </%block>
-                <li class="dropdown-item item has-block-link"><a href="${reverse('logout')}" role="menuitem" class="action dropdown-menuitem">${_("Sign Out")}</a></li>
-            </ul>
-        </div>
-    </div>
 % else:
     <ol class="user">
         <li class="primary">