Fixed line spacing for backtrace, added blank space for empty line numbers

Nathan Broadbent
1 parent 9408ceef
Showing 2 changed files with 14 additions and 4 deletions Show diff stats
app/views/notices/_backtrace.html.haml
public/stylesheets/application.css
@@ -4,8 +4,11 @@
       %th
         %ul.line-numbers
           - lines.each do |line|
-            %li= (@app.github_url? && Notice.in_app_backtrace_line?(line)) ? link_to_github(@app, line, line['number']) : line['number']
-
+            %li
+              - if line['number'].present?
+                = (@app.github_url? && Notice.in_app_backtrace_line?(line)) ? link_to_github(@app, line, line['number']) : line['number']
+              - else
+                &nbsp;
       %td
         %ul.lines
           - lines.each do |line|
@@ -691,16 +691,23 @@ table.backtrace td {
   background-color: #222 !important;
 }
 table.backtrace ul.line-numbers li {
+  border-bottom: 0;
   text-align: right;
+  font-size: 13px;
+  line-height: 21px;
 }
+
 table.backtrace ul.lines {
   padding: 10px 8px;
   overflow: auto;
 }
-table.backtrace li {
+table.backtrace ul.lines li {
+  border-bottom: 1px solid #2B2B2B;
+  font-size: 13px;
+  line-height: 20px;
   white-space: nowrap;
 }
-table.backtrace li.in-app {
+table.backtrace ul.lines li.in-app {
   color: #2adb2e;
   background-color: #2f2f2f;
 }