message_train 0.7.5 → 1.0.0

data/app/views/message_train/application/_attachment_fields.html.haml

@@ -1,7 +1,10 @@
-.nested-fields.panel.panel-default
-  .panel-body
-    = link_to_remove_association '×'.html_safe, f, title: :remove_attachment.l, class: 'close', data: { confirm: :are_you_sure.l }
-    - if f.object.attachment.exists?
-      .attachment
-        = attachment_link(f.object)
-    = f.file_field :attachment
+.message-train-attachment-fields{ data: { 'message-train--nested-form-target': 'row' } }
+  - if f.object.persisted?
+    = f.hidden_field :_destroy, data: { 'message-train--nested-form-target': 'destroyField' }
+  %button{ type: 'button', title: :remove_attachment.l, 'aria-label': :remove_attachment.l, class: 'message-train-button message-train-button--remove-attachment', data: { action: 'message-train--nested-form#remove', confirm: :are_you_sure.l } }
+    = icon('remove')
+  - if f.object.attachment.attached? && f.object.attachment.blob.persisted?
+    .message-train-attachment
+      = attachment_link(f.object)
+  = f.label :attachment
+  = f.file_field :attachment

data/app/views/message_train/application/_attachment_link.html.haml

@@ -1,4 +1,4 @@
 - if attachment.image?
-  = link_to attachment_icon(attachment), '#', class: 'thumbnail', data: { toggle: 'modal', target: '#attachment_preview', src: attachment.attachment.url(:large), original: attachment.attachment.url, text: :click_for_original.l }
+  = link_to attachment_icon(attachment), '#', class: 'thumbnail', data: { controller: 'message-train--lightbox', action: 'message-train--lightbox#open', lightbox_dialog_id: 'attachment_preview', lightbox_src: main_app.url_for(attachment.large), lightbox_alt: attachment.attachment.filename.to_s }
 - else
-  = link_to attachment_icon(attachment), attachment.attachment.url, class: 'thumbnail'
+  = link_to attachment_icon(attachment), main_app.url_for(attachment.attachment), class: 'thumbnail'

data/app/views/message_train/application/_widget.html.haml

@@ -1,7 +1,5 @@
 - if ['boxes', 'conversations', 'messages'].include? controller_name
   - if @collective.nil?
-    - add_widget do
-      = boxes_widget @box_user
+    = boxes_widget @box_user
   - else
-    - add_widget do
-      = collective_boxes_widget @collective, @box_user
+    = collective_boxes_widget @collective, @box_user

data/app/views/message_train/boxes/_widget.html.haml

@@ -1,11 +1,11 @@
 - unless boxes.empty?
   - cache ['message-train', boxes_widget_cache_key(boxes)] do
-    .hidden-print
+    .message-train-widget
       %h3= :messages.l
-      .text-center= link_to :compose.l, message_train.new_box_message_path(:in), class: 'btn btn-lg btn-primary btn-compose'
-      %ul.list-group
+      = link_to :compose.l, message_train.new_box_message_path(:in), class: 'message-train-button message-train-button--large message-train-btn-compose'
+      %ul.message-train-list-group
         - boxes.each do |box|
           - if @collective.nil?
-            = box_list_item box, class: 'list-group-item'
+            = box_list_item box, class: 'message-train-list-group-item'
           - else
-            = collective_list_item box, class: 'list-group-item'
+            = collective_list_item box, class: 'message-train-list-group-item'

data/app/views/message_train/boxes/show.html.haml

@@ -7,54 +7,37 @@
     marking_path = message_train.collective_box_path(@collective.path_part, @box.division)
   end
 - cache ['message-train', 'boxes-show', @box_user, @box] do
-  = form_tag marking_path, method: :put, remote: true, id: 'box', data: { type: :json } do
-    - if @conversations.empty?
-      %em= :box_is_empty.l
-    - else
-      #box-actions.box-actions
-        .btn-group.check-all
-          %button.btn.btn-default{ type: 'button' }
-            = check_box_tag 'check_all', 'all', false
-          %button.btn.btn-default.dropdown-toggle{ aria: { expanded: 'false', haspopup: 'true' }, data: { toggle: 'dropdown'}, type: 'button' }
-            %span.caret
-            %span.sr-only= :toggle_dropdown.l
-          %ul.dropdown-menu
-            %li
-              = link_to :all.l, '#0', class: 'check', data: { selector: '.message_train_conversation'}
-              = link_to :none.l, '#0', class: 'check', data: { selector: ''}
-              = link_to :read.l, '#0', class: 'check', data: { selector: '.message_train_conversation.read'}
-              = link_to :unread.l, '#0', class: 'check', data: { selector: '.message_train_conversation.unread'}
-        .btn-group
-          %button.btn.btn-info.dropdown-toggle{ aria: { expanded: 'false', haspopup: 'true' }, data: { toggle: 'dropdown'}, type: 'button' }
-            = :mark.l
-            %span.caret
-          %ul.dropdown-menu
-            %li
-              = link_to :mark_as_name.l(name: :read.l), '#0', class: 'mark-all-link', id: 'mark-read', data: { mark: 'read'}
-            %li
-              = link_to :mark_as_name.l(name: :unread.l), '#0', class: 'mark-all-link', id: 'mark-unread', data: { mark: 'unread'}
+  - if @conversations.empty?
+    %em= :box_is_empty.l
+  - else
+    %div{ data: { controller: 'message-train--bulk-actions' } }
+      = form_tag marking_path, method: :put, id: 'box' do
+        = hidden_field_tag :mark_to_set, nil, data: { 'message-train--bulk-actions-target': 'markToSet' }
+      #box-actions.message-train-box-actions
+        .message-train-box-actions-row
+          %strong= :check.l
+          .message-train-select-actions
+            %button.message-train-button.message-train-button--small{ type: 'button', data: { action: 'message-train--bulk-actions#selectMatching', selector: '.message_train_conversation' } }= :all.l
+            %button.message-train-button.message-train-button--small{ type: 'button', data: { action: 'message-train--bulk-actions#selectMatching', selector: '' } }= :none.l
+            %button.message-train-button.message-train-button--small{ type: 'button', data: { action: 'message-train--bulk-actions#selectMatching', selector: '.message_train_conversation.read' } }= :read.l
+            %button.message-train-button.message-train-button--small{ type: 'button', data: { action: 'message-train--bulk-actions#selectMatching', selector: '.message_train_conversation.unread' } }= :unread.l
+        .message-train-box-actions-row
+          .message-train-mark-actions
+            %button{ type: 'button', id: 'mark-read', class: 'message-train-button', data: { action: 'message-train--bulk-actions#markAll', mark: 'read' } }= icon('eye-open') + :mark_read.l
+            %button{ type: 'button', id: 'mark-unread', class: 'message-train-button', data: { action: 'message-train--bulk-actions#markAll', mark: 'unread' } }= icon('eye-close') + :mark_unread.l
             - if @box.division == :trash
-              %li
-                = link_to :mark_as_name.l(name: :untrashed.l), '#0', class: 'mark-all-link', id: 'mark-untrash', data: { mark: 'untrash'}
-              %li.divider{ role: 'separator' }
-              %li
-                = link_to :mark_as_name.l(name: :deleted.l), '#0', class: 'mark-all-link', id: 'mark-deleted', data: { mark: 'deleted'}
+              %button{ type: 'button', id: 'mark-untrash', class: 'message-train-button', data: { action: 'message-train--bulk-actions#markAll', mark: 'untrash' } }= icon('inbox') + :untrashed.l
+              %button{ type: 'button', id: 'mark-deleted', class: 'message-train-button', data: { action: 'message-train--bulk-actions#markAll', mark: 'deleted' } }= icon('remove') + :deleted.l
             - else
-              %li
-                = link_to :mark_as_name.l(name: :trashed.l), '#0', class: 'mark-all-link', id: 'mark-trash', data: { mark: 'trash'}
-            %li.divider{ role: 'separator' }
+              %button{ type: 'button', id: 'mark-trash', class: 'message-train-button', data: { action: 'message-train--bulk-actions#markAll', mark: 'trash' } }= icon('trash') + :move_to_trash.l
             - if @box.division == :ignored
-              %li
-                = link_to :mark_as_name.l(name: :unignored.l), '#0', class: 'mark-all-link', id: 'mark-unignored', data: { mark: 'unignore'}
+              %button{ type: 'button', id: 'mark-unignored', class: 'message-train-button', data: { action: 'message-train--bulk-actions#markAll', mark: 'unignore' } }= icon('volume-up') + :unignored.l
             - else
-              %li
-                = link_to :mark_as_name.l(name: :ignored.l), '#0', class: 'mark-all-link', id: 'mark-ignored', data: { mark: 'ignore'}
-        - if @collective.nil?
-          = link_to :compose.l, message_train.new_box_message_path(@box.division), class: 'btn btn-primary compose'
-        - elsif @collective.allows_sending_by? @box_user
-          = link_to :compose_to_collective.l(collective: collective_name(@collective)), message_train.new_collective_box_message_path(@collective.path_part, @box.division), class: 'btn btn-primary compose'
-        %span#spinner.hide= icon 'refresh spinning'
-        = hidden_field_tag :mark_to_set
-      %table#message-train-conversations.message-train-conversations.table.table-condensed
+              %button{ type: 'button', id: 'mark-ignored', class: 'message-train-button', data: { action: 'message-train--bulk-actions#markAll', mark: 'ignore' } }= icon('volume-off') + :ignore.l
+          - if @collective.nil?
+            = link_to icon('plus') + :compose.l, message_train.new_box_message_path(@box.division), class: 'message-train-button compose'
+          - elsif @collective.allows_sending_by? @box_user
+            = link_to icon('plus') + :compose_to_collective.l(collective: collective_name(@collective)), message_train.new_collective_box_message_path(@collective.path_part, @box.division), class: 'message-train-button compose'
+      %table#message-train-conversations.message-train-conversations
         = render @conversations
       = paginate @conversations

data/app/views/message_train/collectives/_widget.html.haml

@@ -1,9 +1,9 @@
 - cache ['message-train', 'collectives-widget', box_user, collective] do
   - unless collective.boxes_for_participant(box_user).empty?
-    .hidden-print
+    .message-train-widget
       %h3= :collective_messages.l(collective: collective_name(collective))
       - if collective.allows_sending_by?(box_user)
-        .text-center= link_to :compose_to_collective.l(collective: collective_name(collective)), message_train.new_collective_box_message_path(collective.path_part, :sent), class: 'btn btn-lg btn-primary btn-compose'
-      %ul.list-group
+        = link_to :compose_to_collective.l(collective: collective_name(collective)), message_train.new_collective_box_message_path(collective.path_part, :sent), class: 'message-train-button message-train-button--large message-train-btn-compose'
+      %ul.message-train-list-group
         - collective.boxes_for_participant(box_user).each do |box|
-          = collective_list_item box, class: 'list-group-item'
+          = collective_list_item box, class: 'message-train-list-group-item'

data/app/views/message_train/conversations/_conversation.html.haml

@@ -1,25 +1,25 @@
 - cache ['message-train', 'conversation-result', @box_user, conversation] do
   %tr.message_train_conversation{ class: conversation_class(@box, conversation), id: 'message_train_conversation_' + conversation.id.to_s }
-    %td.col-xs-1= check_box_tag "objects[conversations][#{conversation.id.to_s}]", conversation.id
+    %td.message-train-conversation-checkbox= check_box_tag "objects[conversations][#{conversation.id.to_s}]", conversation.id, false, form: 'box', data: { 'message-train--bulk-actions-target': 'checkbox' }
     - if @collective.nil?
-      %td.col-xs-2= link_to conversation_senders(conversation), message_train.box_conversation_path(@box.division, conversation)
-      %td.col-xs-8= link_to conversation.subject, message_train.box_conversation_path(@box.division, conversation)
-      %td
+      %td.message-train-conversation-sender= link_to conversation_senders(conversation), message_train.box_conversation_path(@box.division, conversation)
+      %td.message-train-conversation-subject= link_to conversation.subject, message_train.box_conversation_path(@box.division, conversation)
+      %td.message-train-conversation-attachment
         - if conversation.attachments.any?
           = icon 'paperclip'
-      %td.col-xs-1.conversation-actions
+      %td.message-train-conversation-actions
         = conversation_trashed_toggle conversation
         - if @box.division == :trash
           = conversation_deleted_toggle conversation
-      %td.col-xs-1.date-column= fuzzy_date(conversation.updated_at)
+      %td.message-train-date-column= fuzzy_date(conversation.updated_at)
     - else
-      %td.col-xs-2= link_to conversation_senders(conversation), message_train.collective_box_conversation_path(@collective.path_part, @box.division, conversation)
-      %td.col-xs-8= link_to conversation.subject, message_train.collective_box_conversation_path(@collective.path_part, @box.division, conversation)
-      %td
+      %td.message-train-conversation-sender= link_to conversation_senders(conversation), message_train.collective_box_conversation_path(@collective.path_part, @box.division, conversation)
+      %td.message-train-conversation-subject= link_to conversation.subject, message_train.collective_box_conversation_path(@collective.path_part, @box.division, conversation)
+      %td.message-train-conversation-attachment
         - if conversation.attachments.any?
           = icon 'paperclip'
-      %td.col-xs-1.conversation-actions
+      %td.message-train-conversation-actions
         = conversation_trashed_toggle conversation, @collective
         - if @box.division == :trash
           = conversation_deleted_toggle conversation, @collective
-      %td.col-xs-1.date-column= fuzzy_date(conversation.updated_at)
+      %td.message-train-date-column= fuzzy_date(conversation.updated_at)

data/app/views/message_train/conversations/_deleted_toggle.html.haml

@@ -1 +1 @@
-= conversation_toggle conversation, 'remove', :deleted, :put, :mark_as_name.l(name: :deleted.l), data: { confirm: :delete_forever_this_cannot_be_undone.l }, collective: collective
+= conversation_toggle conversation, 'remove', :deleted, :put, :mark_as_name.l(name: :deleted.l), data: { turbo_confirm: :delete_forever_this_cannot_be_undone.l, turbo_frame: '_top' }, collective: collective, kind: 'deleted'

data/app/views/message_train/conversations/_ignored_toggle.html.haml

@@ -1,4 +1,4 @@
 - if conversation.participant_ignored?(@box_user)
-  = conversation_toggle conversation, 'volume-up', :unignore, :delete, :mark_as_name.l(name: :unignored.l), collective: collective
+  = conversation_toggle conversation, 'volume-up', :unignore, :delete, :mark_as_name.l(name: :unignored.l), data: { turbo_frame: '_top' }, collective: collective, kind: 'ignored', show_text: true
 - else
-  = conversation_toggle conversation, 'volume-off', :ignore, :delete, :mark_as_name.l(name: :ignored.l), data: { confirm: :are_you_sure.l}, collective: collective
+  = conversation_toggle conversation, 'volume-off', :ignore, :delete, :mark_as_name.l(name: :ignored.l), data: { turbo_confirm: :are_you_sure.l, turbo_frame: '_top' }, collective: collective, kind: 'ignored', show_text: true

data/app/views/message_train/conversations/_read_toggle.html.haml

@@ -1,4 +1,4 @@
 - if conversation.includes_unread_for?(@box_user)
-  = conversation_toggle conversation, 'eye-open', :read, :put, :mark_as_name.l(name: :read.l), collective: collective
+  = conversation_toggle conversation, 'eye-open', :read, :put, :mark_as_name.l(name: :read.l), collective: collective, kind: 'read'
 - else
-  = conversation_toggle conversation, 'eye-close', :unread, :put, :mark_as_name.l(name: :unread.l), collective: collective
+  = conversation_toggle conversation, 'eye-close', :unread, :put, :mark_as_name.l(name: :unread.l), collective: collective, kind: 'read'

data/app/views/message_train/conversations/_toggle.html.haml

@@ -1,4 +1,6 @@
-- if options[:collective].nil?
-  = icon_link_to icon, '', message_train.box_path(@box.division, objects: { 'conversations' => {conversation.id.to_s => conversation.id.to_s}}, mark_to_set: mark_to_set, format: :json), options
-- else
-  = icon_link_to icon, '', message_train.collective_box_path(options[:collective].path_part, @box.division, objects: { 'conversations' => {conversation.id.to_s => conversation.id.to_s}}, mark_to_set: mark_to_set, format: :json), options
+= turbo_frame_tag "conversation_#{conversation.id}_#{kind}_toggle" do
+  - link_text = show_text ? icon(icon) + title : icon(icon)
+  - if collective.nil?
+    = button_to link_text, message_train.box_path(@box.division, objects: { 'conversations' => {conversation.id.to_s => conversation.id.to_s}}, mark_to_set: mark_to_set), options
+  - else
+    = button_to link_text, message_train.collective_box_path(collective.path_part, @box.division, objects: { 'conversations' => {conversation.id.to_s => conversation.id.to_s}}, mark_to_set: mark_to_set), options

data/app/views/message_train/conversations/_trashed_toggle.html.haml

@@ -1,4 +1,4 @@
 - if conversation.includes_untrashed_for?(@box_user)
-  = conversation_toggle conversation, 'trash', :trash, :put, :mark_as_name.l(name: :trashed.l), data: { confirm: :are_you_sure.l}, collective: collective
+  = conversation_toggle conversation, 'trash', :trash, :put, :mark_as_name.l(name: :trashed.l), data: { turbo_confirm: :are_you_sure.l, turbo_frame: '_top' }, collective: collective, kind: 'trashed'
 - else
-  = conversation_toggle conversation, 'inbox', :untrash, :put, :mark_as_name.l(name: :untrashed.l), collective: collective
+  = conversation_toggle conversation, 'inbox', :untrash, :put, :mark_as_name.l(name: :untrashed.l), data: { turbo_frame: '_top' }, collective: collective, kind: 'trashed'

data/app/views/message_train/conversations/show.html.haml

@@ -2,12 +2,14 @@
 - add_subtitle :updated_at_time.l(time: @conversation.updated_at)
 - cache ['message-train', 'conversations-show', @box_user, @conversation] do
   #box
-    #conversation-actions.conversation-actions.pull-right
-      .btn.btn-default.btn-lg= conversation_ignored_toggle(@conversation)
-    #message_train_conversation.clear-both
+    #conversation-actions.message-train-conversation-actions
+      = conversation_ignored_toggle(@conversation)
+    #message_train_conversation
       - unless @messages.empty?
-        #message_train_messages.messages.panel-group{aria: { multiselectable: 'true' }, role: 'tablist'}
+        #message_train_messages.message-train-messages
           = render @messages
         = paginate @messages
-  = modal 'attachment_preview', :attachment_preview.l do
-    #image_placeholder
+  %dialog#attachment_preview
+    %h2= :attachment_preview.l
+    %button{ type: 'button', 'aria-label': 'Close', data: { action: 'message-train--lightbox#close' } }= icon('remove')
+    %img.message-train-lightbox-image{ data: { 'message-train--lightbox-target': 'image' } }

data/app/views/message_train/messages/_deleted_toggle.html.haml

@@ -1 +1 @@
-= message_toggle message, 'remove', :deleted, :mark_as_name.l(name: :deleted.l), data: { confirm: :delete_forever_this_cannot_be_undone.l }
+= message_toggle message, 'remove', :deleted, :mark_as_name.l(name: :deleted.l), collective, data: { turbo_confirm: :delete_forever_this_cannot_be_undone.l, turbo_frame: '_top' }, kind: 'deleted'

data/app/views/message_train/messages/_form.html.haml

@@ -2,40 +2,32 @@
   - resource = message.new_record? ? message_train.box_messages_path(@box) : message_train.box_message_path(@box, message)
 - else
   - resource = message.new_record? ? message_train.collective_box_messages_path(@collective.path_part, @box) : message_train.collective_box_message_path(@collective.path_part, @box, message)
-= bootstrap_form_for message, url: resource do |f|
+= form_for message, url: resource, html: { class: 'message-train-form' } do |f|
   - if @collective.nil?
     = f.fields_for :recipients_to_save do |g|
-      = g.text_field :users, label: :recipients.l, class: 'recipient-input'
-    - add_footer_javascript do
-      :javascript
-        url = '/box/in/participants/users.json'
-        var suggestions = new Bloodhound({
-            prefetch: url,
-            remote: {
-                wildcard: '%QUERY',
-                url: url + '?query=%QUERY',
-                transform: function(response) {
-                  return $.map(response.participants, function(val, i) {
-                    return val.slug
-                  });
-                }
-            },
-            queryTokenizer: Bloodhound.tokenizers.whitespace,
-            datumTokenizer: function(item) {
-                return Bloodhound.tokenizers.whitespace(item.slug);
-            }
-        });
+      - participants_url = message_train.box_model_participants_path(@box.division, 'users', format: :json)
+      %div{ data: { controller: 'message-train--recipient-autocomplete', message_train__recipient_autocomplete_url_value: participants_url } }
+        = g.label :users, :recipients.l
+        = g.text_field :users, class: 'recipient-input', list: 'recipient-suggestions', data: { 'message-train--recipient-autocomplete-target': 'input', action: 'input->message-train--recipient-autocomplete#search' }
+        %datalist#recipient-suggestions{ data: { 'message-train--recipient-autocomplete-target': 'list' } }
   - else
     = render partial: 'message_train/participants/prefilled_field', locals: { message: message, recipient: @collective }
+  = f.label :subject
   = f.text_field :subject
-  = f.form_group :body, label: { text: MessageTrain::Message.human_attribute_name(:body) } do
-    = f.cktext_area :body
-  .links
+  = f.label :body, MessageTrain::Message.human_attribute_name(:body)
+  %small.message-train-hint= :markdown_supported.l
+  = f.text_area :body
+
+  %div{ data: { controller: 'message-train--nested-form' } }
     %h3
-      = link_to_add_association(icon('plus'), f, :attachments, title: :add_attachment.l, class: 'btn btn-info', id: 'add-attachment', data: { association_insertion_method: 'append', association_insertion_node: '#attachments'})
+      %button{ type: 'button', title: :add_attachment.l, id: 'add-attachment', class: 'message-train-button message-train-button--add-attachment', data: { action: 'message-train--nested-form#add' } }
+        = icon('plus')
       = :attachments.l
-  #attachments
-    = f.fields_for :attachments do |attachment|
-      = render 'attachment_fields', f: attachment
-  = f.submit :save_as_draft.l, name: 'message[draft]', id: 'save-as-draft'
-  = f.submit :send.l, class: 'btn btn-primary', name: 'message[draft]', id: 'send'
+    %template{ data: { 'message-train--nested-form-target': 'template' } }
+      = f.fields_for :attachments, MessageTrain::Attachment.new, child_index: 'NEW_RECORD' do |attachment|
+        = render 'attachment_fields', f: attachment
+    #attachments{ data: { 'message-train--nested-form-target': 'rows' } }
+      = f.fields_for :attachments do |attachment|
+        = render 'attachment_fields', f: attachment
+  = f.submit :save_as_draft.l, class: 'message-train-button', name: 'message[draft]', id: 'save-as-draft'
+  = f.submit :send.l, class: 'message-train-button message-train-button--primary', name: 'message[draft]', id: 'send'

data/app/views/message_train/messages/_message.html.haml

@@ -1,46 +1,28 @@
 - cache ['message-train', 'message-result', @box_user, message] do
-  - if message.draft
-    .panel.panel-default{ class: message_class(@box, message), id: "message_train_message_#{message.id}", data: { mark_path: message_train.box_conversation_path(@box.division, id: message.message_train_conversation_id, objects: { 'messages' => {message.id.to_s => message.id.to_s} }) } }
-      .panel-heading{role: 'tab', id: "message_#{message.id}_heading"}
-        .message-actions.pull-right
-          = message_read_toggle message
-          = message_trashed_toggle message
-          - if @box.division == :trash
-            = message_deleted_toggle message
-        %h4.panel-title
-          = link_to "#message_#{message.id}_collapse", aria: { controls: "message_#{message.id}_collapse", expanded: 'true' }, data: { parent: '#accordion', toggle: 'collapse' }, role: 'button' do
-            %span.caret
-            = :message_draft.l
-            %small= message.updated_at
-      .panel-collapse.collapse.in{aria: { labelledby: "message_#{message.id}_heading" }, role: 'tabpanel', id: "message_#{message.id}_collapse"}
-        .panel-body
-          = render partial: 'message_train/messages/form', locals: { message: message }
-  - else
-    .panel.panel-default{ class: message_class(@box, message), id: "message_train_message_#{message.id}", data: { mark_path: message_train.box_conversation_path(@box.division, id: message.message_train_conversation_id, objects: { 'messages' => {message.id.to_s => message.id.to_s} }) } }
-      .panel-heading{role: 'tab', id: "message_#{message.id}_heading"}
-        .message-actions.pull-right
-          = message_read_toggle message
-          = message_trashed_toggle message
-          - if @box.division == :trash
-            = message_deleted_toggle message
-        %h4.panel-title
-          = link_to "#message_#{message.id}_collapse", aria: { controls: "message_#{message.id}_collapse", expanded: 'true' }, data: { parent: '#accordion', toggle: 'collapse' }, role: 'button' do
-            %span.caret
-            = :from_sender.l(sender: message.sender.send(MessageTrain.configuration.name_columns[message.sender.class.table_name.to_sym]))
-            %small= message.updated_at
-      .panel-collapse.collapse.in{aria: { labelledby: "message_#{message.id}_heading" }, role: 'tabpanel', id: "message_#{message.id}_collapse"}
-        .panel-body
-          %p= :to_recipient.l(recipient: message_recipients(message))
-          %p.lead= message.subject
-          = sanitize message.body
-          - if message.attachments.any?
-            .row.attachment-thumbnails
-              - for attachment in message.attachments
-                .col-md-4
-                  = attachment_link(attachment)
-        - if @box.parent.allows_sending_by?(@box_user)
-          .panel-footer
-            - if @collective.nil?
-              = link_to :reply.l, message_train.new_box_message_path(@box, conversation_id: message.message_train_conversation_id), class: 'btn btn-primary'
-            - else
-              = link_to :reply.l, message_train.new_collective_box_message_path(@collective.path_part, @box, conversation_id: message.message_train_conversation_id), class: 'btn btn-primary'
+  %details.message-train-message{ class: message_class(@box, message), id: "message_train_message_#{message.id}", open: true, data: { mark_path: message_train.box_conversation_path(@box.division, id: message.message_train_conversation_id, objects: { 'messages' => {message.id.to_s => message.id.to_s} }) } }
+    %summary
+      .message-train-message-actions
+        = message_read_toggle message, @collective
+        = message_trashed_toggle message, @collective
+        - if @box.division == :trash
+          = message_deleted_toggle message, @collective
+      - if message.draft
+        = :message_draft.l
+      - else
+        = :from_sender.l(sender: message.sender.send(MessageTrain.configuration.name_columns[message.sender.class.table_name.to_sym]))
+      %time= message.updated_at
+    - if message.draft
+      = render partial: 'message_train/messages/form', locals: { message: message }
+    - else
+      %p= :to_recipient.l(recipient: message_recipients(message))
+      %p.lead= message.subject
+      .message-train-body= message_body(message)
+      - if message.attachments.any?
+        .message-train-attachment-thumbnails
+          - for attachment in message.attachments
+            = attachment_link(attachment)
+      - if @box.parent.allows_sending_by?(@box_user)
+        - if @collective.nil?
+          = link_to :reply.l, message_train.new_box_message_path(@box, conversation_id: message.message_train_conversation_id), class: 'message-train-button'
+        - else
+          = link_to :reply.l, message_train.new_collective_box_message_path(@collective.path_part, @box, conversation_id: message.message_train_conversation_id), class: 'message-train-button'

data/app/views/message_train/messages/_read_toggle.html.haml

@@ -1,4 +1,4 @@
 - if message.is_unread_for?(@box.participant)
-  = message_toggle message, 'eye-open', :read, :mark_as_name.l(name: :read.l)
+  = message_toggle message, 'eye-open', :read, :mark_as_name.l(name: :read.l), collective, data: { turbo_frame: '_top' }, kind: 'read'
 - else
-  = message_toggle message, 'eye-close', :unread, :mark_as_name.l(name: :unread.l)
+  = message_toggle message, 'eye-close', :unread, :mark_as_name.l(name: :unread.l), collective, data: { turbo_frame: '_top' }, kind: 'read'

data/app/views/message_train/messages/_toggle.html.haml

@@ -1 +1,5 @@
-= icon_link_to icon, '', message_train.box_path(@box.division, objects: { 'messages' => {message.id.to_s => message.id.to_s}}, mark_to_set: mark_to_set, format: :json), options
+= turbo_frame_tag "message_#{message.id}_#{kind}_toggle" do
+  - if collective.nil?
+    = icon_button_to nil, icon, title, message_train.box_conversation_path(@box.division, @conversation, objects: { 'messages' => {message.id.to_s => message.id.to_s}}, mark_to_set: mark_to_set), options
+  - else
+    = icon_button_to nil, icon, title, message_train.collective_box_conversation_path(collective.path_part, @box.division, @conversation, objects: { 'messages' => {message.id.to_s => message.id.to_s}}, mark_to_set: mark_to_set), options

data/app/views/message_train/messages/_trashed_toggle.html.haml

@@ -1,4 +1,4 @@
 - if message.is_untrashed_for?(@box.participant)
-  = message_toggle message, 'trash', :trash, :mark_as_name.l(name: :trashed.l), data: { confirm: :are_you_sure.l}
+  = message_toggle message, 'trash', :trash, :mark_as_name.l(name: :trashed.l), collective, data: { turbo_confirm: :are_you_sure.l, turbo_frame: '_top' }, kind: 'trashed'
 - else
-  = message_toggle message, 'inbox', :untrash, :mark_as_name.l(name: :untrashed.l)
+  = message_toggle message, 'inbox', :untrash, :mark_as_name.l(name: :untrashed.l), collective, data: { turbo_frame: '_top' }, kind: 'trashed'

data/app/views/message_train/unsubscribes/index.html.haml

@@ -1,19 +1,16 @@
 - add_title :manage_your_email_notifications.l
 - if @box_user.unsubscribed_from_all?
   %p= :you_have_chosen_not_to_be_emailed.l
-  %p= icon_button_to 'primary', 'check', :enable_some_notifications.l, message_train.unsubscribes_all_path, method: :delete
+  %p= icon_button_to '', 'check', :enable_some_notifications.l, message_train.unsubscribes_all_path, method: :delete
 - else
-  .row
-    .col-md-8
-      %table#subscriptions.table.table-bordered.table-condensed
-        - @subscriptions.each do |subscription|
-          %tr.subscription
-            %td= subscription[:from_name]
-            - if subscription[:unsubscribe].nil?
-              %td= :subscribed.l
-              %td= button_to :disable_notifications.l, message_train.unsubscribes_path(unsubscribe: { from_type: subscription[:from_type], from_id: subscription[:from_id] }), class: 'btn btn-danger', id: "unsubscribe-#{subscription[:from_type].downcase}-#{subscription[:from_id]}"
-            - else
-              %td= :unsubscribed.l
-              %td= button_to :enable_notifications.l, message_train.unsubscribe_path(subscription[:unsubscribe].id), method: :delete, class: 'btn btn-primary', id: "remove-unsubscribe-#{subscription[:unsubscribe].id}"
-    .col-md-4
-      %p= icon_button_to 'danger btn-lg', 'remove', :disable_all_notifications.l, message_train.unsubscribes_all_path, method: :post
+  %table#subscriptions
+    - @subscriptions.each do |subscription|
+      %tr.subscription
+        %td= subscription[:from_name]
+        - if subscription[:unsubscribe].nil?
+          %td= :subscribed.l
+          %td= button_to :disable_notifications.l, message_train.unsubscribes_path(unsubscribe: { from_type: subscription[:from_type], from_id: subscription[:from_id] }), id: "unsubscribe-#{subscription[:from_type].downcase}-#{subscription[:from_id]}"
+        - else
+          %td= :unsubscribed.l
+          %td= button_to :enable_notifications.l, message_train.unsubscribe_path(subscription[:unsubscribe].id), method: :delete, id: "remove-unsubscribe-#{subscription[:unsubscribe].id}"
+  %p= icon_button_to '', 'remove', :disable_all_notifications.l, message_train.unsubscribes_all_path, method: :post

data/badges/coverage.json

@@ -0,0 +1 @@
+{"schemaVersion":1,"label":"coverage","message":"90.1%","color":"brightgreen"}

data/config/importmap.rb

@@ -0,0 +1,2 @@
+pin_all_from MessageTrain::Engine.root.join('app/javascript/controllers/message_train'),
+             under: 'controllers/message_train'

data/config/locales/en.yml

@@ -35,7 +35,7 @@ en:
   cannot_mark_type: "Cannot mark %{type}"
   cannot_mark_with_data_type: "Cannot mark with %{data_type}"
   cannot_unignore_type: "Cannot unignore %{type}"
-  check_all: 'Check All'
+  check: 'Check:'
   class_id_not_found_in_box: "%{class} %{id} not found in box"
   click_for_original: 'Click for Original'
   collective_messages: "%{collective} Messages"
@@ -52,18 +52,22 @@ en:
   enable_notifications: 'Enable Notifications'
   enable_some_notifications: 'Enable Some Notifications'
   from_sender: "From: %{sender}"
+  ignore: 'Ignore'
   ignored: 'Ignored'
   invalid_sender_for_thing: "Invalid sender for %{thing}"
   just_now: 'Just Now'
   manage_your_email_notifications: 'Manage Your Email Notifications'
-  mark: 'Mark'
+  markdown_supported: 'Markdown supported'
   mark_as_name: "Mark as %{name}" #e.g. Read, Unread etc.
+  mark_read: 'Mark Read'
+  mark_unread: 'Mark Unread'
   message_draft: 'Message Draft'
   message_saved_as_draft: 'Message saved as draft.'
   message_sent: 'Message sent.'
   messages: 'Messages'
   messages_to_collective: "Messages to %{collective}"
   messages_directly_to_myself: 'Messages Directly to Myself'
+  move_to_trash: 'Move to Trash'
   name_not_found: "%{name} not found"
   new_message: 'New Message'
   new_message_on_site_name: "New Message on %{site_name}"
@@ -80,7 +84,6 @@ en:
   subscribed: 'Subscribed'
   table_recipients: "%{table} Recipients"
   to_recipient: 'To: %{recipient}'
-  toggle_dropdown: 'Toggle Dropdown'
   trashed: 'Trashed'
   unignored: 'Unignored'
   unread: 'Unread'

data/db/migrate/20150721145319_create_message_train_conversations.rb

@@ -1,4 +1,4 @@
-class CreateMessageTrainConversations < ActiveRecord::Migration
+class CreateMessageTrainConversations < ActiveRecord::Migration[4.2]
   def change
     create_table :message_train_conversations do |t|
       t.string :subject

data/db/migrate/20150721160322_create_message_train_messages.rb

@@ -1,4 +1,4 @@
-class CreateMessageTrainMessages < ActiveRecord::Migration
+class CreateMessageTrainMessages < ActiveRecord::Migration[4.2]
   def change
     create_table :message_train_messages do |t|
       t.references :message_train_conversation, index: true, foreign_key: true

data/db/migrate/20150721161144_create_message_train_attachments.rb

@@ -1,8 +1,11 @@
-class CreateMessageTrainAttachments < ActiveRecord::Migration
+class CreateMessageTrainAttachments < ActiveRecord::Migration[4.2]
   def change
     create_table :message_train_attachments do |t|
       t.references :message_train_message, index: true, foreign_key: true
-      t.attachment :attachment
+      t.string :attachment_file_name
+      t.string :attachment_content_type
+      t.integer :attachment_file_size
+      t.datetime :attachment_updated_at
 
       t.timestamps null: false
     end

data/db/migrate/20150721161940_create_message_train_receipts.rb

@@ -1,4 +1,4 @@
-class CreateMessageTrainReceipts < ActiveRecord::Migration
+class CreateMessageTrainReceipts < ActiveRecord::Migration[4.2]
   def change
     create_table :message_train_receipts do |t|
       t.references :recipient, polymorphic: true

data/db/migrate/20150721163838_create_message_train_ignores.rb

@@ -1,4 +1,4 @@
-class CreateMessageTrainIgnores < ActiveRecord::Migration
+class CreateMessageTrainIgnores < ActiveRecord::Migration[4.2]
   def change
     create_table :message_train_ignores do |t|
       t.references :participant, polymorphic: true

data/db/migrate/20150901183458_add_received_through_to_message_train_receipts.rb

@@ -1,4 +1,4 @@
-class AddReceivedThroughToMessageTrainReceipts < ActiveRecord::Migration
+class AddReceivedThroughToMessageTrainReceipts < ActiveRecord::Migration[4.2]
   def change
     add_reference :message_train_receipts, :received_through, polymorphic: true
     add_index :message_train_receipts, [:received_through_type, :received_through_id], name: :index_message_train_receipts_on_received_through

data/db/migrate/20151004184347_add_unique_index_to_receipts.rb

@@ -1,4 +1,4 @@
-class AddUniqueIndexToReceipts < ActiveRecord::Migration
+class AddUniqueIndexToReceipts < ActiveRecord::Migration[4.2]
   def change
     add_index :message_train_receipts, [:message_train_message_id, :recipient_type, :recipient_id], name: :message_recipient, unique: true
   end

data/db/migrate/20151124000820_create_message_train_unsubscribes.rb

@@ -1,4 +1,4 @@
-class CreateMessageTrainUnsubscribes < ActiveRecord::Migration
+class CreateMessageTrainUnsubscribes < ActiveRecord::Migration[4.2]
   def change
     create_table :message_train_unsubscribes do |t|
       t.references :recipient, polymorphic: true

data/db/migrate/20260827095938_remove_paperclip_columns_from_message_train_attachments.rb

@@ -0,0 +1,8 @@
+class RemovePaperclipColumnsFromMessageTrainAttachments < ActiveRecord::Migration[7.1]
+  def change
+    remove_column :message_train_attachments, :attachment_file_name, :string
+    remove_column :message_train_attachments, :attachment_content_type, :string
+    remove_column :message_train_attachments, :attachment_file_size, :integer
+    remove_column :message_train_attachments, :attachment_updated_at, :datetime
+  end
+end