Skip to content

Commit

Permalink
Remove notes column from applications table
Browse files Browse the repository at this point in the history
Remove the "Notes" column from the applications table

This takes up valuable UX real estate for essentially one app - notes are still visible on the individual application page.
  • Loading branch information
theseanything committed Jan 3, 2025
1 parent 9fdcda1 commit 8cd9e80
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 13 deletions.
4 changes: 0 additions & 4 deletions app/assets/stylesheets/_tables.scss
Original file line number Diff line number Diff line change
Expand Up @@ -2,10 +2,6 @@
color: govuk-colour("dark-grey");
}

.release__application-notes {
width: 50%;
}

.release__application-link {
display: inline-block;
margin-bottom: 10px;
Expand Down
9 changes: 0 additions & 9 deletions app/views/applications/_applications_table.html.erb
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,6 @@
<% @environments.each do |environment| %>
<%= t.header environment.humanize %>
<% end %>
<%= t.header "Notes" %>
<% end %>

<%= t.body do %>
Expand All @@ -28,13 +27,6 @@
<%= render partial: "shared/badges", locals: { application: application } %>
<% end %>

<% application_notes = capture do %>
<p class="govuk-body-s release__application-notes">
<%= application.status_notes %>
<%= link_to "edit", edit_application_path(application), class: "govuk-link" %>
</p>
<% end %>

<%= t.cell application_name %>

<% application_status = capture do %>
Expand All @@ -58,7 +50,6 @@
<% end %>
<%= t.cell env_deploy %>
<% end %>
<%= t.cell application_notes %>
<% end %>
<% end %>
<% end %>
Expand Down

0 comments on commit 8cd9e80

Please sign in to comment.