Commit 60425256e6e0d19ffbe0432494ad76c8c7dbc697
1 parent
a844dead
Exists in
master
and in
1 other branch
Removed fixed width from error message width, so the table does not exceed wrapper #content block.
Unresolved errors link goes to application page (so I see only errors from selected application). Fixed "no errors" message (now it spreads over whole table columns).
Showing
3 changed files
with
3 additions
and
4 deletions
Show diff stats
app/views/apps/index.html.haml
| ... | ... | @@ -15,7 +15,7 @@ |
| 15 | 15 | %td.deploy= app.last_deploy_at ? link_to( app.last_deploy_at.to_s(:micro), app_deploys_path(app)) : 'n/a' |
| 16 | 16 | %td.count |
| 17 | 17 | - if app.errs.count > 0 |
| 18 | - = link_to app.errs.unresolved.count, app_errs_path(app) | |
| 18 | + = link_to app.errs.unresolved.count, app_path(app) | |
| 19 | 19 | - else |
| 20 | 20 | \- |
| 21 | 21 | - if @apps.none? | ... | ... |
app/views/errs/_table.html.haml
| ... | ... | @@ -12,7 +12,7 @@ |
| 12 | 12 | %tr{:class => err.resolved? ? 'resolved' : 'unresolved'} |
| 13 | 13 | %td.app |
| 14 | 14 | = link_to err.app.name, app_path(err.app) |
| 15 | - - if(current_page?(:controller => 'errs')) | |
| 15 | + - if current_page?(:controller => 'errs') | |
| 16 | 16 | %span.environment= link_to err.environment, errs_path(environment: err.environment) |
| 17 | 17 | - else |
| 18 | 18 | %span.environment= link_to err.environment, app_path(environment: err.environment) |
| ... | ... | @@ -25,6 +25,6 @@ |
| 25 | 25 | %td.resolve= link_to image_tag("thumbs-up.png"), resolve_app_err_path(err.app, err), :title => "Resolve", :method => :put, :confirm => err_confirm, :class => 'resolve' if err.unresolved? |
| 26 | 26 | - if errs.none? |
| 27 | 27 | %tr |
| 28 | - %td{:colspan => (@app ? 5 : 6)} | |
| 28 | + %td{:colspan => 6} | |
| 29 | 29 | %em No errs here |
| 30 | 30 | = will_paginate @errs, :previous_label => '« Previous', :next_label => 'Next »' | ... | ... |