From 75658ad2ca1f0a31a569d3f4955ccf5b58fa5d8c Mon Sep 17 00:00:00 2001
From: Dan Marsden <dan@danmarsden.com>
Date: Sat, 29 Oct 2011 15:28:09 +1300
Subject: [PATCH] SCORM MDL-29965 improve display of SCORM package - decrease
 unecessary padding

---
 mod/scorm/player.php | 18 +++++++++---------
 1 file changed, 9 insertions(+), 9 deletions(-)

diff --git a/mod/scorm/player.php b/mod/scorm/player.php
index c72a91b7969..572affe276a 100644
--- a/mod/scorm/player.php
+++ b/mod/scorm/player.php
@@ -188,18 +188,18 @@
         <div id='scormapi-parent'>
             <script id="external-scormapi" type="text/JavaScript"></script>
         </div>
-        <div id="scormtop">
-        <?php echo $mode == 'browse' ? '<div id="scormmode" class="scorm-left">'.get_string('browsemode','scorm')."</div>\n" : ''; ?>
-        <?php echo $mode == 'review' ? '<div id="scormmode" class="scorm-left">'.get_string('reviewmode','scorm')."</div>\n" : ''; ?>
-            <div id="scormnav" class="scorm-right">
 <?php
-if ($scorm->hidetoc == SCORM_TOC_POPUP) {
-    echo $result->tocmenu;
+if ($scorm->hidetoc == SCORM_TOC_POPUP or $mode=='browse' or $mode=='review') {
+    echo '<div id="scormtop">';
+    echo $mode == 'browse' ? '<div id="scormmode" class="scorm-left">'.get_string('browsemode', 'scorm')."</div>\n" : '';
+    echo $mode == 'review' ? '<div id="scormmode" class="scorm-left">'.get_string('reviewmode', 'scorm')."</div>\n" : '';
+    if ($scorm->hidetoc == SCORM_TOC_POPUP) {
+        echo '<div id="scormnav" class="scorm-right">'.$result->tocmenu.'</div>';
+    }
+    echo '</div>';
 }
 ?>
-            </div> <!-- Scormnav -->
-        </div> <!-- Scormtop -->
-            <div id="toctree" class="generalbox">
+            <div id="toctree">
                 <?php
                 if (empty($scorm->popup) || $displaymode == 'popup') {
                     echo $result->toc;