From 48b69300050b7e098a67c6f5a8833f9fbf8867bc Mon Sep 17 00:00:00 2001
From: Angel Fernando Quiroz Campos <angelfqc.18@gmail.com>
Date: Mon, 22 Feb 2016 10:29:16 -0500
Subject: [PATCH] Minor - Change to Bootstrap style - refs BT#10856 #ofj

---
 main/inc/lib/tracking.lib.php | 52 +++++++++++++++++++++++------------
 1 file changed, 35 insertions(+), 17 deletions(-)

diff --git a/main/inc/lib/tracking.lib.php b/main/inc/lib/tracking.lib.php
index 76580cb43c..cbf9fc984b 100755
--- a/main/inc/lib/tracking.lib.php
+++ b/main/inc/lib/tracking.lib.php
@@ -4490,7 +4490,9 @@ class Tracking
                 Display::return_icon('session.png', get_lang('Sessions'), array(), ICON_SIZE_SMALL) . ' ' . get_lang('Sessions')
             );
 
-            $html .= '<table class="data_table" width="100%">';
+            $html .= '<div class="table-responsive">';
+            $html .= '<table class="table table-striped table-hover">';
+            $html .= '<thead>';
             $html .= '<tr>
                   '.Display::tag('th', get_lang('Session'), array('width'=>'300px')).'
                   '.Display::tag('th', get_lang('PublishedExercises'), array('width'=>'300px')).'
@@ -4498,6 +4500,8 @@ class Tracking
                   '.Display::tag('th', get_lang('AverageExerciseResult'), array('class'=>'head')).'
                   '.Display::tag('th', get_lang('Details'), array('class'=>'head')).'
                   </tr>';
+            $html .= '</thead>';
+            $html .= '<tbody>';
 
             foreach ($course_in_session as $my_session_id => $session_data) {
                 $course_list  = $session_data['course_list'];
@@ -4574,7 +4578,8 @@ class Tracking
                 $html .= Display::tag('td', $icon);
                 $html .= '</tr>';
             }
-            $html .= '</table><br />';
+            $html .= '</tbody>';
+            $html .= '</table></div><br />';
             $html .= Display::div($main_session_graph, array('id'=>'session_graph','class'=>'chart-session', 'style'=>'position:relative; text-align: center;') );
 
             // Checking selected session.
@@ -4586,9 +4591,11 @@ class Tracking
 
                 $html .= Display::tag('h3',$session_data['name'].' - '.get_lang('CourseList'));
 
-                $html .= '<table class="data_table" width="100%">';
+                $html .= '<div class="table-responsive">';
+                $html .= '<table class="table table-hover table-striped">';
                 //'.Display::tag('th', get_lang('DoneExercises'),         array('class'=>'head')).'
                 $html .= '
+                    <thead>
                     <tr>
                       <th width="300px">'.get_lang('Course').'</th>
                       '.Display::tag('th', get_lang('PublishedExercises'),    	array('class'=>'head')).'
@@ -4600,7 +4607,10 @@ class Tracking
                       '.Display::tag('th', get_lang('Score').Display::return_icon('info3.gif', get_lang('ScormAndLPTestTotalAverage'), array ('align' => 'absmiddle', 'hspace' => '3px')), array('class'=>'head')).'
                       '.Display::tag('th', get_lang('LastConnexion'),         	array('class'=>'head')).'
                       '.Display::tag('th', get_lang('Details'),               	array('class'=>'head')).'
-                    </tr>';
+                    </tr>
+                    </thead>
+                    <tbody>
+                ';
 
                 foreach ($course_list as $course_data) {
                     $course_code  = $course_data['code'];
@@ -4717,7 +4727,7 @@ class Tracking
                     $html .= Display::tag('td', $details, array('align'=>'center'));
                     $html .= '</tr>';
                 }
-                $html .= '</table>';
+                $html .= '</tbody></table></div>';
             }
         }
 
@@ -4743,18 +4753,22 @@ class Tracking
             $em = Database::getManager();
 
             $html .= Display::page_subheader($course_info['title']);
-            $html .= '<table class="data_table" width="100%">';
+            $html .= '<div class="table-responsive">';
+            $html .= '<table class="table table-striped table-hover">';
 
             //Course details
             $html .= '
+                <thead>
                 <tr>
-                <th class="head" style="color:#000">'.get_lang('Exercises').'</th>
-                <th class="head" style="color:#000">'.get_lang('Attempts').'</th>
-                <th class="head" style="color:#000">'.get_lang('BestAttempt').'</th>
-                <th class="head" style="color:#000">'.get_lang('Ranking').'</th>
-                <th class="head" style="color:#000">'.get_lang('BestResultInCourse').'</th>
-                <th class="head" style="color:#000">'.get_lang('Statistics').' '.Display :: return_icon('info3.gif', get_lang('OnlyBestResultsPerStudent'), array('align' => 'absmiddle', 'hspace' => '3px')).'</th>
-                </tr>';
+                <th>'.get_lang('Exercises').'</th>
+                <th>'.get_lang('Attempts').'</th>
+                <th>'.get_lang('BestAttempt').'</th>
+                <th>'.get_lang('Ranking').'</th>
+                <th>'.get_lang('BestResultInCourse').'</th>
+                <th>'.get_lang('Statistics').' '.Display :: return_icon('info3.gif', get_lang('OnlyBestResultsPerStudent'), array('align' => 'absmiddle', 'hspace' => '3px')).'</th>
+                </tr>
+                </thead>
+                <tbody>';
 
             if (empty($session_id)) {
                 $user_list = CourseManager::get_user_list_from_course_code(
@@ -4918,17 +4932,19 @@ class Tracking
             } else {
                 $html .= '<tr><td colspan="5" align="center">'.get_lang('NoEx').'</td></tr>';
             }
-            $html .= '</table>';
+            $html .= '</tbody></table></div>';
 
 
             // LP table results
-            $html .='<table class="data_table">';
+            $html .= '<div class="table-responsive">';
+            $html .='<table class="table table-striped table-hover">';
+            $html .= '<thead><tr>';
             $html .= Display::tag('th', get_lang('Learnpaths'), array('class'=>'head', 'style'=>'color:#000'));
             $html .= Display::tag('th', get_lang('LatencyTimeSpent'), array('class'=>'head', 'style'=>'color:#000'));
             $html .= Display::tag('th', get_lang('Progress'), array('class'=>'head', 'style'=>'color:#000'));
             $html .= Display::tag('th', get_lang('Score'), array('class'=>'head', 'style'=>'color:#000'));
             $html .= Display::tag('th', get_lang('LastConnexion'), array('class'=>'head', 'style'=>'color:#000'));
-            $html .= '</tr>';
+            $html .= '</tr></thead><tbody>';
 
             $list = new LearnpathList(
                 api_get_user_id(),
@@ -4986,7 +5002,7 @@ class Tracking
                         </td>
                       </tr>';
             }
-            $html .='</table>';
+            $html .='</tbody></table></div>';
 
             if (api_get_setting('allow_skills_tool') === 'true') {
                 $skillsRelUser = $em->getRepository('ChamiloCoreBundle:SkillRelUser')->findBy([
@@ -5034,6 +5050,8 @@ class Tracking
                                             </ul>
                                         </div>
                     ';
+                } else {
+                    $html .= get_lang('WithoutAchievedSkills');
                 }
 
                 $html .= '