Browse Source

Add option to setting to hide lp navigation with arrows see BT#14938

New option in setting added "hide_lp_arrow_navigation":

$_configuration['lp_view_settings'] = [
    'display' => [
        'show_reporting_icon' => true,
        'hide_lp_arrow_navigation' => false,
    ],
];
Julio Montoya 6 years ago
parent
commit
744479d6df
2 changed files with 28 additions and 12 deletions
  1. 6 1
      main/install/configuration.dist.php
  2. 22 11
      main/lp/learnpath.class.php

+ 6 - 1
main/install/configuration.dist.php

@@ -817,7 +817,12 @@ ALTER TABLE skill_rel_course ADD CONSTRAINT FK_E7CEC7FA613FECDF FOREIGN KEY (ses
 //$_configuration['send_two_inscription_confirmation_mail'] = false;
 
 // LP view custom settings
-// $_configuration['lp_view_settings'] = ['display' => ['show_reporting_icon' => true]];
+/*$_configuration['lp_view_settings'] = [
+    'display' => [
+        'show_reporting_icon' => true,
+        'hide_lp_arrow_navigation' => false,
+    ],
+];*/
 
 // Force to hide the invisible course documents in sessions
 //$_configuration['hide_invisible_course_documents_in_sessions'] = false;

+ 22 - 11
main/lp/learnpath.class.php

@@ -2022,6 +2022,8 @@ class learnpath
                 <span class="fa fa-info"></span><span class="sr-only">'.$reportingText.'</span>
             </a>';
 
+
+
         if (!empty($display)) {
             $showReporting = isset($display['show_reporting_icon']) ? $display['show_reporting_icon'] : true;
             if ($showReporting == false) {
@@ -2029,19 +2031,28 @@ class learnpath
             }
         }
 
-        $previousIcon = '
-            <a class="icon-toolbar" id="scorm-previous" href="#" 
-                onclick="switch_item('.$mycurrentitemid.',\'previous\');return false;" title="'.$previousText.'">
-                <span class="fa fa-chevron-left"></span><span class="sr-only">'.$previousText.'</span>
-            </a>';
+        $hideArrows = false;
+        if (isset($settings['display']) && isset($settings['display']['hide_lp_arrow_navigation'])) {
+            $hideArrows = $settings['display']['hide_lp_arrow_navigation'];
+        }
 
-        $nextIcon = '
-            <a class="icon-toolbar" id="scorm-next" href="#" 
-                onclick="switch_item('.$mycurrentitemid.',\'next\');return false;" title="'.$nextText.'">
-                <span class="fa fa-chevron-right"></span><span class="sr-only">'.$nextText.'</span>
-            </a>';
+        $previousIcon = '';
+        $nextIcon = '';
+        if ($hideArrows === false) {
+            $previousIcon = '
+                <a class="icon-toolbar" id="scorm-previous" href="#" 
+                    onclick="switch_item('.$mycurrentitemid.',\'previous\');return false;" title="'.$previousText.'">
+                    <span class="fa fa-chevron-left"></span><span class="sr-only">'.$previousText.'</span>
+                </a>';
+
+            $nextIcon = '
+                <a class="icon-toolbar" id="scorm-next" href="#" 
+                    onclick="switch_item('.$mycurrentitemid.',\'next\');return false;" title="'.$nextText.'">
+                    <span class="fa fa-chevron-right"></span><span class="sr-only">'.$nextText.'</span>
+                </a>';
+        }
 
-        if ($this->mode == 'fullscreen') {
+        if ($this->mode === 'fullscreen') {
             $navbar = '
                   <span id="'.$idBar.'" class="buttons">
                     '.$reportingIcon.'