MDL-78106 gradereport_grader: Fixed sort arrow size in grader report

juacas · Aug 28, 2023 · a9ea85a · a9ea85a
1 parent cccc009
commit a9ea85a
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 4 deletions.
diff --git a/grade/report/grader/styles.css b/grade/report/grader/styles.css
@@ -158,9 +158,8 @@
 }
 
 .path-grade-report-grader .gradeparent .sorticon {
-    margin-left: 5px;
     vertical-align: middle;
-    margin-right: 1px;
+    margin: 0;
 }
 
 .path-grade-report-grader .gradeparent td.grade_type_value,

diff --git a/grade/report/grader/templates/headercell.mustache b/grade/report/grader/templates/headercell.mustache
@@ -34,7 +34,9 @@
     <div class="d-flex" data-collapse="content">
         <div class="d-flex flex-grow-1">
             {{{headerlink}}}
-            {{{arrow}}}
+            <div class="mx-2 mt-2">
+                {{{arrow}}}
+            </div>
         </div>
         <div>
             {{{singleview}}}

diff --git a/grade/report/lib.php b/grade/report/lib.php
@@ -598,7 +598,8 @@ protected function get_sort_arrow(string $direction = 'down', ?moodle_url $sortl
         $matrix = ['up' => 'desc', 'down' => 'asc'];
         $strsort = get_string($matrix[$direction], 'moodle');
         $arrow = $OUTPUT->pix_icon($pix[$direction], '', '', ['class' => 'sorticon']);
-        return html_writer::link($sortlink, $arrow, ['title' => $strsort, 'aria-label' => $strsort, 'data-collapse' => 'sort']);
+        return html_writer::link($sortlink, $arrow, ['title' => $strsort, 'aria-label' => $strsort, 'data-collapse' => 'sort',
+            'class' => 'arrow_link py-1']);
     }
 
     /**