LAYOUT: remove text "game" from "it's your turn" list
[e-DoKo.git] / include / game.php
index 3564d9cc382efbdbff0b705a8be1e33cb1505cba..cef2c6537c2a11141a38ac387f546943f8624b1b 100644 (file)
@@ -49,7 +49,6 @@ global $GAME,$RULES,$CARDS;
 $gameid   = DB_get_gameid_by_hash($me);
 $myname   = DB_get_name('hash',$me);
 $mystatus = DB_get_status_by_hash($me);
-$origmystatus = DB_get_status_by_hash($me); /* to show "it's your turn" menu when game has just finished */
 $mypos    = DB_get_pos_by_hash($me);
 $myhand   = DB_get_handid('hash',$me);
 $myparty  = DB_get_party_by_hash($me);
@@ -175,6 +174,30 @@ if(myisset('call')  && $_REQUEST['call']  == '0' && can_call(0,$me))
   }
 
 
+
+/*****************************************************************
+ * output other games where it is the users turn
+ * make sure that the people looking at old games don't see the wrong games here
+ *****************************************************************/
+
+if( $gamestatus != 'gameover'  && isset($_SESSION['id']) )
+  {
+    /* game isn't over, only valid user can get here, so show menu */
+    display_user_menu($myid, $me);
+  }
+else if( $mystatus == 'gameover' && isset($_SESSION['id']) )
+  {
+    /* user is looking at someone else's game, show the menu for the correct user */
+    display_user_menu($_SESSION['id'],$me);
+  }
+else
+  {
+    echo "<div class=\"usermenu\">\n";
+    echo "It's your turn in these games: \n";
+    echo "Please log in to see this information.\n";
+    echo "</div>\n\n";
+  }
+
 /*****************************************************************
  * output extra division in case this game is part of a session
  *****************************************************************/
@@ -308,6 +331,83 @@ if($mystatus!='gameover')
      DB_update_user_timestamp($_SESSION['id']);
 
 
+/******************************************************************************
+ * Output menu for selecting tricks
+ ******************************************************************************/
+
+switch($mystatus)
+  {
+  case 'start':
+    break;
+  case 'init':
+  case 'check':
+    /* output sickness of other playes, in case they already selected and are sitting in front of the current player */
+    echo "\n<ul class=\"tricks\">\n";
+    echo "  <li onclick=\"hl('0');\" class=\"current\"><a href=\"#\">Pre</a>\n";
+
+    echo "    </li>\n</ul>\n";  /* end div trick, end li trick , end tricks*/
+    /* end displaying sickness */
+    break;
+  case 'poverty':
+    /* output pre-game trick in case user reloads,
+     * only needs to be done when a team has been formed */
+    if($myparty=='re' || $myparty=='contra')
+      {
+       echo "\n<ul class=\"tricks\">\n";
+
+       $mygametype =  DB_get_gametype_by_gameid($gameid);
+
+       echo "  <li onclick=\"hl('0');\" class=\"current\"><a href=\"#\">Pre</a>\n";
+       echo "  </li>\n</ul>\n\n";  /* end div trick, end li trick , end ul tricks */
+      }
+    /* end output pre-game trick */
+    break;
+  case 'play':
+  case 'gameover':
+
+    echo "\n<ul class=\"tricks\">\n";
+
+    /* output vorbehalte */
+    $mygametype = DB_get_gametype_by_gameid($gameid);
+    $mygamesolo = DB_get_solo_by_gameid($gameid);
+    if($mygametype != 'normal') /* only show when needed */
+      if(!( $mygametype == 'solo' && $mygamesolo == 'silent') )
+       echo "  <li onclick=\"hl('0');\" class=\"old\"><a href=\"#\">Pre</a></li>\n";
+
+    $result = DB_query("SELECT Trick.id ".
+                      "FROM Trick ".
+                      "WHERE Trick.game_id='".$gameid."' ".
+                      "GROUP BY Trick.id ".
+                      "ORDER BY Trick.id ASC");
+    $trickNR   = 1;
+    $lasttrick = DB_get_max_trickid($gameid);
+
+    /* output tricks */
+    while($r = DB_fetch_array($result))
+      {
+       $trick=$r[0];
+       if($trick!=$lasttrick)
+         echo "  <li onclick=\"hl('$trickNR');\" class=\"old\"><a href=\"#\">"._('Trick')." $trickNR</a></li>\n";
+       else if($trick==$lasttrick)
+         echo "  <li onclick=\"hl('$trickNR');\" class=\"current\"><a href=\"#\">"._('Trick')." $trickNR</a></li>\n";
+       $trickNR++;
+      }
+
+    /* if game is over, also output link to Score tab */
+    if($mystatus=='gameover' && DB_get_game_status_by_gameid($gameid)=='gameover' )
+      echo "  <li onclick=\"hl('13');\" class=\"current\"><a href=\"#\">"._('Score')."</a></li>\n";
+
+    /* output previous/next buttons */
+    echo "  <li onclick=\"hl_prev();\" ><button>"._('prev')."</button></li>\n";
+    echo "  <li onclick=\"hl_next();\" ><button>"._('next')."</button></li>\n";
+
+    echo "</ul>\n\n";
+
+    break;
+  default:
+  }
+
+
 /******************************************************************************
  * Output tricks played, table, messages, and cards (depending on game status)
  ******************************************************************************/
@@ -316,7 +416,68 @@ if($mystatus!='gameover')
 echo "<form action=\"index.php?action=game&amp;me=$me\" method=\"post\">\n";
 
 /* display the table and the names */
-display_table();
+display_table_begin();
+
+
+/******************************
+ * Output pre-trick if needed *
+ ******************************/
+
+switch($mystatus)
+  {
+  case 'start':
+    break;
+  case 'init':
+  case 'check':
+    /* output sickness of other playes, in case they already selected and are sitting in front of the current player */
+    echo "\n<div class=\"tricks\">\n";
+    echo "    <div class=\"trick\" id=\"trick0\">\n";
+
+    for($pos=1;$pos<5;$pos++)
+      {
+       $usersick   = DB_get_sickness_by_pos_and_gameid($pos,$gameid);
+       $userid     = DB_get_userid('gameid-position',$gameid,$pos);
+       $userstatus = DB_get_hand_status_by_userid_and_gameid($userid,$gameid);
+
+       if($userstatus=='start' || $userstatus=='init')
+         echo " <div class=\"vorbehalt".($pos-1)."\"> still needs <br />to decide </div>\n"; /* show this to everyone */
+       else
+         if($usersick!=NULL) /* in the init-phase we only showed players with $pos<$mypos, now we can show all */
+           echo " <div class=\"vorbehalt".($pos-1)."\"> sick </div>\n";
+         else
+           echo " <div class=\"vorbehalt".($pos-1)."\"> healthy </div>\n";
+      }
+    echo "    </div>\n  </div>\n";  /* end div trick, end li trick , end tricks*/
+    /* end displaying sickness */
+    break;
+  case 'poverty':
+    /* output pre-game trick in case user reloads,
+     * only needs to be done when a team has been formed */
+    if($myparty=='re' || $myparty=='contra')
+      {
+       echo "\n<ul class=\"tricks\">\n";
+
+       $mygametype =  DB_get_gametype_by_gameid($gameid);
+
+       echo "    <div class=\"trick\" id=\"trick0\">\n";
+
+       /* get information so show the cards that have been handed over in a poverty game */
+       output_exchanged_cards();
+
+       echo "    </div>\n  </div>\n\n";  /* end div trick, end li trick , end ul tricks */
+      }
+    /* end output pre-game trick */
+    break;
+  case 'play':
+  case 'gameover':
+
+    /* taken care further down */
+    break;
+  default:
+  }
+
+
+
 
 /* mystatus gets the player through the different stages of a game.
  * start:    does the player want to play?
@@ -333,6 +494,13 @@ display_table();
  */
 $cards_status = CARDS_EMPTY;
 
+/* Also collect message that should be displayed to the user, so that we can show
+ * them after showing the table. This makes the html flow more consistent and easier
+ * tournament change layouts, especially for smaller displays, e.g. mobile phones
+ */
+$messages = array();
+
+
 switch($mystatus)
   {
   case 'start':
@@ -400,28 +568,6 @@ switch($mystatus)
      ***************************/
     if(!myisset('solo','wedding','poverty','nines','lowtrump') )
       {
-       /* output sickness of other playes, in case they already selected and are sitting in front of the current player */
-       echo "\n<ul class=\"tricks\">\n";
-       echo "  <li onclick=\"hl('0');\" class=\"current\"><a href=\"#\">Pre</a>\n".
-         "    <div class=\"trick\" id=\"trick0\">\n";
-
-       for($pos=1;$pos<5;$pos++)
-         {
-           $usersick   = DB_get_sickness_by_pos_and_gameid($pos,$gameid);
-           $userid     = DB_get_userid('gameid-position',$gameid,$pos);
-           $userstatus = DB_get_hand_status_by_userid_and_gameid($userid,$gameid);
-
-           if($userstatus=='start' || $userstatus=='init')
-             echo " <div class=\"vorbehalt".($pos-1)."\"> still needs <br/ > to decide </div>\n"; /* show this to everyone */
-           else
-             if($usersick!=NULL && $pos<=$mypos ) /* only show this for people sitting before the player */
-               echo " <div class=\"vorbehalt".($pos-1)."\"> sick </div>\n";
-             else if($usersick==NULL && $pos<=$mypos)
-               echo " <div class=\"vorbehalt".($pos-1)."\"> healthy </div>\n";
-         }
-       echo "    </div>\n  </li>\n</ul>\n";  /* end div trick, end li trick , end tricks*/
-       /* end displaying sickness */
-
        $mycards = DB_get_hand($me);
        output_check_for_sickness($me,$mycards);
 
@@ -441,8 +587,8 @@ switch($mystatus)
 
        if($Nsickness>1)
          {
-           echo "<p class=\"message\"> You selected more than one sickness, please go back ".
-             "and answer the <a href=\"$INDEX?action=game&amp;me=$me&amp;in=yes\">question</a> again.</p>";
+           $messages[] = "You selected more than one sickness, please go back ".
+             "and answer the <a href=\"$INDEX?action=game&amp;me=$me&amp;in=yes\">question</a> again.";
 
            $card_status = CARDS_SHOW;
 
@@ -451,7 +597,7 @@ switch($mystatus)
        else
          {
            /* everything is ok, save what user said and proceed */
-           echo "<p class=\"message\">Processing what you selected in the last step...";
+           $messages[] = "Processing what you selected in the last step...";
 
            /* check if this sickness needs to be handled first */
            $gametype    = DB_get_gametype_by_gameid($gameid);
@@ -465,7 +611,7 @@ switch($mystatus)
                DB_set_solo_by_hash($me,$_REQUEST['solo']);
                DB_set_sickness_by_hash($me,'solo');
 
-               echo "<br />Seems like you want to play a {$_REQUEST['solo']} solo. Got it.<br />\n";
+               $messages[] = "<br />Seems like you want to play a {$_REQUEST['solo']} solo. Got it.<br />\n";
 
                if($gametype == 'solo' && $startplayer<$mypos)
                  {}/* do nothing, since someone else already is playing solo */
@@ -482,33 +628,31 @@ switch($mystatus)
            else if($_REQUEST['wedding'] == 'yes')
              {
                /* silent solo is set further down */
-               echo _("Ok, you don't want to play a silent solo...wedding was chosen.")."<br />\n";
+               $messages[] = _("Ok, you don't want to play a silent solo...wedding was chosen.")."<br />\n";
                DB_set_sickness_by_hash($me,'wedding');
              }
            else if($_REQUEST['poverty'] == 'yes')
              {
-               echo _("Don't think you can win with just a few trump...? Ok, poverty chosen.")." <br />\n";
+               $messages[] = _("Don't think you can win with just a few trump...? Ok, poverty chosen.")." <br />\n";
                DB_set_sickness_by_hash($me,'poverty');
              }
            else if($_REQUEST['nines'] == 'yes')
              {
-               echo _("What? You just don't want to play a game because you have a few nines? Well, if no one".
+               $messages[] = _("What? You just don't want to play a game because you have a few nines? Well, if no one".
                       " is playing solo, this game will be canceled.")."<br />\n";
                DB_set_sickness_by_hash($me,'nines');
              }
            else if($_REQUEST['lowtrump'] == 'yes')
              {
                if($RULES['lowtrump']=='cancel')
-                 echo _("What? You just don't want to play a game because you have low trump? Well, if no one".
+                 $messages[] = _("What? You just don't want to play a game because you have low trump? Well, if no one".
                         " is playing solo, this game will be canceled.")."<br />\n";
                else
-                 echo _("Don't think you can win with low trumps...? Ok, poverty chosen.")." <br />.<br />\n";
+                 $messages[] = _("Don't think you can win with low trumps...? Ok, poverty chosen.")." <br />.<br />\n";
 
                DB_set_sickness_by_hash($me,'lowtrump');
              }
 
-           echo "</p>\n";
-
            /* move on to the next stage*/
            DB_set_hand_status_by_hash($me,'check');
            $mystatus='check';
@@ -520,30 +664,7 @@ switch($mystatus)
      * this can therefore only be handled once all players finished the last stage
      */
 
-    /* output sickness of other playes, in case they already selected and are sitting in front of the current player */
-    echo "\n<ul class=\"tricks\">\n";
-    echo "  <li onclick=\"hl('0');\" class=\"current\"><a href=\"#\">Pre</a>\n".
-      "    <div class=\"trick\" id=\"trick0\">\n";
-
-    for($pos=1;$pos<5;$pos++)
-      {
-       $usersick   = DB_get_sickness_by_pos_and_gameid($pos,$gameid);
-       $userid     = DB_get_userid('gameid-position',$gameid,$pos);
-       $userstatus = DB_get_hand_status_by_userid_and_gameid($userid,$gameid);
-
-       if($userstatus=='start' || $userstatus=='init')
-         echo " <div class=\"vorbehalt".($pos-1)."\"> still needs <br />to decide </div>\n"; /* show this to everyone */
-       else
-         if($usersick!=NULL) /* in the init-phase we only showed players with $pos<$mypos, now we can show all */
-           echo " <div class=\"vorbehalt".($pos-1)."\"> sick </div>\n";
-         else
-           echo " <div class=\"vorbehalt".($pos-1)."\"> healthy </div>\n";
-      }
-    echo "    </div>\n  </li>\n</ul>\n";  /* end div trick, end li trick , end tricks*/
-    /* end displaying sickness */
-
-    echo "<div class=\"message\">\n";
-    echo '<p> '._('Checking if someone else selected solo, nines, wedding or poverty.').'</p>';
+    $messages[] = _('Checking if someone else selected solo, nines, wedding or poverty.');
 
     /* check if everyone has reached this stage */
     $userids = DB_get_all_userid_by_gameid($gameid);
@@ -561,10 +682,10 @@ switch($mystatus)
 
     if(!$ok)
       {
-       echo '<p>'._('This step can only be handled after everyone finished the last step. '.
+       $messages[] = _('This step can only be handled after everyone finished the last step. '.
          'Seems like this is not the case, so you need to wait a bit... '.
          'you will get an email once that is the case, please use the link in '.
-         'that email to continue the game.').'</p></div>';
+         'that email to continue the game.');
 
        /* display cards, if player was just at the init-phase he will still see the cards from there
         * we can put this one here, since the last player to finish the init state won't get here and
@@ -581,7 +702,7 @@ switch($mystatus)
         * are playing, in case there are any solos this already
         * will have the correct information in it */
 
-       echo '<p>'._('Ok, everyone is done... figuring out what kind of game we are playing.').'</p>';
+       $messages[] = _('Ok, everyone is done... figuring out what kind of game we are playing.');
 
        $gametype    = DB_get_gametype_by_gameid($gameid);
        $startplayer = DB_get_startplayer_by_gameid($gameid);
@@ -627,8 +748,8 @@ switch($mystatus)
                /* update game status */
                cancel_game('nines',$gameid);
 
-               echo "<p>The game has been canceled because ".DB_get_name('userid',$cancel).
-                 " has five or more nines and nobody is playing solo.</p>\n";
+               $messages[] = "The game has been canceled because ".DB_get_name('userid',$cancel).
+                 " has five or more nines and nobody is playing solo.";
              }
            else if ($cancelsick == 'lowtrump')
              {
@@ -640,8 +761,8 @@ switch($mystatus)
                /* update game status */
                cancel_game('lowtrump',$gameid);
 
-               echo "<p>The game has been canceled because ".DB_get_name('userid',$cancel).
-                 " has low trump and nobody is playing solo.</p>\n";
+               $messages[] = "The game has been canceled because ".DB_get_name('userid',$cancel).
+                 " has low trump and nobody is playing solo.";
              };
 
            $userids = DB_get_all_userid_by_gameid($gameid);
@@ -650,7 +771,6 @@ switch($mystatus)
                mymail($user,$gameid, GAME_CANCELED, $email_message);
              }
 
-           echo "</div>\n"; /* end div message */
            break;
          }
        else if($poverty==1) /* one person has poverty */
@@ -694,7 +814,7 @@ switch($mystatus)
            $gametype = 'wedding';
          };
        /* now the gametype is set correctly in the database */
-       echo '<p>'._('Got it').' :)</p>';
+       $messages[] = _('Got it').' :)';
 
        /* loop over all players, set re/contra if possible and start the game if possible */
        $userids = DB_get_all_userid_by_gameid($gameid);
@@ -788,7 +908,7 @@ switch($mystatus)
                  }
              }
            else
-             echo "Please, <a href=\"$INDEX?action=game&amp;me=$me\">start</a> the game.<br />\n";
+             $messages[] = "Please, <a href=\"$INDEX?action=game&amp;me=$me\">start</a> the game.<br />\n";
          }
        else
          {
@@ -798,7 +918,7 @@ switch($mystatus)
 
            $whoid = DB_get_userid('gameid-position',$gameid,$who);
            if($whoid==$myid)
-             echo "Please, <a href=\"$INDEX?action=game&amp;me=$me\">start</a> the game.<br /\n";
+             $messages[] =  "Please, <a href=\"$INDEX?action=game&amp;me=$me\">start</a> the game.<br /\n";
            else
              {
                $whohash = DB_get_hash_from_game_and_pos($gameid,$who);
@@ -813,7 +933,6 @@ switch($mystatus)
                  }
              }
          }
-       echo "</div>\n";
        $card_status = CARDS_SHOW;
       }
     break;
@@ -856,24 +975,6 @@ switch($mystatus)
          };
       }
 
-    /* output pre-game trick in case user reloads,
-     * only needs to be done when a team has been formed */
-    if($myparty=='re' || $myparty=='contra')
-      {
-       echo "\n<ul class=\"tricks\">\n";
-
-       $mygametype =  DB_get_gametype_by_gameid($gameid);
-
-       echo "  <li onclick=\"hl('0');\" class=\"current\"><a href=\"#\">Pre</a>\n".
-         "    <div class=\"trick\" id=\"trick0\">\n";
-
-       /* get information so show the cards that have been handed over in a poverty game */
-       output_exchanged_cards();
-
-       echo "    </div>\n  </li>\n</ul>\n\n";  /* end div trick, end li trick , end ul tricks */
-      }
-    /* end output pre-game trick */
-
     /* get hand */
     $mycards = DB_get_hand($me);
 
@@ -907,7 +1008,7 @@ switch($mystatus)
              }
          }
        else
-         echo "<div class=\"message\">Please, <a href=\"$INDEX?action=game&amp;me=$me\">start</a> the game.</div>\n";
+         $messages[]= "Please, <a href=\"$INDEX?action=game&amp;me=$me\">start</a> the game.";
       }
 
     /* the following is part A) of what needs to be done */
@@ -940,7 +1041,7 @@ switch($mystatus)
                  }
              }
            echo "<a href=\"index.php?action=game&amp;me=$me&amp;trump=no\">No way</a> <br />\n";
-           echo "</div><div>\n";
+           echo "</div>\n";
          }
        break;
       }
@@ -986,7 +1087,7 @@ switch($mystatus)
                /* update game status */
                cancel_game('trump',$gameid);
 
-               echo "<p class=\"message\";>Game ".DB_format_gameid($gameid)." has been canceled.<br /><br /></p>";
+               $messages[] = "Game ".DB_format_gameid($gameid)." has been canceled.";
                break;
              }
            else
@@ -1083,7 +1184,7 @@ switch($mystatus)
                        $next=2;
 
                    if($mypos+$next>4)
-                     echo "<div class=\"message\">Error in poverty, please contact the Admin</div>\n";
+                     $messages[] = "Error in poverty, please contact the Admin";
 
                    $userhash = DB_get_hash_from_game_and_pos($gameid,$mypos+$next);
                    $userid   = DB_get_userid('hash',$userhash);
@@ -1096,10 +1197,9 @@ switch($mystatus)
                    mymail($userid,$gameid, GAME_DPOVERTY, $email_message);
                  }
              }
-           echo "<div class=\"message\">Please, <a href=\"$INDEX?action=game&amp;me=$me\">continue</a> here.</div>\n";
+           $messages[] = "Please, <a href=\"$INDEX?action=game&amp;me=$me\">continue</a> here";
          }
       }
-    echo "</div>\n";
     break;
 
   case 'play':
@@ -1112,19 +1212,19 @@ switch($mystatus)
     switch($gamestatus)
       {
       case 'cancel-noplay':
-       echo "<div class=\"message\"><p>The game has been canceled due to the request of one player.</p><p>If this was a mistake all 4 players need to send an Email to $ADMIN_NAME at $ADMIN_EMAIL requesting that the game should be restarted.</p></div>";
+       $messages[] = "The game has been canceled due to the request of one player.</p><p>If this was a mistake all 4 players need to send an Email to $ADMIN_NAME at $ADMIN_EMAIL requesting that the game should be restarted.";
        break;
       case 'cancel-timedout':
-       echo "<div class=\"message\"><p>The game has been canceled because one player wasn't responding.</p><p>If this was a mistake all 4 players need to send an Email to $ADMIN_NAME at $ADMIN_EMAIL requesting that the game should be restarted.</p></div>";
+       $messages[] = "The game has been canceled because one player wasn't responding.<br />If this was a mistake all 4 players need to send an Email to $ADMIN_NAME at $ADMIN_EMAIL requesting that the game should be restarted.";
        break;
       case 'cancel-nines':
-       echo "<div class=\"message\"><p>The game has been canceled because one player had too many nines.</p></div>";
+       $messages[] = "The game has been canceled because one player had too many nines.";
        break;
       case 'cancel-lowtrump':
-       echo "<div class=\"message\"><p>The game has been canceled because one player had low trump.</p></div>";
+       $messages[] = "The game has been canceled because one player had low trump.";
        break;
       case 'cancel-trump':
-       echo "<div class=\"message\"><p>The game has been canceled because nobody wanted to take the trump.</p></div>";
+       $messages[] = "The game has been canceled because nobody wanted to take the trump.";
        break;
       }
     /* for these two types, we shouldn't show the cards, since we might want to restart the game */
@@ -1197,8 +1297,8 @@ switch($mystatus)
     /* has the game started? No, then just wait here...*/
     if($gamestatus == 'pre')
       {
-       echo '<p class="message"> '._('You finished the setup, but not everyone else finished it... '.
-         'You need to wait for the others. Just wait for an email.').' </p>';
+       $messages[] = _('You finished the setup, but not everyone else finished it... '.
+         'You need to wait for the others. Just wait for an email.');
 
        $card_status = CARDS_SHOW;
 
@@ -1248,7 +1348,7 @@ switch($mystatus)
     $pos  = DB_get_startplayer_by_gameid($gameid)-1;
     $firstcard = ''; /* first card in a trick */
 
-    echo "\n<ul class=\"tricks\">\n";
+    echo "\n<div class=\"tricks\">\n";
 
     /* output vorbehalte */
     $mygametype = DB_get_gametype_by_gameid($gameid);
@@ -1256,15 +1356,15 @@ switch($mystatus)
     if($mygametype != 'normal') /* only show when needed */
       if(!( $mygametype == 'solo' && $mygamesolo == 'silent') )
        {
-         echo "  <li onclick=\"hl('0');\" class=\"current\"><a href=\"#\">Pre</a>\n".
-           "    <div class=\"trick\" id=\"trick0\">\n";
+         echo "    <div class=\"trick\" id=\"trick0\">\n";
 
          /* get information so show the cards that have been handed over in a poverty game */
          output_exchanged_cards();
 
-         echo "    </div>\n  </li>\n";  /* end div trick, end li trick */
+         echo "    </div>\n";  /* end div trick, end li trick */
        }
 
+
     /* output tricks */
     while($r = DB_fetch_array($result))
       {
@@ -1294,15 +1394,13 @@ switch($mystatus)
            if($trick!=$lasttrick)
              {
                /* start of an old trick? */
-               echo "  <li onclick=\"hl('$trickNR');\" class=\"old\"><a href=\"#\">"._('Trick')." $trickNR</a>\n".
-                 "    <div class=\"trick\" id=\"trick".$trickNR."\">\n".
+               echo  "    <div class=\"trick\" id=\"trick".$trickNR."\">\n".
                  "      <img class=\"arrow\" src=\"pics/arrow".($pos-1).".png\" alt=\"table\" />\n";
              }
            else if($trick==$lasttrick)
              {
                /* start of a last trick? */
-               echo "  <li onclick=\"hl('$trickNR');\" class=\"current\"><a href=\"#\">"._('Trick')." $trickNR</a>\n".
-                 "    <div class=\"trick\" id=\"trick".$trickNR."\">\n".
+               echo "    <div class=\"trick\" id=\"trick".$trickNR."\">\n".
                  "      <img class=\"arrow\" src=\"pics/arrow".($pos-1).".png\" alt=\"table\" />\n";
              };
 
@@ -1326,7 +1424,7 @@ switch($mystatus)
        if($seq==4)
          {
            $winner    = get_winner($play,$gametype); /* returns the position */
-           echo "    </div>\n  </li>\n";  /* end div trick, end li trick */
+           echo "    </div>\n";  /* end div trick, end li trick */
          }
       }
 
@@ -1536,7 +1634,7 @@ switch($mystatus)
                if($winner>0)
                  DB_query("UPDATE Trick SET winner='$winner' WHERE id='$trickid'");
                else
-                 echo "ERROR during scoring";
+                 $messages[] = "ERROR during scoring";
 
                if($debug)
                  echo "DEBUG: position $winner won the trick <br />";
@@ -1564,8 +1662,7 @@ switch($mystatus)
            $pos = DB_get_pos_by_hash($me);
            if($sequence==1)
              {
-               echo "  <li onclick=\"hl('".($tricknr)."');\" class=\"current\"><a href=\"#\">"._('Trick').' '.($tricknr)."</a>\n".
-                 "    <div class=\"trick\" id=\"trick".($tricknr)."\">\n".
+               echo "    <div class=\"trick\" id=\"trick".($tricknr)."\">\n".
                  "      <img class=\"arrow\" src=\"pics/arrow".($pos-1).".png\" alt=\"table\" />\n";
              }
 
@@ -1577,7 +1674,7 @@ switch($mystatus)
              echo "\n        <span class=\"comment\"> ".$comment."</span>\n";
            echo "      </div>\n";
 
-           echo "    </div>\n  </li>\n";  /* end div trick, end li trick */
+           echo "    </div>\n";  /* end div trick, end li trick */
 
            /*check if we still have cards left, else set status to gameover */
            if(sizeof(DB_get_hand($me))==0)
@@ -1859,22 +1956,21 @@ switch($mystatus)
          }
        else
          {
-           echo "can't find that card?! <br />\n";
+           $messages[] = "can't find that card?!";
          }
       }
     else if(myisset('card') && !$myturn )
       {
-       echo _("please wait until it's your turn!")."<br />\n";
+       $messages[] = _("please wait until it's your turn!");
       }
 
     if($seq!=4 && $trickNR>=1 && !(myisset('card') && $myturn) )
-      echo "    </div>\n  </li>\n";  /* end div trick, end li trick */
+      echo "    </div>\n";  /* end div trick, end li trick */
 
     /* display points in case game is over */
     if($mystatus=='gameover' && DB_get_game_status_by_gameid($gameid)=='gameover' )
       {
-       echo "  <li onclick=\"hl('13');\" class=\"current\"><a href=\"#\">"._('Score')."</a>\n".
-         "    <div class=\"trick\" id=\"trick13\">\n";
+       echo "    <div class=\"trick\" id=\"trick13\">\n";
        /* add pic for re/contra
         "      <img class=\"arrow\" src=\"pics/arrow".($pos-1).".png\" alt=\"table\" />\n";*/
 
@@ -1900,39 +1996,37 @@ switch($mystatus)
                           " LEFT JOIN Card ON Card.id=Hand_Card.card_id".
                           " WHERE Hand.game_id='$gameid'".
                           " GROUP BY Hand.party" );
-       echo "<div class=\"total\">\n  Totals:<br />\n";
+       echo "    <div class=\"total\">\n  Totals:<br />\n";
        while( $r = DB_fetch_array($result))
-         echo "  ".$r[0]." ".$r[1]."<br />\n";
+         echo "      ".$r[0]." ".$r[1]."<br />\n";
 
        $queryresult = DB_query("SELECT timediff(mod_date,create_date) ".
                                " FROM Game WHERE id='$gameid'");
        $r = DB_fetch_array($queryresult);
-       echo "  <p>This game took ".$r[0]." hours.</p>\n";
+       echo "      <p>This game took ".$r[0]." hours.</p>\n";
 
-       echo "  <div class=\"re\">\n   Points Re: <br />\n";
+       echo "      <div class=\"re\">\n   Points Re: <br />\n";
        $queryresult = DB_query("SELECT score FROM Score ".
                                "  WHERE game_id=$gameid AND party='re'".
                                " ");
        while($r = DB_fetch_array($queryresult) )
-         echo "   ".$r[0]."<br />\n";
-       echo "  </div>\n";
+         echo "       ".$r[0]."<br />\n";
+       echo "      </div>\n";
 
-       echo "  <div class=\"contra\">\n   Points Contra: <br />\n";
+       echo "      <div class=\"contra\">\n   Points Contra: <br />\n";
        $queryresult = DB_query("SELECT score FROM Score ".
                                "  WHERE game_id=$gameid AND party='contra'".
                                " ");
        while($r = DB_fetch_array($queryresult) )
-         echo "   ".$r[0]."<br />\n";
-       echo "  </div>\n";
+         echo "       ".$r[0]."<br />\n";
+       echo "      </div>\n";
 
-       echo "</div>\n";
+       echo "    </div>\n";
 
-       echo "    </div>\n  </li>\n";  /* end div trick, end li trick */
+       echo "    </div>\n";  /* end div trick, end li trick */
       }
 
-    echo "  <li onclick=\"hl_prev();\" class=\"old\"><a href=\"#\">"._('prev')."</a></li>\n";
-    echo "  <li onclick=\"hl_next();\" class=\"old\"><a href=\"#\">"._('next')."</a></li>\n";
-    echo "</ul>\n"; /* end ul tricks*/
+    echo "</div>\n"; /* end ul tricks*/
 
     if($myturn && !myisset('card') && $mystatus=='play' )
       {
@@ -1965,6 +2059,9 @@ switch($mystatus)
     myerror("error in testing the status");
   } /*end of output: tricks, table, messages, card */
 
+/* display the 2nd half of table and the names */
+display_table_end();
+
 /**************
  * show cards *
  **************/
@@ -1972,6 +2069,7 @@ switch($mystatus)
 $mycards = DB_get_hand($me);
 $mycards = mysort($mycards,$gametype);
 
+echo "\n";
 echo '<div class="mycards">';
 switch ($card_status) {
  case CARDS_SHOW:
@@ -2090,40 +2188,26 @@ switch ($card_status) {
  default:
    break;
  }
-echo '</div>';
+echo "</div>\n";
 
+/*****************
+ * show messages *
+ *****************/
+
+if( sizeof($messages) )
+  {
+    echo "\n<div class=\"message\">\n";
+    foreach($messages as $message)
+      {
+       echo "  <div>$message <div>close</div> </div>\n";
+      }
+    echo "</div>\n\n";
+  }
 
 /***********************************************
  * Comments, re/contra calls, user menu
  ***********************************************/
 
-/* output other games where it is the users turn
- * make sure that the people looking at old games don't see the wrong games here
- */
-if( $gamestatus != 'gameover' )
-  {
-    /* game isn't over, only valid user can get here, so show menu */
-    display_user_menu($myid);
-  }
-else if( $origmystatus != 'gameover' )
-  {
-    /* user just played the very last card, game is now over, it's still ok to show the menu though */
-    display_user_menu($myid);
-  }
-else if( $mystatus == 'gameover'
-        && isset($_SESSION['id']) )
-  {
-    /* user is looking at someone else's game, show the menu for the correct user */
-    display_user_menu($_SESSION['id']);
-  }
-else
-  {
-    echo "<div class=\"usermenu\">\n";
-    echo "It's your turn in these games:<br />\n";
-    echo "Please log in to see this information.\n";
-    echo "</div>\n";
-  }
-
 /*
  * display gameinfo: re/contra, comment-box, play-card button, games played by others
  */
@@ -2168,7 +2252,7 @@ if(sizeof($other_game_ids)>0 && $mystatus=='gameover')
     echo "  </p>\n";
   }
 
-echo "</div>\n"; /* end gameinfo */
+echo "</div>\n\n"; /* end gameinfo */
 
 /* make sure that we don't show the notes to the wrong person
  * (e.g. other people looking at an old game)