From c590905a92105a8f02034fb412e8554ead7d35b2 Mon Sep 17 00:00:00 2001
From: Damien Regad <dregad@mantisbt.org>
Date: Wed, 26 Nov 2014 19:01:45 +0100
Subject: [PATCH] HTML/CSS fixes in bug_revision_view_page.php

---
 bug_revision_view_page.php | 83 +++++++++++++++++++-------------------
 1 file changed, 41 insertions(+), 42 deletions(-)

diff --git a/bug_revision_view_page.php b/bug_revision_view_page.php
index 561db6cc56..dcd9e44c1f 100644
--- a/bug_revision_view_page.php
+++ b/bug_revision_view_page.php
@@ -135,57 +135,56 @@ function show_revision( array $p_revision ) {
 	$t_by_string = sprintf( lang_get( 'revision_by' ), string_display_line( date( config_get( 'normal_date_format' ), $p_revision['timestamp'] ) ), prepare_user_name( $p_revision['user_id'] ) );
 
 ?>
-<tr class="spacer"><td><a id="revision-<?php echo $p_revision['id'] ?>"></a></td></tr>
-
-<tr>
-<th class="category"><?php echo lang_get( 'revision' ) ?></th>
-<td colspan="2"><?php echo $t_by_string ?></td>
-<td class="center" width="5%">
-<?php if( $s_can_drop ) {
-	print_bracket_link( 'bug_revision_drop.php?id=' . $p_revision['id'] . $s_drop_token, lang_get( 'revision_drop' ) );
-} ?>
-</tr>
-
-<tr>
-<th class="category"><?php echo $t_label ?></th>
-<td colspan="3"><?php echo string_display_links( $p_revision['value'] ) ?></td>
-</tr>
-
-	<?php
-}
-
-html_page_top( bug_format_summary( $t_bug_id, SUMMARY_CAPTION ) );
 
-print_recently_visited();
+		<tr class="spacer"><td><a id="revision-<?php echo $p_revision['id'] ?>"></a></td></tr>
 
+		<tr>
+			<th class="category"><?php echo lang_get( 'revision' ) ?></th>
+			<td colspan="2"><?php echo $t_by_string ?></td>
+			<td class="center" width="5%">
+<?php
+	if( $s_can_drop ) {
+		print_bracket_link( 'bug_revision_drop.php?id=' . $p_revision['id'] . $s_drop_token, lang_get( 'revision_drop' ) );
+	}
 ?>
+		</tr>
 
-<br/>
-<div class="table-container">
-<table class="width100" cellspacing="1">
-
-<tr>
-<td class="form-title" colspan="2"><?php echo lang_get( 'view_revisions' ), ': ', $t_title ?></td>
-<td class="right" colspan="2">
+		<tr>
+			<th class="category"><?php echo $t_label ?></th>
+			<td colspan="3"><?php echo string_display_links( $p_revision['value'] ) ?></td>
+		</tr>
 <?php
-if( !$f_bug_id && !$f_bugnote_id ) {
-	print_bracket_link( '?bug_id=' . $t_bug_id, lang_get( 'all_revisions' ) );
 }
-print_bracket_link( 'view.php?id=' . $t_bug_id, lang_get( 'back_to_issue' ) );
-?>
-</td>
-</tr>
 
-<tr>
-<th class="category" width="15%"><?php echo lang_get( 'summary' ) ?></th>
-<td colspan="3"><?php echo bug_format_summary( $t_bug_id, SUMMARY_FIELD ) ?></td>
-</tr>
+html_page_top( bug_format_summary( $t_bug_id, SUMMARY_CAPTION ) );
+
+print_recently_visited();
 
-<?php foreach( $t_bug_revisions as $t_rev ) {
-	show_revision( $t_rev );
-} ?>
+?>
 
-</table>
+
+<div id="bug-revision-div" class="table-container">
+	<h2><?php echo lang_get( 'view_revisions' ), ': ', $t_title ?></h2>
+	<div class="section-link">
+		<?php
+			if( !$f_bug_id && !$f_bugnote_id ) {
+				print_bracket_link( '?bug_id=' . $t_bug_id, lang_get( 'all_revisions' ) );
+			}
+			print_bracket_link( 'view.php?id=' . $t_bug_id, lang_get( 'back_to_issue' ) );
+		?>
+	</div>
+
+	<table>
+		<tr>
+			<th class="category" width="15%"><?php echo lang_get( 'summary' ) ?></th>
+			<td colspan="3"><?php echo bug_format_summary( $t_bug_id, SUMMARY_FIELD ) ?></td>
+		</tr>
+		<?php
+			foreach( $t_bug_revisions as $t_rev ) {
+				show_revision( $t_rev );
+			}
+		?>
+	</table>
 </div>
 
 <?php