chats 0.5.1 → 0.6.2

data/app/views/chats/conversations/index.html.erb

@@ -3,6 +3,25 @@
     re-renders this page per-viewer — see Chats::Broadcasts for why refreshes
     beat surgical row patches here.
 
+    Two panes (`config.inbox_layout = :split`, the default): the list in a
+    <nav> on the left, and on the right the `chats_conversation` frame with
+    the open conversation (/messages/:id renders this same template, see
+    ConversationsController#show) or the empty pane (/messages). chats.css
+    decides with one media query: at least 700px wide both panes show; below
+    that the page is the list (/messages) or the thread (/messages/:id), as
+    with `:stacked`, which renders the list alone and the thread on its own
+    page (show.html.erb). Every request renders the same markup; only CSS
+    knows the width, so it works before JavaScript runs and without it.
+
+    A row is a real link to /messages/:id. On a wide screen it targets the
+    frame: the server answers with the pane alone (pane.html.erb) and the
+    frame's data-turbo-action="advance" moves the address bar and history
+    there. A reload, a deep link or back/forward renders the whole page.
+    The frame is `disabled` until chats--split enables it, and only while
+    the screen is wide: on a phone, and without JavaScript, the same link
+    opens the thread page. The pane keeps its draft, scroll and stream
+    through every refresh of the list (see _conversation_pane).
+
     Rows come from Chats::Inbox and are either a Chats::Conversation or a
     Chats::InboxGroup (every direct thread with a `inbox: :grouped` messager,
     stacked into one row).
@@ -31,6 +50,9 @@
     `.restore` visit that honors this. Requires the host layout to
     `yield :head` (the Rails convention); harmless no-op otherwise.
     https://turbo.hotwired.dev/handbook/building#opting-out-of-caching %>
+<%# The page's title, for a layout that shows `yield(:title)`: the inbox's,
+    or the open conversation's (chats_page_title, config.page_title). %>
+<% content_for :title, chats_page_title(@conversation) %>
 <% content_for :head do %>
   <meta name="turbo-cache-control" content="no-cache">
 <% end %>
@@ -39,8 +61,9 @@
   <%= turbo_stream_from chats_current_messager, :chats_inbox %>
 </div>
 
-<%# data-chats-dock-hide: the messaging dock hides itself on this page. %>
-<div class="chats chats-inbox" data-chats-dock-hide>
+<% split = chats_two_panes? %>
+<% inbox_empty = @rows.none? && params[:q].blank? %>
+<% list = capture do %>
   <header class="chats-inbox__header">
     <h1 class="chats-inbox__title"><%= t("chats.inbox.title") %></h1>
   </header>
@@ -53,15 +76,17 @@
   <% end %>
 
   <% if Chats.config.search && !@inbox.filtered? %>
-    <%# The inbox's own path exactly as it was opened (/messages or
-        /messages/: the engine answers both), so the URL a search puts in the
-        address bar is still this page to Turbo, and the next refresh morphs
-        it instead of re-rendering it from scratch. %>
-    <%= form_with url: request.original_fullpath.split("?").first,
-          method: :get,
+    <%# No `action` (so a bare tag.form: form_with always writes one): a form
+        without one submits to the page's own URL (the HTML standard, and
+        Turbo too), whatever that is NOW — /messages,
+        /messages/ (the engine answers both), or /messages/:id after a row
+        opened a conversation in the pane and moved the address bar. So the
+        URL a search puts in the address bar is still this page to Turbo,
+        the next refresh morphs it instead of re-rendering it from scratch,
+        and on the two-pane page the open conversation stays open. %>
+    <%= tag.form method: :get,
           class: "chats-search",
-          # `html:`: form_with drops attributes it doesn't know (role among them).
-          html: { role: "search" },
+          role: "search",
           data: {
             controller: "chats--debounced-submit",
             action: "input->chats--debounced-submit#queue search->chats--debounced-submit#queue",
@@ -88,12 +113,17 @@
       <ul class="chats-inbox__list">
         <% @rows.each do |row| %>
           <% if row.is_a?(Chats::InboxGroup) %>
-            <%= render "chats/conversations/group", group: row, viewer: chats_current_messager %>
+            <%= render "chats/conversations/group", group: row, viewer: chats_current_messager,
+                                                    pane: split, current: @conversation,
+                                                    filter: chats_inbox_filter %>
           <% else %>
             <%= render "chats/conversations/conversation_row",
                   conversation: row,
                   viewer: chats_current_messager,
-                  unread_count: @unread_counts.fetch(row.id, 0) %>
+                  unread_count: @unread_counts.fetch(row.id, 0),
+                  pane: split,
+                  current: @conversation,
+                  filter: chats_inbox_filter %>
           <% end %>
         <% end %>
       </ul>
@@ -102,7 +132,8 @@
         <p class="chats-empty__title"><%= t("chats.inbox.no_results_title") %></p>
         <p class="chats-empty__hint"><%= t("chats.inbox.no_results_hint", query: params[:q]) %></p>
       </div>
-    <% else %>
+    <% elsif !split %>
+      <%# (Two panes: the empty pane says it, once — chats/conversations/_empty_pane.) %>
       <div class="chats-empty">
         <span class="chats-empty__icon" aria-hidden="true">💬</span>
         <p class="chats-empty__title"><%= t("chats.inbox.empty_title") %></p>
@@ -111,4 +142,40 @@
       </div>
     <% end %>
   <% end %>
-</div>
+<% end %>
+
+<%# data-chats-dock-hide: the messaging dock hides itself on this page. %>
+<% if split %>
+  <%= tag.div class: "chats chats-inbox chats-split",
+              data: {
+                chats_dock_hide: true,
+                controller: "chats--split",
+                action: "turbo:before-prefetch->chats--split#prefetch " \
+                        "turbo:frame-load->chats--split#opened " \
+                        "chats:read->chats--split#threadRead " \
+                        "turbo:before-morph-attribute->chats--split#keepEnabled"
+              } do %>
+    <nav class="chats-split__list" aria-label="<%= t("chats.inbox.title") %>" data-chats--split-target="list">
+      <%= list %>
+    </nav>
+
+    <%= turbo_frame_tag Chats::ConversationsController::PANE_FRAME,
+          class: "chats-split__conversation",
+          target: "_top",
+          disabled: true,
+          data: { turbo_action: "advance", "chats--split-target": "frame" } do %>
+      <% if @conversation %>
+        <%= render "chats/conversations/conversation_pane",
+                   conversation: @conversation, participant: @participant,
+                   messages: @messages, more: @more_messages, first_unread_id: @first_unread_id,
+                   counterpart: chats_counterpart, read_pending: @read_pending %>
+      <% else %>
+        <%= render "chats/conversations/empty_pane", inbox_empty: inbox_empty, inbox: @inbox %>
+      <% end %>
+    <% end %>
+  <% end %>
+<% else %>
+  <div class="chats chats-inbox" data-chats-dock-hide>
+    <%= list %>
+  </div>
+<% end %>

data/app/views/chats/conversations/pane.html.erb

@@ -0,0 +1,14 @@
+<%# The answer to a row click on the two-pane messaging page: the request
+    comes from the page's `chats_conversation` frame (Turbo-Frame header),
+    so the answer is that frame with the conversation pane in it and
+    nothing else — no list, turbo-rails' minimal frame layout
+    (ConversationsController#show). The pane carries its title
+    (data-title), which chats--split gives the tab.
+    Turbo swaps it into the frame and, the frame being
+    data-turbo-action="advance", moves the address bar to /messages/:id. %>
+<%= turbo_frame_tag Chats::ConversationsController::PANE_FRAME do %>
+  <%= render "chats/conversations/conversation_pane",
+             conversation: @conversation, participant: @participant,
+             messages: @messages, more: @more_messages, first_unread_id: @first_unread_id,
+             counterpart: chats_counterpart, read_pending: @read_pending %>
+<% end %>

data/app/views/chats/conversations/show.html.erb

@@ -1,6 +1,10 @@
-<%# The thread page: its own header around the shared thread body
-    (chats/conversations/_thread, which a floating dock window renders too —
-    see that partial for how the live thread works). %>
+<%# The thread page under `config.inbox_layout = :stacked`: its header
+    (chats/conversations/_thread_header) over the shared thread body
+    (chats/conversations/_thread, which a floating dock window and the
+    two-pane page's conversation pane render too — see that partial for how
+    the live thread works). Under the default `:split`, /messages/:id is
+    the two-pane page instead (index.html.erb). %>
+<% content_for :title, chats_page_title(@conversation) %>
 <%= chats_styles %>
 
 <%= render layout: "chats/conversations/thread",
@@ -9,67 +13,9 @@
              messages: @messages,
              more: @more_messages,
              first_unread_id: @first_unread_id,
-             counterpart: chats_counterpart
+             counterpart: chats_counterpart,
+             read_pending: @read_pending
            } do %>
-  <header class="chats-thread__header">
-    <%= link_to root_path, class: "chats-thread__back", "aria-label": t("chats.thread.back") do %>
-      <span aria-hidden="true">‹</span>
-    <% end %>
-
-    <%# The counterpart is already resolved for the title, the profile link
-        and the stack link — reuse it here instead of asking again. %>
-    <% if chats_counterpart %>
-      <%= chats_messager_avatar(chats_counterpart) %>
-    <% else %>
-      <%= chats_conversation_avatar(@conversation, chats_current_messager) %>
-    <% end %>
-
-    <div class="chats-thread__identity">
-      <%# Both arms wrap the name in .chats-thread__name: the <h1> is a flex
-          row (the badge is its second child), and only a real element can
-          carry the ellipsis. %>
-      <h1 class="chats-thread__title">
-        <% if chats_counterpart %>
-          <%= chats_messager_name(chats_counterpart, css_class: "chats-thread__name") %>
-          <%= chats_verified_badge(chats_counterpart) %>
-        <% else %>
-          <span class="chats-thread__name"><%= @conversation.title_for(chats_current_messager) %></span>
-        <% end %>
-      </h1>
-      <% if @conversation.subject_label %>
-        <p class="chats-thread__subject"><%= @conversation.subject_label %></p>
-      <% end %>
-      <%# A stacked counterpart's thread is one of many: offer the way back
-          to the whole stack (chats' filtered inbox, or the host's own
-          screen via `group_path:`). %>
-      <% if Chats.grouped_inbox?(chats_counterpart) %>
-        <p class="chats-thread__see-all">
-          <%= link_to t("chats.thread.see_all"), chats_group_path_for(chats_counterpart), class: "chats-thread__see-all-link" %>
-        </p>
-      <% end %>
-    </div>
-
-    <details class="chats-menu">
-      <summary class="chats-menu__trigger" aria-label="<%= t("chats.thread.options") %>">⋯</summary>
-      <div class="chats-menu__panel">
-        <% if @participant&.muted? %>
-          <%= button_to t("chats.thread.unmute"), unmute_conversation_path(@conversation), method: :post, class: "chats-menu__item" %>
-        <% else %>
-          <%= button_to t("chats.thread.mute"), mute_conversation_path(@conversation), method: :post, class: "chats-menu__item" %>
-        <% end %>
-        <% if @conversation.group? %>
-          <%= button_to t("chats.thread.leave"), leave_conversation_path(@conversation), method: :post, class: "chats-menu__item chats-menu__item--danger" %>
-        <% end %>
-        <%# Slot: the host's own thread actions (block, report, archive…).
-            `blockable` is false for messagers declared
-            `acts_as_messager blockable: false`, so safety affordances
-            disappear against a support desk without any class check. %>
-        <%= chats_slot :conversation_header_actions,
-              conversation: @conversation,
-              viewer: chats_current_messager,
-              counterpart: chats_counterpart,
-              blockable: chats_blockable?(chats_counterpart) %>
-      </div>
-    </details>
-  </header>
+  <%= render "chats/conversations/thread_header",
+             conversation: @conversation, participant: @participant, counterpart: chats_counterpart %>
 <% end %>

data/app/views/chats/dock/_dock.html.erb

@@ -49,7 +49,8 @@
                 "input->chats--dock#typed " \
                 "click@document->chats--dock#delegate " \
                 "turbo:before-stream-render@document->chats--dock#beforeStreamRender " \
-                "turbo:load@document->chats--dock#pageChanged turbo:render@document->chats--dock#pageChanged"
+                "turbo:load@document->chats--dock#pageChanged turbo:render@document->chats--dock#pageChanged " \
+                "turbo:frame-load@document->chats--dock#pageFrameLoaded"
       } do %>
   <%= turbo_stream_from viewer, :chats_dock %>
   <%= render "chats/dock/signal" %>

data/config/importmap.rb

@@ -17,6 +17,7 @@ pin "controllers/chats/debounced_submit_controller", to: "chats/debounced_submit
 pin "controllers/chats/refresh_inbox_controller", to: "chats/refresh_inbox_controller.js"
 pin "controllers/chats/dock_controller", to: "chats/dock_controller.js"
 pin "controllers/chats/unread_badge_controller", to: "chats/unread_badge_controller.js"
+pin "controllers/chats/split_controller", to: "chats/split_controller.js"
 
 # Not a controller: what the inbox, thread, badge and dock controllers share
 # to catch up on broadcasts they missed. Outside controllers/ so the eager

data/config/locales/en.yml

@@ -1,5 +1,8 @@
 en:
   chats:
+    page_title:
+      inbox: "Messages"
+      conversation: "%{name} · Messages"
     inbox:
       title: "Messages"
       search_placeholder: "Search conversations"
@@ -8,6 +11,7 @@ en:
       no_results_title: "No results"
       no_results_hint: "Nothing matched “%{query}”."
       no_messages: "No messages yet"
+      select_hint: "Select a conversation"
       filtered_by: "Conversations with %{name}"
       clear_filter: "Show all conversations"
       group_count:

data/config/locales/es.yml

@@ -1,5 +1,8 @@
 es:
   chats:
+    page_title:
+      inbox: "Mensajes"
+      conversation: "%{name} · Mensajes"
     inbox:
       title: "Mensajes"
       search_placeholder: "Buscar conversaciones"
@@ -8,6 +11,7 @@ es:
       no_results_title: "Sin resultados"
       no_results_hint: "No hay nada que coincida con «%{query}»."
       no_messages: "Sin mensajes todavía"
+      select_hint: "Selecciona una conversación"
       filtered_by: "Conversaciones con %{name}"
       clear_filter: "Ver todas las conversaciones"
       group_count:

data/lib/chats/configuration.rb

@@ -15,6 +15,7 @@ module Chats
   class Configuration
     ATTACHMENT_MODES = [false, :images, :any].freeze
     DELETION_MODES = [false, :soft, :hard].freeze
+    INBOX_LAYOUTS = %i[split stacked].freeze
 
     # --- Host integration -----------------------------------------------------
 
@@ -92,6 +93,15 @@ module Chats
     # threads.
     attr_accessor :inbox_limit
 
+    # How the inbox and a conversation share the screen. `:split` (the
+    # default): on screens at least 700px wide, /messages is two panes, the
+    # conversation list on the left and the open conversation on the right,
+    # and /messages/:id is that same page with the conversation open
+    # (LinkedIn's messaging page); narrower, the list and the thread are
+    # separate pages. `:stacked`: separate pages at every width, as before
+    # 0.6.
+    attr_reader :inbox_layout
+
     # Per-sender send throttle, enforced with Rails 8's built-in controller
     # `rate_limit` when available (feature-detected; on Rails 7.1 it's a
     # no-op). Shape: `{ to: Integer, within: ActiveSupport::Duration }`.
@@ -164,6 +174,14 @@ module Chats
     # suppress the badge on some messagers without touching the models.
     attr_reader :verified_badge
 
+    # ->(conversation, viewer) { String } — the title of the messaging page:
+    # the inbox (conversation nil) or a conversation. The bundled views
+    # give it to `content_for :title` (a layout doing `yield(:title)` shows
+    # it), and the two-pane page gives the tab the conversation's when a
+    # row opens it in the pane. nil (the default) is the localized
+    # `chats.page_title.inbox` / `chats.page_title.conversation`.
+    attr_reader :page_title
+
     def initialize
       @messager_class = "User"
       @parent_controller = "::ApplicationController"
@@ -187,6 +205,7 @@ module Chats
       @max_attachments_per_message = 4
       @send_rate_limit = { to: 60, within: 60 } # 60 messages per minute per sender
       @inbox_limit = 200
+      @inbox_layout = :split
 
       @encrypt_messages = false
 
@@ -199,6 +218,7 @@ module Chats
       @messager_url = ->(_messager) { nil }
       @message_signature = nil
       @verified_badge = nil
+      @page_title = nil
 
       @messager_display_name = lambda do |messager|
         messager.try(:display_name) || messager.try(:name) ||
@@ -244,6 +264,18 @@ module Chats
       @deletion = normalized
     end
 
+    def inbox_layout=(value)
+      normalized = value.respond_to?(:to_sym) ? value.to_sym : value
+      unless INBOX_LAYOUTS.include?(normalized)
+        raise ConfigurationError, "inbox_layout must be one of #{INBOX_LAYOUTS.inspect}, got #{value.inspect}"
+      end
+
+      @inbox_layout = normalized
+    end
+
+    # Whether the inbox and the open conversation are one two-pane page.
+    def split_inbox? = inbox_layout == :split
+
     def send_rate_limit=(value)
       if value.nil?
         @send_rate_limit = nil
@@ -312,6 +344,10 @@ module Chats
       @verified_badge = value.nil? ? nil : ensure_callable(value, "verified_badge")
     end
 
+    def page_title=(value)
+      @page_title = value.nil? ? nil : ensure_callable(value, "page_title")
+    end
+
     def messager_display_name=(value)
       @messager_display_name = ensure_callable(value, "messager_display_name")
     end

data/lib/chats/models/conversation.rb

@@ -449,11 +449,19 @@ module Chats
       field.to_s == "title" && title.present?
     end
 
+    # Whether anything was removed (see Chats::Message#remove_reported_field!):
+    # false for a title already gone, decided under the row lock.
     def remove_reported_field!(field)
       return false unless field.to_s == "title"
 
-      update!(title: nil)
-      true
+      with_lock do
+        next false if title.blank?
+
+        update!(title: nil)
+        true
+      end
+    rescue ActiveRecord::RecordNotFound
+      false
     end
 
     def report_visible_to?(viewer, field: nil)

data/lib/chats/models/message.rb

@@ -356,12 +356,28 @@ module Chats
     # A moderator removing a reported message body = the soft-delete
     # tombstone path, so the thread shows "Message deleted" instead of a
     # hole, and attachments are purged with it.
+    #
+    # Returns whether anything was removed: true only when it actually
+    # removed something (the tombstone, or the row under `deletion: :hard`);
+    # false for a message already deleted (by its sender, by an earlier
+    # decision), already gone, or whose body is already blank. moderate
+    # 1.0.0.beta3 and later take that as the truth and refuse a removal that
+    # removed nothing, rather than record "content removed" with a statement
+    # of reasons that would be false; earlier versions ignore the return
+    # value. Decided under the message's row lock, read again under it: two
+    # decisions taken at once cannot both see it standing.
     def remove_reported_field!(field)
       return false unless field.to_s == "body"
 
-      # Trust & Safety outranks a product lock: a closed conversation must
-      # never be a place reported content can hide.
-      soft_delete!(enforce_lock: false)
+      with_lock do
+        next false if deleted? || body.blank?
+
+        # Trust & Safety outranks a product lock: a closed conversation must
+        # never be a place reported content can hide.
+        soft_delete!(enforce_lock: false)
+      end
+    rescue ActiveRecord::RecordNotFound
+      false # hard-deleted meanwhile: nothing left to remove
     end
 
     # Only people *in* the conversation may report a message (a message

data/lib/chats/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Chats
-  VERSION = "0.5.1"
+  VERSION = "0.6.2"
 end

data/lib/generators/chats/templates/initializer.rb

@@ -84,6 +84,23 @@ Chats.configure do |config|
   #
   # config.inbox_limit = 200
   #
+  # The messaging page's layout. :split (the default): from 700px wide,
+  # /messages is two panes, LinkedIn-style: the conversation list on the
+  # left, the open conversation on the right (/messages/:id is that same
+  # page, its conversation open); narrower, the list and the thread are
+  # separate pages. :stacked keeps them separate pages at every width.
+  # (Views you ejected with `rails g chats:views` keep rendering as they are.)
+  #
+  # config.inbox_layout = :split
+  #
+  # The messaging page's title (the bundled views give it to
+  # `content_for :title`, and the two-pane page to the tab when a row opens
+  # a conversation). nil: "Messages" / "Bob · Messages" (chats.page_title.*).
+  #
+  # config.page_title = ->(conversation, viewer) {
+  #   conversation ? "#{conversation.title_for(viewer)} | MyApp" : "Messages | MyApp"
+  # }
+  #
   # Encrypt message bodies at rest (ActiveRecord Encryption; requires
   # `bin/rails db:encryption:init`). Body search degrades when enabled.
   #

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification
 name: chats
 version: !ruby/object:Gem::Version
-  version: 0.5.1
+  version: 0.6.2
 platform: ruby
 authors:
 - rameerez
 bindir: exe
 cert_chain: []
-date: 2026-10-18 00:00:00.000000000 Z
+date: 2026-10-19 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
@@ -138,18 +138,23 @@ files:
 - app/javascript/chats/dock_controller.js
 - app/javascript/chats/recovery.js
 - app/javascript/chats/refresh_inbox_controller.js
+- app/javascript/chats/split_controller.js
 - app/javascript/chats/thread_controller.js
 - app/javascript/chats/unread_badge_controller.js
+- app/views/chats/conversations/_conversation_pane.html.erb
 - app/views/chats/conversations/_conversation_row.html.erb
+- app/views/chats/conversations/_empty_pane.html.erb
 - app/views/chats/conversations/_group.html.erb
 - app/views/chats/conversations/_history.html.erb
 - app/views/chats/conversations/_locked_composer.html.erb
 - app/views/chats/conversations/_messages_page.html.erb
 - app/views/chats/conversations/_read_state.html.erb
 - app/views/chats/conversations/_thread.html.erb
+- app/views/chats/conversations/_thread_header.html.erb
 - app/views/chats/conversations/badge.turbo_stream.erb
 - app/views/chats/conversations/dock.html.erb
 - app/views/chats/conversations/index.html.erb
+- app/views/chats/conversations/pane.html.erb
 - app/views/chats/conversations/refresh.turbo_stream.erb
 - app/views/chats/conversations/show.html.erb
 - app/views/chats/conversations/window.html.erb
@@ -168,7 +173,6 @@ files:
 - app/views/chats/messages/locked.turbo_stream.erb
 - app/views/chats/shared/_unread_badge.html.erb
 - app/views/chats/shared/_verified_badge.html.erb
-- chats-0.3.1.gem
 - config/importmap.rb
 - config/locales/en.yml
 - config/locales/es.yml
@@ -183,6 +187,10 @@ files:
 - docs/screenshots/dock-phone-panel.png
 - docs/screenshots/dock-phone-window.png
 - docs/screenshots/inbox.png
+- docs/screenshots/split-inbox-empty.png
+- docs/screenshots/split-inbox-phone-list.png
+- docs/screenshots/split-inbox-phone-thread.png
+- docs/screenshots/split-inbox.png
 - docs/screenshots/system-note-action.png
 - docs/screenshots/thread-actions-phone.png
 - gemfiles/rails_7.1.gemfile