Moved mark aware actions to the sidebar

Author: dutow

app/javascript/controllers/topics_aware_controller.js

@@ -4,10 +4,15 @@ export default class extends Controller {
   static values = {
     awareUrl: String,
     awareAllUrl: String,
+    confirmVisible: String,
+    confirmAll: String,
   }
 
   markVisibleAware(event) {
     event.preventDefault()
+    if (this.confirmVisibleValue && !window.confirm(this.confirmVisibleValue)) {
+      return
+    }
     const rows = Array.from(document.querySelectorAll("[data-topic-id][data-last-message-id]"))
     const payload = rows.map(row => ({
       topic_id: Number(row.dataset.topicId),
@@ -32,9 +37,7 @@ export default class extends Controller {
 
   markAllAware(event) {
     event.preventDefault()
-    if (!window.confirm("Mark ALL threads as aware up to now? This will clear the 'New' status everywhere.")) {
-      return
-    }
+    if (this.confirmAllValue && !window.confirm(this.confirmAllValue)) return
     fetch(this.awareAllUrlValue, {
       method: "POST",
       headers: {

app/views/topics/_sidebar.html.slim

@@ -1,5 +1,13 @@
 - content_for :sidebar do
   .sidebar
+
+    - if user_signed_in?
+      .sidebar-section
+        .sidebar-content
+          .topics-actions data-controller="topics-aware" data-topics-aware-aware-url-value=aware_bulk_topics_path data-topics-aware-aware-all-url-value=aware_all_topics_path data-topics-aware-confirm-visible-value="Mark the displayed threads as aware?" data-topics-aware-confirm-all-value="Mark ALL threads as aware up to now? This will clear the 'New' status everywhere."
+            button.mark-aware-button data-action="click->topics-aware#markVisibleAware" Mark displayed threads aware
+            button.mark-aware-button.secondary data-action="click->topics-aware#markAllAware" Mark everything up to now aware
+
     .sidebar-section
       h3.sidebar-heading Search
       .sidebar-search.sidebar-content

app/views/topics/index.html.slim

@@ -8,11 +8,6 @@
       = link_to "Clear", topics_path, class: "clear-tag-filter"
 
   .topics-table
-    - if user_signed_in?
-      .topics-actions data-controller="topics-aware" data-topics-aware-aware-url-value=aware_bulk_topics_path data-topics-aware-aware-all-url-value=aware_all_topics_path
-        button.mark-aware-button data-action="click->topics-aware#markVisibleAware" Mark displayed threads aware
-        button.mark-aware-button.secondary data-action="click->topics-aware#markAllAware" Mark everything up to now aware
-
     table
       thead
         tr