diff options
author | Ben Harris <ben@tilde.team> | 2021-09-14 17:25:37 -0400 |
---|---|---|
committer | Ben Harris <ben@tilde.team> | 2021-09-14 17:25:37 -0400 |
commit | d1a328f413f83a33398a3b1682c8a8327184b2b0 (patch) | |
tree | 02cbf8a46cd3de3688843f1b1f987e7247fa43da /app/views/players | |
parent | 68f5f71701f53dffba53b3390b8e01cc23cc3d9c (diff) |
tidy up lists
Diffstat (limited to 'app/views/players')
-rw-r--r-- | app/views/players/_form.html.erb | 2 | ||||
-rw-r--r-- | app/views/players/edit.html.erb | 2 | ||||
-rw-r--r-- | app/views/players/index.html.erb | 30 |
3 files changed, 23 insertions, 11 deletions
diff --git a/app/views/players/_form.html.erb b/app/views/players/_form.html.erb index 170b6d3..5e20a27 100644 --- a/app/views/players/_form.html.erb +++ b/app/views/players/_form.html.erb @@ -1,4 +1,4 @@ -<%= form_with model: @player do |form| %> +<%= form_with model: @player, data: { "turbo" => "false" } do |form| %> <br> <div> <%= form.label :name %><br> diff --git a/app/views/players/edit.html.erb b/app/views/players/edit.html.erb index 49cf670..1504b65 100644 --- a/app/views/players/edit.html.erb +++ b/app/views/players/edit.html.erb @@ -2,7 +2,7 @@ <%= render "form", machine: @player %> -<%= link_to "Delete", player_path(@player), +<%= link_to "Delete #{@player.name}", player_path(@player), method: :delete, data: { confirm: "Are you sure?" } %> diff --git a/app/views/players/index.html.erb b/app/views/players/index.html.erb index 7ad7944..aea4252 100644 --- a/app/views/players/index.html.erb +++ b/app/views/players/index.html.erb @@ -1,13 +1,25 @@ <h1>Pinball Players</h1> + +<table> + <thead> + <th>Name</th> + <th>Paid</th> + <th>Strikes</th> + </thead> + <tbody> + <% @players.order("strikes").each do |player| %> + <tr> + <td><%= link_to player.name, edit_player_path(player) %></td> + <td><% if player.paid %>✓<% end %></td> + <td><%= player.strikes %></td> + </tr> + <% end %> + </tbody> +</table> + +<hr> <p><%= @players.size %> available players</p> -<ul> - <% @players.each do |player| %> - <li> - <%= link_to player.name, player %> - </li> - <% end %> -</ul> - -<%= link_to "Add player", new_player_path %> + +<%= link_to "+ Add player", new_player_path %> |