Merge pull request #214 from tojonmz/job-wrap-firefox

Bug 1067486 - Fix unwanted job wrap when space is available
mozilla · Oct 1, 2014 · 63a39a2 · 63a39a2
2 parents c88d7f6 + 1cca818
commit 63a39a2
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 3 deletions.
diff --git a/webapp/app/css/treeherder.css b/webapp/app/css/treeherder.css
@@ -377,6 +377,7 @@ th-watched-repo {
 .job-btn {
   padding: 0 2px 0 2px;
   vertical-align: 0;
+  line-height: 1.32;
 }
 
 .table-super-condensed thead > tr > th,
@@ -914,7 +915,7 @@ ul.failure-summary-list li .btn-xs {
  .job-btn.btn-red,
  .job-btn.btn-orange,
  .job-btn.btn-purple {
-    margin: 0 -3px 0 0;
+    margin: 0 -3px 1px 0;
  }
 
 .btn-view-nav {

diff --git a/webapp/app/js/directives/clonejobs.js b/webapp/app/js/directives/clonejobs.js
@@ -600,14 +600,12 @@ treeherder.directive('thCloneJobs', [
     };
     var showHideJob = function(job, show) {
         // Note: I was using
-        //    jobEl.style.display = "inline";
         //    jobEl.className += " filter-shown";
         // but that didn't work reliably with the jquery selectors
         // when it came to hiding/showing platforms and groups.  Jquery
         // couldn't detect that I'd added or removed ``filter-shown`` in
         // all cases.  So, while this is a bit slower, it's reliable.
         if (show) {
-            job.css("display", "inline");
             job.addClass("filter-shown");
         } else {
             job.css("display", "none");