collavre 0.8.2 → 0.8.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: f586b6b08d3de11a9c9212f4ff7e11113143bdf0dbbd5963668e9613041b3f02
-  data.tar.gz: 5ad7d024a1c53506c0ef1ee08731041de3a66842de48f26912b5eb56051c7029
+  metadata.gz: 82e806ce0c2a6e03029c43d817fb83a6c8114e04bf13be10b05b16fefbeb253c
+  data.tar.gz: bfa7142c80cd7868c4ad9375a7b9f96ceed0b9b35ad031a6962f92a9d655f029
 SHA512:
-  metadata.gz: 187df2c9024426107e264eb14d7af711cc10255f87eec3380287465c2f0b19198b4a67f50b8d8e3920105166d268af57e1a9a525cb9c49ea84e11a2031ac9503
-  data.tar.gz: eb58fba86ff137079da0f834c3f1b82e2fbdede8c5e2d930ebff1602b6c38e45744d6a31bb01d568142b4bee42b5bd0bc68a5bf6e77f6187993f901bee3449ad
+  metadata.gz: e443416101117e0e3440f60d889f4c317fc19d59e5d3e92fdc511f1caf0fee52470e3e37b036709fbc63eea01d266c622e25ded80dc584d77c7fd0e2feed257b
+  data.tar.gz: 33e264807609298b92f8786f755bce2ef7846e653d6d5b4c7ff0eb1c267cf0273a56753d8e3758cf99e8300d14f1f6cfbde99b21e9f99e39645e44d3a0ec58e6

data/app/assets/stylesheets/collavre/comments_popup.css

@@ -10,9 +10,11 @@
     min-width: 200px;
     min-height: 200px;
     background: var(--surface-bg);
-    border: 1px solid var(--border-color);
+    border: var(--border-1) solid var(--color-active);
     border-radius: var(--radius-3);
-    box-shadow: var(--shadow-3);
+    box-shadow: var(--shadow-3),
+                0 0 6px color-mix(in srgb, var(--color-active) 25%, transparent),
+                0 0 14px color-mix(in srgb, var(--color-active) 12%, transparent);
     padding: 1em;
     flex-direction: column;
     transition: top 0.25s ease, left 0.25s ease, width 0.25s ease, height 0.25s ease,

data/app/assets/stylesheets/collavre/creatives.css

@@ -440,6 +440,13 @@ creative-tree-row:not([expanded]) .creative-toggle-btn {
     background-color: var(--border-drag-over);
 }
 
+creative-tree-row.chat-active .creative-row {
+    border: var(--border-1) solid var(--color-active);
+    border-radius: var(--radius-2);
+    box-shadow: 0 0 6px color-mix(in srgb, var(--color-active) 25%, transparent),
+                0 0 14px color-mix(in srgb, var(--color-active) 12%, transparent);
+}
+
 .creative-row:hover,
 .creative-row.level-1:hover,
 .creative-row.level-2:hover,

data/app/assets/stylesheets/collavre/dark_mode.css

@@ -79,12 +79,7 @@ body.dark-mode {
 /* ============================================================================
  * BASE ELEMENT STYLES
  * ============================================================================ */
-html {
-  scroll-behavior: smooth;
-}
-
 @media (prefers-reduced-motion: reduce) {
-  html { scroll-behavior: auto; }
 
   /* Collapse decorative transitions to instant; state changes still
      apply but without visible motion (hover bg, theme switch, etc.). */
@@ -302,3 +297,28 @@ input[type="datetime-local"]::-webkit-datetime-edit-minute-field {
 .inline-block {
   display: inline-block;
 }
+
+/* --- Dark mode: stronger glow for visibility on dark backgrounds --- */
+body.dark-mode #comments-popup {
+  box-shadow: var(--shadow-3),
+              0 0 8px color-mix(in srgb, var(--color-active) 45%, transparent),
+              0 0 20px color-mix(in srgb, var(--color-active) 25%, transparent);
+}
+
+body.dark-mode creative-tree-row.chat-active .creative-row {
+  box-shadow: 0 0 8px color-mix(in srgb, var(--color-active) 45%, transparent),
+              0 0 20px color-mix(in srgb, var(--color-active) 25%, transparent);
+}
+
+@media (prefers-color-scheme: dark) {
+  body:not(.light-mode):not(.dark-mode) #comments-popup {
+    box-shadow: var(--shadow-3),
+                0 0 8px color-mix(in srgb, var(--color-active) 45%, transparent),
+                0 0 20px color-mix(in srgb, var(--color-active) 25%, transparent);
+  }
+
+  body:not(.light-mode):not(.dark-mode) creative-tree-row.chat-active .creative-row {
+    box-shadow: 0 0 8px color-mix(in srgb, var(--color-active) 45%, transparent),
+                0 0 20px color-mix(in srgb, var(--color-active) 25%, transparent);
+  }
+}

data/app/javascript/controllers/comments/popup_controller.js

@@ -157,6 +157,8 @@ export default class extends Controller {
     this.element.dataset.canComment = canComment ? 'true' : 'false'
     this.titleTarget.textContent = snippet
 
+    this._markChatActiveRow(resolvedCreativeId)
+
     this.prepareSize()
 
     this.showPopup()
@@ -184,6 +186,8 @@ export default class extends Controller {
     this.element.dataset.canComment = canComment ? 'true' : 'false'
     this.titleTarget.textContent = snippet
 
+    this._markChatActiveRow(resolvedCreativeId)
+
     this.showPopup()
 
     await this.notifyChildControllers({ creativeId: resolvedCreativeId, canComment })
@@ -268,6 +272,8 @@ export default class extends Controller {
       }
     }))
 
+    this._clearChatActiveRow()
+
     this.element.style.display = 'none'
     this.element.classList.remove('open')
     this.element.style.width = ''
@@ -855,4 +861,17 @@ export default class extends Controller {
       this.openFromUrlTimeout = null
     }
   }
+
+  _markChatActiveRow(creativeId) {
+    this._clearChatActiveRow()
+    if (!creativeId) return
+    const row = document.querySelector(`creative-tree-row[creative-id="${creativeId}"]`)
+    if (row) row.classList.add('chat-active')
+  }
+
+  _clearChatActiveRow() {
+    document.querySelectorAll('creative-tree-row.chat-active').forEach(el => {
+      el.classList.remove('chat-active')
+    })
+  }
 }

data/app/javascript/controllers/creatives/expansion_controller.js

@@ -100,6 +100,7 @@ export default class extends Controller {
     const childrenDiv = this.childrenContainerFor(row)
     this.ensureLoaded(row, childrenDiv).then((hasChildren) => {
       if (!hasChildren || !childrenDiv) {
+        row.hasChildren = false
         this.collapseRow(row, { persist: false })
         return
       }
@@ -177,6 +178,13 @@ export default class extends Controller {
 
   syncInitialState(row) {
     const childrenDiv = this.childrenContainerFor(row)
+
+    // If the row claims it has children but no children container exists,
+    // correct the hasChildren flag so the chevron is hidden.
+    if (row.hasChildren && !childrenDiv) {
+      row.hasChildren = false
+    }
+
     const shouldExpand =
       this.allExpanded ||
       row.expanded ||

data/lib/collavre/version.rb

@@ -1,3 +1,3 @@
 module Collavre
-  VERSION = "0.8.2"
+  VERSION = "0.8.3"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: collavre
 version: !ruby/object:Gem::Version
-  version: 0.8.2
+  version: 0.8.3
 platform: ruby
 authors:
 - Collavre