@bbki.ng/bb-msg-history 0.13.1 → 0.14.0

package/dist/component.d.ts

@@ -5,9 +5,6 @@ export declare class BBMsgHistory extends HTMLElement {
     private _lastAuthor;
     private _lastGroupTimestamp;
     private _scrollButtonVisible;
-    private _userHasScrolledManually;
-    private _isProgrammaticScroll;
-    private _lastScrollTop;
     static get observedAttributes(): string[];
     constructor();
     attributeChangedCallback(name: string): void;

package/dist/component.js

@@ -13,9 +13,6 @@ export class BBMsgHistory extends HTMLElement {
         this._userAuthors = new Map();
         this._lastAuthor = '';
         this._scrollButtonVisible = false;
-        this._userHasScrolledManually = false;
-        this._isProgrammaticScroll = false;
-        this._lastScrollTop = 0;
         this.attachShadow({ mode: 'open' });
     }
     attributeChangedCallback(name) {
@@ -138,16 +135,10 @@ export class BBMsgHistory extends HTMLElement {
         }
         // Smooth scroll to bottom (skip in infinite mode)
         if (!this.hasAttribute('infinite')) {
-            // Mark this as a programmatic scroll so the scroll handler ignores it
-            this._isProgrammaticScroll = true;
             container.scrollTo({
                 top: container.scrollHeight,
                 behavior: 'smooth',
             });
-            // Reset the flag after smooth scroll animation completes (~300ms)
-            setTimeout(() => {
-                this._isProgrammaticScroll = false;
-            }, 300);
             // Hide scroll button since we're scrolling to bottom
             if (this._scrollButtonVisible) {
                 this._scrollButtonVisible = false;
@@ -308,12 +299,7 @@ export class BBMsgHistory extends HTMLElement {
             const scrollButton = this.shadowRoot.querySelector('.scroll-to-bottom');
             const isInfinite = this.hasAttribute('infinite');
             if (container && !isInfinite) {
-                // Mark as programmatic scroll to prevent triggering user scroll detection
-                this._isProgrammaticScroll = true;
                 container.scrollTop = container.scrollHeight;
-                requestAnimationFrame(() => {
-                    this._isProgrammaticScroll = false;
-                });
                 this._setupScrollTracking(container, scrollButton, { skipInitialCheck: true });
             }
             if (scrollButton && !isInfinite) {
@@ -349,21 +335,11 @@ export class BBMsgHistory extends HTMLElement {
     }
     _setupScrollTracking(container, button, options) {
         const checkScrollPosition = () => {
-            // Ignore programmatic scrolls - they don't indicate user intent
-            if (this._isProgrammaticScroll)
-                return;
-            // Mark that user has manually scrolled
-            if (!this._userHasScrolledManually) {
-                this._userHasScrolledManually = true;
-            }
-            const currentScrollTop = container.scrollTop;
-            const isScrollingUp = currentScrollTop < this._lastScrollTop;
-            this._lastScrollTop = currentScrollTop;
             const threshold = 50; // pixels from bottom
             const isAtBottom = container.scrollHeight - container.scrollTop - container.clientHeight < threshold;
             const hasOverflow = container.scrollHeight > container.clientHeight;
-            // Only show button when: user has scrolled, is not at bottom, is scrolling up
-            const shouldShow = !isAtBottom && hasOverflow && this._userHasScrolledManually && isScrollingUp;
+            // Show button when not at bottom and content has overflow
+            const shouldShow = !isAtBottom && hasOverflow;
             if (shouldShow !== this._scrollButtonVisible) {
                 this._scrollButtonVisible = shouldShow;
                 // Only toggle button visibility if button exists
@@ -378,8 +354,6 @@ export class BBMsgHistory extends HTMLElement {
                 }));
             }
         };
-        // Initialize last scroll position
-        this._lastScrollTop = container.scrollTop;
         // Check initial state unless skipped
         if (!options?.skipInitialCheck) {
             checkScrollPosition();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bbki.ng/bb-msg-history",
-  "version": "0.13.1",
+  "version": "0.14.0",
   "description": "A chat-style message history web component",
   "main": "dist/index.js",
   "type": "module",
@@ -23,19 +23,6 @@
     "dist",
     "src"
   ],
-  "scripts": {
-    "start": "tsc -w",
-    "preview": "python3 -m http.server 8000",
-    "prepare": "tsc && cp dist/index.js dist/index.dev.js && terser dist/index.js --compress --mangle --source-map -o dist/index.js",
-    "build": "tsc && cp dist/index.js dist/index.dev.js && terser dist/index.js --compress --mangle --source-map -o dist/index.js",
-    "lint": "eslint src/**/*.ts",
-    "lint:fix": "eslint src/**/*.ts --fix",
-    "format": "prettier --write \"src/**/*.ts\"",
-    "format:check": "prettier --check \"src/**/*.ts\"",
-    "test": "vitest run",
-    "test:watch": "vitest",
-    "release": "release-it"
-  },
   "lint-staged": {
     "*.ts": [
       "eslint --fix",
@@ -60,5 +47,17 @@
     "terser": "^5.46.0",
     "typescript": "^5.9.3",
     "vitest": "^3.2.4"
+  },
+  "scripts": {
+    "start": "tsc -w",
+    "preview": "python3 -m http.server 8000",
+    "build": "tsc && cp dist/index.js dist/index.dev.js && terser dist/index.js --compress --mangle --source-map -o dist/index.js",
+    "lint": "eslint src/**/*.ts",
+    "lint:fix": "eslint src/**/*.ts --fix",
+    "format": "prettier --write \"src/**/*.ts\"",
+    "format:check": "prettier --check \"src/**/*.ts\"",
+    "test": "vitest run",
+    "test:watch": "vitest",
+    "release": "release-it"
   }
-}
+}

package/src/component.ts

@@ -12,9 +12,6 @@ export class BBMsgHistory extends HTMLElement {
   private _lastAuthor = '';
   private _lastGroupTimestamp: string | undefined;
   private _scrollButtonVisible = false;
-  private _userHasScrolledManually = false;
-  private _isProgrammaticScroll = false;
-  private _lastScrollTop = 0;
 
   static get observedAttributes() {
     return ['theme', 'loading', 'hide-scroll-bar', 'infinite', 'hide-scroll-button'];
@@ -176,19 +173,11 @@ export class BBMsgHistory extends HTMLElement {
 
     // Smooth scroll to bottom (skip in infinite mode)
     if (!this.hasAttribute('infinite')) {
-      // Mark this as a programmatic scroll so the scroll handler ignores it
-      this._isProgrammaticScroll = true;
-
       container.scrollTo({
         top: container.scrollHeight,
         behavior: 'smooth',
       });
 
-      // Reset the flag after smooth scroll animation completes (~300ms)
-      setTimeout(() => {
-        this._isProgrammaticScroll = false;
-      }, 300);
-
       // Hide scroll button since we're scrolling to bottom
       if (this._scrollButtonVisible) {
         this._scrollButtonVisible = false;
@@ -386,12 +375,7 @@ export class BBMsgHistory extends HTMLElement {
       const isInfinite = this.hasAttribute('infinite');
 
       if (container && !isInfinite) {
-        // Mark as programmatic scroll to prevent triggering user scroll detection
-        this._isProgrammaticScroll = true;
         container.scrollTop = container.scrollHeight;
-        requestAnimationFrame(() => {
-          this._isProgrammaticScroll = false;
-        });
         this._setupScrollTracking(container, scrollButton, { skipInitialCheck: true });
       }
 
@@ -435,24 +419,12 @@ export class BBMsgHistory extends HTMLElement {
     options?: { skipInitialCheck?: boolean }
   ): void {
     const checkScrollPosition = () => {
-      // Ignore programmatic scrolls - they don't indicate user intent
-      if (this._isProgrammaticScroll) return;
-
-      // Mark that user has manually scrolled
-      if (!this._userHasScrolledManually) {
-        this._userHasScrolledManually = true;
-      }
-
-      const currentScrollTop = container.scrollTop;
-      const isScrollingUp = currentScrollTop < this._lastScrollTop;
-      this._lastScrollTop = currentScrollTop;
-
       const threshold = 50; // pixels from bottom
       const isAtBottom =
         container.scrollHeight - container.scrollTop - container.clientHeight < threshold;
       const hasOverflow = container.scrollHeight > container.clientHeight;
-      // Only show button when: user has scrolled, is not at bottom, is scrolling up
-      const shouldShow = !isAtBottom && hasOverflow && this._userHasScrolledManually && isScrollingUp;
+      // Show button when not at bottom and content has overflow
+      const shouldShow = !isAtBottom && hasOverflow;
 
       if (shouldShow !== this._scrollButtonVisible) {
         this._scrollButtonVisible = shouldShow;
@@ -472,9 +444,6 @@ export class BBMsgHistory extends HTMLElement {
       }
     };
 
-    // Initialize last scroll position
-    this._lastScrollTop = container.scrollTop;
-
     // Check initial state unless skipped
     if (!options?.skipInitialCheck) {
       checkScrollPosition();