concertina 0.2.0 → 0.3.1

package/dist/index.cjs

@@ -34,7 +34,7 @@ function pinToScrollTop(el) {
   let parent = el.parentElement;
   while (parent) {
     const { overflowY } = getComputedStyle(parent);
-    if (overflowY === "auto" || overflowY === "scroll") {
+    if ((overflowY === "auto" || overflowY === "scroll") && parent.scrollHeight > parent.clientHeight) {
       const box = parent.getBoundingClientRect();
       const target = el.getBoundingClientRect();
       let stickyOffset = 0;

package/dist/index.d.cts

@@ -32,6 +32,11 @@ declare function useConcertina(): UseConcertinaReturn;
  * clearing any sticky headers. Only adjusts one container's
  * scrollTop. Never cascades to the viewport, which matters on
  * mobile where scrollIntoView pulls the whole page.
+ *
+ * Skips elements that have overflow: auto/scroll in CSS but
+ * don't actually scroll (scrollHeight <= clientHeight). Without
+ * this check, a non-scrolling ancestor with overflow-auto traps
+ * the walk and the real scroll container never gets adjusted.
  */
 declare function pinToScrollTop(el: HTMLElement | null): void;
 

package/dist/index.d.ts

@@ -32,6 +32,11 @@ declare function useConcertina(): UseConcertinaReturn;
  * clearing any sticky headers. Only adjusts one container's
  * scrollTop. Never cascades to the viewport, which matters on
  * mobile where scrollIntoView pulls the whole page.
+ *
+ * Skips elements that have overflow: auto/scroll in CSS but
+ * don't actually scroll (scrollHeight <= clientHeight). Without
+ * this check, a non-scrolling ancestor with overflow-auto traps
+ * the walk and the real scroll container never gets adjusted.
  */
 declare function pinToScrollTop(el: HTMLElement | null): void;
 

package/dist/index.js

@@ -13,7 +13,7 @@ function pinToScrollTop(el) {
   let parent = el.parentElement;
   while (parent) {
     const { overflowY } = getComputedStyle(parent);
-    if (overflowY === "auto" || overflowY === "scroll") {
+    if ((overflowY === "auto" || overflowY === "scroll") && parent.scrollHeight > parent.clientHeight) {
       const box = parent.getBoundingClientRect();
       const target = el.getBoundingClientRect();
       let stickyOffset = 0;

package/dist/styles.css

@@ -40,15 +40,15 @@
   }
 }
 
-/* When switching between items, suppress all animations so the layout
-   is immediately in its final state for scroll pinning.
+/* When switching between items, run animations instantly so layout
+   is in its final state for scroll pinning. Uses duration: 0s rather
+   than animation: none to avoid re-triggering the close animation
+   when data-switching is cleared after paint.
    data-switching is set by useConcertina(), cleared after paint. */
 [data-switching] .concertina-content[data-state="closed"] {
-  animation: none;
-  height: 0;
-  opacity: 0;
+  animation-duration: 0s;
 }
 
 [data-switching] .concertina-content[data-state="open"] {
-  animation: none;
+  animation-duration: 0s;
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "concertina",
-  "version": "0.2.0",
+  "version": "0.3.1",
   "description": "React hook for scroll-pinned Radix Accordion panels. Zero dependencies.",
   "type": "module",
   "main": "./dist/index.cjs",