app/assets/stylesheets/characters.css

@@ -69,10 +69,9 @@
     margin: 0;
     padding: .5em;
   }
-  input, .stat-value {
+  input {
     text-align: center;
     font-size: 3em;
     width: 2.5em;
-    border: 1px solid var(--background-color);
   }
 }

app/assets/stylesheets/colors.css

@@ -9,7 +9,6 @@
   --secondary-text-color: #777;
 
   --inset-bg-color: #eee;
-  --notification-bg-color: #eee;
 
   --border-radius: .5em;
 

app/assets/stylesheets/forms.css

@@ -3,7 +3,7 @@ form, fieldset {
   gap: 1rem;
   grid-template-columns: 1fr 2fr;
 
-  input, label, .stat-value {
+  input, label {
     padding: .5em;
   }
 
@@ -45,8 +45,12 @@ form.stat-form, form.counter-form {
   display: flex;
   flex-direction: column;
 
-  .stat-value {
+  input:disabled {
     color: var(--text-color);
     background-color: var(--input-background);
   }
+
+  input[type=number]:disabled {
+    -moz-appearance: textfield;
+  }
 }

app/assets/stylesheets/notifications.css

@@ -1,13 +0,0 @@
-#table-notifications {
-  position: fixed;
-  width: 60em;
-  max-width: 80%;
-  top: 1em; left: 0;
-  transform: translate(calc(50vw - 50%));
-  li {
-    background-color: var(--notification-bg-color);
-    list-style-type: none;
-    padding: .5em;
-    border-radius: var(--border-radius);
-  }
-}

app/assets/stylesheets/tables.css

@@ -1,30 +0,0 @@
-#table-content{
-  padding: 1em;
-  background-color: var(--inset-bg-color);
-  border: 1px solid #000;
-  z-index: 1;
-}
-
-.table-tabs {
-  ul {
-    margin: 0;
-  }
-  li {
-    list-style-type: none;
-    display: inline;
-    a:link, a:visited {
-      color: inherit;
-      text-decoration: none;
-      background-color: var(--inset-bg-color);
-      padding: .5em 1em;
-      border: 1px solid black;
-      position: relative;
-      top: 1px;
-    }
-    a:link.active, a:visited.active {
-      display: inline-block;
-  z-index: 1000;
-      border-bottom: none;
-    }
-  }
-}

app/controllers/dice_rolls_controller.rb

@@ -1,27 +0,0 @@
-# frozen_string_literal: true
-
-class DiceRollsController < ApplicationController
-  before_action :set_table
-
-  def create
-    rollable = dice_roll_params[:rollable_type].constantize.find(dice_roll_params[:rollable_id])
-    @table.dice_rolls.create!(
-      rollable:,
-      result: DiceRoller.new(rollable.roll_command, stat: rollable).roll,
-    )
-    head :ok
-  end
-
-  private
-
-    def set_table
-      @table = Current.user.owned_tables.find(params[:table_id])
-    end
-
-    def dice_roll_params
-      params.require(:dice_roll).permit(
-        :rollable_type,
-        :rollable_id,
-      )
-    end
-end

app/controllers/stats_controller.rb

@@ -17,7 +17,6 @@ class StatsController < ApplicationController
   end
 
   def update
-    @editable = true
     @stat.update(stat_params)
   end
 

app/javascript/controllers/dice_roll_controller.js

@@ -1,9 +0,0 @@
-import { Controller } from "@hotwired/stimulus"
-
-export default class extends Controller {
-  static targets = [ "input" ]
-
-  rollDice() {
-    this.element.requestSubmit()
-  }
-}

app/javascript/controllers/notifications_controller.js

@@ -1,9 +0,0 @@
-import { Controller } from "@hotwired/stimulus"
-
-export default class extends Controller {
-  connect() {
-    setTimeout(() => {
-      this.element.remove()
-    }, 5000)
-  }
-}

app/models/dice_roll.rb

@@ -7,12 +7,6 @@ class DiceRoll < ApplicationRecord
   validates :result, presence: true,
                      numericality: { only_integer: true }
 
-  after_create_commit do
-    broadcast_append_to table, target: "events"
-    broadcast_append_to table, target: "table-notifications",
-                               partial: "shared/notification", locals: { object: self }
-  end
-
   def display_text
     "#{rollable.character.name} rolled #{rollable.name}: <strong>#{result}</strong>".html_safe
   end

app/views/dice_rolls/_dice_roll.turbo_stream.erb

@@ -1,2 +0,0 @@
-<%= turbo_stream.append("events", partial: "dice_rolls/dice_roll",
-    locals: { dice_roll: dice_roll }) %>

app/views/events/index.html.erb

@@ -1,5 +1,5 @@
 <% content_for :title, @table.name %>
 
-<ul id="events">
+<ul>
   <%= render @events %>
 </ul>

app/views/layouts/application.html.erb

@@ -14,7 +14,6 @@
 
   <body>
     <header>
-      <%= yield(:header_content) if content_for?(:header_content) %>
       <h1><%= link_to t("site_name"), root_path %></h1>
       <nav>
         <ul>

app/views/layouts/table.html.erb

@@ -1,18 +1,12 @@
-<%= content_for :header_content do %>
-  <ul id="table-notifications"></ul>
-<% end %>
-
 <% content_for :submenu do %>
-  <%= turbo_stream_from @table %>
   <h2><%= @table.name %></h2>
-  <nav class="table-tabs">
+  <nav>
     <ul>
-      <li><%= link_to t(".overview"), @table, class: class_names(active: params[:controller] == "tables") %></li>
+      <li><%= link_to t(".overview"), @table %></li>
       <% @characters.each do |character| %>
-        <li><%= link_to character.name, table_character_character_sheet_sections_path(@table, character),
+        <li><%= link_to character.name, table_character_character_sheet_sections_path(@table, character) %></li>
-                class: class_names(active: params[:controller] == "character_sheet_sections") %></li>
       <% end %>
-      <li><%= link_to t(".events"), table_events_path(@table), class: class_names(active: params[:controller] == "events") %></li>
+      <li><%= link_to t(".events"), table_events_path(@table) %></li>
     </ul>
   </nav>
 <% end %>

app/views/shared/_notification.html.erb

@@ -1,3 +0,0 @@
-<div data-controller="notifications">
-  <%= render object %>
-</div>

app/views/stats/_stat.html.erb

@@ -4,16 +4,8 @@
                  data: { turbo_method: :delete, turbo_confirm: t(".confirm_delete", name: stat.name) }) %>
   <% end %>
   <h6><%= stat.name %></h6>
-  <% if @editable %>
+  <%= form_with model: stat, class: "stat-form",
-    <%= form_with model: stat, class: "stat-form", data: { controller: "auto-update" } do |f| %>
+                data: { controller: "auto-update", auto_update_update_url_value: stat_path(stat) } do |f| %>
-      <%= f.number_field :value %>
+    <%= f.number_field :value, disabled: !@editable %>
-    <% end %>
-  <% else %>
-    <%= form_with model: DiceRoll.new, url: table_dice_rolls_path(stat.character.table), class: "stat-form",
-        data: { controller: "dice-roll" } do |f| %>
-      <%= f.hidden_field :rollable_type, value: "Stat" %>
-      <%= f.hidden_field :rollable_id, value: stat.id %>
-      <div class="stat-value" data-action="click->dice-roll#rollDice"><%= stat.value %></div>
-    <% end %>
   <% end %>
 </div>

app/views/tables/show.html.erb

@@ -1,7 +1,7 @@
 <% content_for :title, @table.name %>
 
-<%= link_to t(".invite_user"), new_table_table_invite_path(@table), data: { turbo_frame: "_top" } %>
+<%= link_to t(".invite_user"), new_table_table_invite_path(@table) %>
-<%= link_to t(".edit_table"), edit_table_path(@table), data: { turbo_frame: "_top" } %>
+<%= link_to t(".edit_table"), edit_table_path(@table) %>
 
 <dl>
   <dt><%= t(".game_system") %>:</dt>
@@ -21,7 +21,7 @@
 <% if @table.users.any? %>
   <ul>
     <% @table.users.each do |user| %>
-      <li><%= link_to user.username, user, data: { turbo_frame: "_top" } %></li>
+      <li><%= link_to user.username, user %></li>
     <% end %>
   </ul>
 <% else %>

config/cable.yml

@@ -5,4 +5,6 @@ test:
   adapter: test
 
 production:
-  adapter: postgresql
+  adapter: redis
+  url: <%= ENV.fetch("REDIS_URL") { "redis://localhost:6379/1" } %>
+  channel_prefix: tabletop_companion_production

config/routes.rb

@@ -29,7 +29,6 @@ Rails.application.routes.draw do
     resources :characters, only: [] do
       resources :character_sheet_sections, only: [ :index ]
     end
-    resources :dice_rolls
     resources :events, only: [ :index ]
     resources :table_invites, only: [ :new, :create ]
   end

todo.md

@@ -1,5 +1,5 @@
-- test for rolls controller & system tests
+- character sheet on table
-- table notifications for rolls
+- table tabs
 - roll on click
 - Sheets
   - Lists