Bug 1153100: add mozreview's table to bug-modal

2 files changed, 26 insertions, 3 deletions

diff --git a/extensions/MozReview/web/js/mozreview.js b/extensions/MozReview/web/js/mozreview.js
index 714863a74..2d5fdb0fa 100644
--- a/extensions/MozReview/web/js/mozreview.js
+++ b/extensions/MozReview/web/js/mozreview.js
@@ -46,7 +46,7 @@ MozReview.elapsedTime = function(d) {
 };
 
 MozReview.getReviewRequest = function() {
-    var hostUrl = $('.mozreview-table').data('mozreviewUrl');
+    var hostUrl = $('.mozreview-requests').data('mozreviewUrl');
     var tr = $('<tr/>');
     var td = $('<td/>');
 
@@ -93,7 +93,8 @@ MozReview.getReviewRequest = function() {
         return trCommit;
     }
 
-    $('.mozreview-hide-discarded-link').click(function() {
+    $('.mozreview-hide-discarded-link').click(function(event) {
+        event.preventDefault();
         if ($('.bz_default_hidden.mozreview-discarded-request').length) {
             $('.mozreview-discarded-request').removeClass('bz_default_hidden');
             $('.mozreview-discarded-action').text('Hide');
@@ -101,7 +102,6 @@ MozReview.getReviewRequest = function() {
             $('.mozreview-discarded-request').addClass('bz_default_hidden');
             $('.mozreview-discarded-action').text('Show');
         }
-        return false;
     });
 
     $('.mozreview-request').each(function() {
diff --git a/extensions/MozReview/web/style/mozreview.css b/extensions/MozReview/web/style/mozreview.css
index 5c021ea96..8efea8950 100644
--- a/extensions/MozReview/web/style/mozreview.css
+++ b/extensions/MozReview/web/style/mozreview.css
@@ -52,3 +52,26 @@
 .mozreview-child-request-summary {
     text-indent: 1em;
 }
+
+/* bug-modal specific */
+
+#module-mozreview-requests .module-content {
+    padding: 0;
+}
+
+.bug_modal .mozreview-table {
+    width: 100%;
+}
+
+.bug_modal .mozreview-request td {
+    padding-left: 8px;
+}
+
+.bug_modal .mozreview-requests th {
+    text-align: left;
+    padding-left: 8px;
+}
+
+.bug_modal .mozreview-hide-discarded-row {
+    padding: 4px;
+}