Adding a basic attachment sidebar

The latest attachment is the first, everything is in descending order.
This should be the more useful ordering for the typical "I want to see
the latest patch version" usecase.

Author: dutow

app/assets/stylesheets/components/sidebar.css

@@ -57,3 +57,53 @@
     text-decoration: underline;
   }
 }
+
+.sidebar .attachments-list {
+  list-style: none;
+  padding-left: 0;
+  margin: var(--spacing-3) 0;
+}
+
+.sidebar .attachments-list li + li {
+  margin-top: var(--spacing-2);
+}
+
+.sidebar .attachment-link {
+  display: flex;
+  justify-content: space-between;
+  gap: var(--spacing-2);
+  color: var(--color-text-link);
+  text-decoration: none;
+  font-weight: var(--font-weight-medium);
+}
+
+.sidebar .attachment-link:hover {
+  text-decoration: underline;
+}
+
+.sidebar .attachment-target {
+  font-weight: var(--font-weight-bold);
+}
+
+.sidebar .attachment-date {
+  color: var(--color-text-secondary);
+  font-size: var(--font-size-sm);
+}
+
+.sidebar .attachment-name {
+  display: block;
+  padding-left: var(--spacing-4);
+  margin-top: var(--spacing-1);
+}
+
+.sidebar .attachment-more {
+  display: block;
+  padding-left: var(--spacing-4);
+  color: var(--color-text-secondary);
+  font-size: var(--font-size-sm);
+  margin-top: var(--spacing-1);
+}
+
+.sidebar .attachment-names {
+  color: var(--color-text-secondary);
+}

app/controllers/topics_controller.rb

@@ -33,7 +33,7 @@ def new_topics_count
   end
 
   def show
-    messages_scope = @topic.messages.includes(:sender, reply_to: :sender)
+    messages_scope = @topic.messages.includes(:sender, { reply_to: :sender }, :attachments)
 
     @messages = messages_scope.order(created_at: :asc)
     @message_numbers = @messages.each_with_index.to_h { |msg, idx| [msg.id, idx + 1] }

app/views/topics/_message.html.slim

@@ -38,7 +38,7 @@
       = render_message_body(message.body)
 
     - if message.attachments.any?
-      .message-attachments
+      .message-attachments id="message-#{message.id}-attachments"
         h4 Attachments:
         - message.attachments.each do |attachment|
           - if attachment.patch?

app/views/topics/show.html.slim

@@ -15,6 +15,27 @@
         - else
           p No participants yet
 
+    - attachment_messages = @messages.select { |msg| msg.attachments.any? }
+    details.sidebar-section open=true
+      summary.sidebar-heading Attachments
+      .sidebar-section
+        - if attachment_messages.any?
+          ul.attachments-list
+            - attachment_messages.sort_by(&:created_at).reverse_each do |msg|
+              - number = @message_numbers[msg.id]
+              li
+                = link_to "#message-#{msg.id}-attachments", class: "attachment-link" do
+                  span.attachment-target = "##{number}"
+                  span.attachment-date = smart_time_display(msg.created_at)
+                - names = msg.attachments.map(&:file_name)
+                - names.first(2).each do |name|
+                  .attachment-name = name
+                - if names.size > 2
+                  .attachment-more = "+ #{names.size - 2} more"
+        - else
+          p No attachments in this thread
+
+
     details.sidebar-section open=true
       summary.sidebar-heading Thread Outline
       - if @thread_outline.present?