MDL-56511 Theme: behat fixes for theme clean

diff --git a/blocks/tests/behat/behat_blocks.php b/blocks/tests/behat/behat_blocks.php
index 9f459f9..b05d84c 100644 (file)
--- a/blocks/tests/behat/behat_blocks.php
+++ b/blocks/tests/behat/behat_blocks.php
@@ -108,7 +108,7 @@ class behat_blocks extends behat_base {
         }
 
         $this->execute('behat_general::i_click_on_in_the',
-            array("a[role='button']", "css_element", $this->escape($blockname), "block")
+            array(get_string('actions'), "link", $this->escape($blockname), "block")
         );
     }
 

diff --git a/course/classes/management_renderer.php b/course/classes/management_renderer.php
index 13b9e59..56e47e9 100644 (file)
--- a/course/classes/management_renderer.php
+++ b/course/classes/management_renderer.php
@@ -528,7 +528,7 @@ class core_course_management_renderer extends plugin_renderer_base {
             array('id' => 'course-listing-title', 'tabindex' => '0'));
         $html .= $this->course_listing_actions($category, $course, $perpage);
         $html .= $this->listing_pagination($category, $page, $perpage);
-        $html .= html_writer::start_tag('ul', array('class' => 'ml-1', 'role' => 'group'));
+        $html .= html_writer::start_tag('ul', array('class' => 'ml-1 course-list', 'role' => 'group'));
         foreach ($category->get_courses($options) as $listitem) {
             $html .= $this->course_listitem($category, $listitem, $courseid);
         }

diff --git a/course/tests/behat/category_change_visibility.feature b/course/tests/behat/category_change_visibility.feature
index a59850c..55087be 100644 (file)
--- a/course/tests/behat/category_change_visibility.feature
+++ b/course/tests/behat/category_change_visibility.feature
@@ -134,9 +134,9 @@ Feature: We can change the visibility of categories in the management interface.
     And I should see "Cat 2" in the "#category-listing ul" "css_element"
     And I should see "Cat 3" in the "#category-listing ul" "css_element"
     And I should see "Cat 4" in the "#category-listing ul" "css_element"
-    And I should see "Course 1" in the "#course-listing ul" "css_element"
-    And I should see "Course 2" in the "#course-listing ul" "css_element"
-    And I should see "Course 3" in the "#course-listing ul" "css_element"
+    And I should see "Course 1" in the "#course-listing ul.course-list" "css_element"
+    And I should see "Course 2" in the "#course-listing ul.course-list" "css_element"
+    And I should see "Course 3" in the "#course-listing ul.course-list" "css_element"
     And category in management listing should be visible "CAT1"
     And category in management listing should be visible "CAT2"
     And category in management listing should be visible "CAT3"
@@ -211,9 +211,9 @@ Feature: We can change the visibility of categories in the management interface.
     And I should see "Cat 2" in the "#category-listing ul" "css_element"
     And I should see "Cat 3" in the "#category-listing ul" "css_element"
     And I should see "Cat 4" in the "#category-listing ul" "css_element"
-    And I should see "Course 1" in the "#course-listing ul" "css_element"
-    And I should see "Course 2" in the "#course-listing ul" "css_element"
-    And I should see "Course 3" in the "#course-listing ul" "css_element"
+    And I should see "Course 1" in the "#course-listing ul.course-list" "css_element"
+    And I should see "Course 2" in the "#course-listing ul.course-list" "css_element"
+    And I should see "Course 3" in the "#course-listing ul.course-list" "css_element"
     And category in management listing should be visible "CAT1"
     And category in management listing should be visible "CAT2"
     And category in management listing should be visible "CAT3"

diff --git a/course/tests/behat/category_management.feature b/course/tests/behat/category_management.feature
index 151cd7e..969fcc2 100644 (file)
--- a/course/tests/behat/category_management.feature
+++ b/course/tests/behat/category_management.feature
@@ -101,7 +101,7 @@ Feature: Test category management actions
     And I should see "Cat 1" in the "#category-listing ul" "css_element"
     And I should not see "Cat 2" in the "#category-listing ul" "css_element"
     And I should not see "Cat 3" in the "#category-listing ul" "css_element"
-    And I should see "Course 1" in the "#course-listing ul" "css_element"
+    And I should see "Course 1" in the "#course-listing ul.course-list" "css_element"
 
   Scenario: Test I can assign roles for a category through the management interface.
     Given the following "categories" exist:
@@ -298,8 +298,8 @@ Feature: Test category management actions
     And I click on category "Cat 3" in the management interface
     #Redirect
     And I should see the "Course categories and courses" management page
-    And I should see "Course 1" in the "#course-listing ul" "css_element"
-    And I should see "Course 2" in the "#course-listing ul" "css_element"
+    And I should see "Course 1" in the "#course-listing ul.course-list" "css_element"
+    And I should see "Course 2" in the "#course-listing ul.course-list" "css_element"
     And the "movecategoriesto" "select" should be disabled
     And the "resortcategoriesby" "select" should be disabled
     And the "resortcoursesby" "select" should be disabled

diff --git a/course/tests/behat/course_change_visibility.feature b/course/tests/behat/course_change_visibility.feature
index 2c41116..25c5014 100644 (file)
--- a/course/tests/behat/course_change_visibility.feature
+++ b/course/tests/behat/course_change_visibility.feature
@@ -20,7 +20,7 @@ Feature: We can change the visibility of courses in the management interface.
     # Redirect.
     And I should see the "Course categories and courses" management page
     And I should see "Cat 1" in the "#category-listing ul" "css_element"
-    And I should see "Course 1" in the "#course-listing ul" "css_element"
+    And I should see "Course 1" in the "#course-listing ul.course-list" "css_element"
     And category in management listing should be visible "CAT1"
     And course in management listing should be visible "C1"
     And I toggle visibility of course "C1" in management listing
@@ -69,7 +69,7 @@ Feature: We can change the visibility of courses in the management interface.
     And I start watching to see if a new page loads
     And I should see the "Course categories and courses" management page
     And I should see "Cat 1" in the "#category-listing ul" "css_element"
-    And I should see "Course 1" in the "#course-listing ul" "css_element"
+    And I should see "Course 1" in the "#course-listing ul.course-list" "css_element"
     And category in management listing should be visible "CAT1"
     And course in management listing should be visible "C1"
     And I toggle visibility of course "C1" in management listing
@@ -98,7 +98,7 @@ Feature: We can change the visibility of courses in the management interface.
     And I click on "Courses" "link" in the ".view-mode-selector" "css_element"
     And a new page should have loaded since I started watching
     And I start watching to see if a new page loads
-    And I should see "Course 1" in the "#course-listing ul" "css_element"
+    And I should see "Course 1" in the "#course-listing ul.course-list" "css_element"
     And I toggle visibility of course "C1" in management listing
     And a new page should not have loaded since I started watching
     And course in management listing should be dimmed "C1"

diff --git a/lib/templates/action_menu_trigger.mustache b/lib/templates/action_menu_trigger.mustache
index fce1a8b..2a79115 100644 (file)
--- a/lib/templates/action_menu_trigger.mustache
+++ b/lib/templates/action_menu_trigger.mustache
@@ -27,4 +27,4 @@
         "triggerextraclasses": ""
     }
 }}
-<a href="#" class="{{triggerextraclasses}} toggle-display {{#menutrigger}}textmenu{{/menutrigger}}" id="action-menu-toggle-{{instance}}" title="{{title}}" role="menuitem">{{{actiontext}}}{{{menutrigger}}}{{#icon}}{{#pix}}{{key}}, {{component}}, {{title}}{{/pix}}{{/icon}}{{#rawicon}}{{{.}}}{{/rawicon}}{{#menutrigger}}<b class="caret"></b>{{/menutrigger}}</a>
+<a href="#" class="{{triggerextraclasses}} toggle-display {{#menutrigger}}textmenu{{/menutrigger}}" id="action-menu-toggle-{{instance}}" title="{{title}}">{{{actiontext}}}{{{menutrigger}}}{{#icon}}{{#pix}}{{key}}, {{component}}, {{title}}{{/pix}}{{/icon}}{{#rawicon}}{{{.}}}{{/rawicon}}{{#menutrigger}}<b class="caret"></b>{{/menutrigger}}</a>

diff --git a/lib/templates/initials_bar.mustache b/lib/templates/initials_bar.mustache
index dcb7001..2fe1331 100644 (file)
--- a/lib/templates/initials_bar.mustache
+++ b/lib/templates/initials_bar.mustache
@@ -76,10 +76,10 @@
 <div class="initialbar {{class}}">
     <span class="initialbarlabel">{{title}}</span>
     {{#current}}
-        <a class="initialbarall letter" href="{{url}}">{{all}}</a>
+        <a class="initialbarall page-item letter" href="{{url}}">{{all}}</a>
     {{/current}}
     {{^current}}
-        <div class="initialbarall letter active">{{all}}</div>
+        <div class="initialbarall letter page-item active">{{all}}</div>
     {{/current}}
 
     <div class="initialbargroups">
@@ -87,10 +87,10 @@
             <ul class="initialbargroup">
                 {{#letter}}
                     {{#selected}}
-                        <li><span class="letter active {{name}}">{{name}}</span></li>
+                        <li><span class="letter page-item active {{name}}">{{name}}</span></li>
                     {{/selected}}
                     {{^selected}}
-                        <li><a class="letter {{name}}" href="{{url}}">{{name}}</a></li>
+                        <li><a class="letter page-item {{name}}" href="{{url}}">{{name}}</a></li>
                     {{/selected}}
                 {{/letter}}
             </ul>

diff --git a/theme/boost/classes/output/core_course/management/renderer.php b/theme/boost/classes/output/core_course/management/renderer.php
index d626a66..236de1c 100644 (file)
--- a/theme/boost/classes/output/core_course/management/renderer.php
+++ b/theme/boost/classes/output/core_course/management/renderer.php
@@ -461,7 +461,7 @@ class renderer extends \core_course_management_renderer {
         $html .= html_writer::start_div('card-body');
         $html .= $this->course_listing_actions($category, $course, $perpage);
         $html .= $this->listing_pagination($category, $page, $perpage);
-        $html .= html_writer::start_tag('ul', array('class' => 'ml', 'role' => 'group'));
+        $html .= html_writer::start_tag('ul', array('class' => 'ml course-list', 'role' => 'group'));
         foreach ($category->get_courses($options) as $listitem) {
             $html .= $this->course_listitem($category, $listitem, $courseid);
         }

diff --git a/theme/boost/templates/core/action_menu_trigger.mustache b/theme/boost/templates/core/action_menu_trigger.mustache
index 93995e1..f5078f9 100644 (file)
--- a/theme/boost/templates/core/action_menu_trigger.mustache
+++ b/theme/boost/templates/core/action_menu_trigger.mustache
@@ -78,7 +78,7 @@
     }
 }}
 <div class="dropdown">
-    <a href="#" class="{{triggerextraclasses}} dropdown-toggle" id="dropdown-{{instance}}" title="{{title}}" role="button" data-toggle="dropdown" aria-haspopup="true" aria-expanded="false">
+    <a href="#" class="{{triggerextraclasses}} dropdown-toggle" id="dropdown-{{instance}}" title="{{title}}" data-toggle="dropdown" aria-haspopup="true" aria-expanded="false">
         {{{actiontext}}}
         {{{menutrigger}}}
             {{#icon}}