jqtree 1.9.0 → 1.9.1

package/bower.json

@@ -1,6 +1,6 @@
 {
     "name": "jqTree",
-    "version": "1.9.0",
+    "version": "1.9.1",
     "main": ["jqtree.css", "jqtree-circle.png", "tree.jquery.js"],
     "dependencies": {
         "jquery": ">=1.9"

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "jqtree",
-    "version": "1.9.0",
+    "version": "1.9.1",
     "description": "Tree widget for jQuery",
     "keywords": [
         "jquery-plugin",
@@ -36,48 +36,48 @@
         "@babel/core": "^8.0.1",
         "@babel/preset-env": "^8.0.2",
         "@babel/preset-typescript": "^8.0.1",
-        "@cspell/eslint-plugin": "^10.0.1",
-        "@eslint/js": "^9.39.5",
-        "@playwright/test": "^1.62.0",
+        "@cspell/eslint-plugin": "^10.2.1",
+        "@eslint/js": "^10.0.1",
+        "@playwright/test": "^1.62.1",
         "@rollup/plugin-babel": "^7.1.0",
         "@rollup/plugin-node-resolve": "^16.0.3",
         "@rollup/plugin-terser": "^1.0.0",
         "@testing-library/dom": "^10.4.1",
-        "@testing-library/jest-dom": "^7.0.0",
-        "@testing-library/user-event": "^14.6.1",
+        "@testing-library/jest-dom": "^7.0.1",
+        "@testing-library/user-event": "^14.6.7",
         "@types/debug": "^4.1.13",
         "@types/jquery": "^4.0.1",
-        "@types/node": "^26.1.1",
-        "@vitest/coverage-istanbul": "^4.1.10",
-        "@vitest/eslint-plugin": "^1.6.24",
+        "@types/node": "^26.4.1",
+        "@vitest/coverage-istanbul": "^4.1.11",
+        "@vitest/eslint-plugin": "^1.6.27",
         "autoprefixer": "^10.5.4",
-        "axe-core": "^4.12.1",
+        "axe-core": "^4.13.0",
         "babel-plugin-istanbul": "^8.0.0",
-        "eslint": "^9.39.5",
+        "eslint": "^10.9.1",
         "eslint-import-resolver-typescript": "^4.4.5",
         "eslint-plugin-import-x": "^4.17.1",
-        "eslint-plugin-jest-dom": "^5.5.0",
-        "eslint-plugin-jest-extended": "^3.0.1",
-        "eslint-plugin-perfectionist": "^5.10.0",
-        "eslint-plugin-playwright": "^2.10.5",
+        "eslint-plugin-jest-dom": "^5.10.1",
+        "eslint-plugin-jest-extended": "github:Aesthermortis/eslint-plugin-jest-extended#v4.0.0",
+        "eslint-plugin-perfectionist": "^5.11.0",
+        "eslint-plugin-playwright": "^2.11.0",
         "eslint-plugin-testing-library": "^7.16.2",
         "jest-extended": "^7.0.0",
-        "jsdom": "^29.1.1",
+        "jsdom": "^30.0.1",
         "jsdom-testing-mocks": "^1.16.0",
         "jsonfile": "^6.2.1",
         "lodash": "^4.18.1",
         "msw": "^2.15.0",
-        "postcss": "^8.5.23",
+        "postcss": "^8.5.28",
         "postcss-cli": "^11.0.1",
-        "postcss-import": "^16.1.1",
+        "postcss-import": "^17.0.0",
         "postcss-load-config": "^6.0.1",
-        "postcss-nested": "^7.0.2",
+        "postcss-nested": "^8.0.1",
         "prettier": "^3.9.6",
-        "rollup": "^4.62.2",
+        "rollup": "^4.63.1",
         "rollup-plugin-serve": "^3.0.0",
         "tslib": "^2.8.1",
         "typescript": "^6.0.3",
-        "typescript-eslint": "^8.65.0",
-        "vitest": "^4.1.10"
+        "typescript-eslint": "^8.69.0",
+        "vitest": "^4.1.11"
     }
 }

package/pnpm-workspace.yaml

@@ -1,4 +1,5 @@
 allowBuilds:
-  esbuild: true
-  msw: true
-  unrs-resolver: true
+    esbuild: true
+    msw: true
+    unrs-resolver: true
+    eslint-plugin-jest-extended@https://codeload.github.com/Aesthermortis/eslint-plugin-jest-extended/tar.gz/0b24376411f45e9f3f11f9104cc9966ebc80dd8c: true

package/src/nodeElement/folderElement.ts

@@ -59,7 +59,7 @@ class FolderElement extends NodeElement {
         const doClose = (): void => {
             this.element.classList.add("jqtree-closed");
 
-            const titleSpan = this.getTitleSpan();
+            const titleSpan = this._getTitleSpan();
             titleSpan.setAttribute("aria-expanded", "false");
 
             this._triggerEvent("tree.close", {
@@ -67,7 +67,7 @@ class FolderElement extends NodeElement {
             });
         };
 
-        const ul = this.getUl();
+        const ul = this._getUl();
 
         if (slide) {
             slideUp(ul, animationSpeed, doClose);
@@ -102,7 +102,7 @@ class FolderElement extends NodeElement {
         const doOpen = (): void => {
             this.element.classList.remove("jqtree-closed");
 
-            const titleSpan = this.getTitleSpan();
+            const titleSpan = this._getTitleSpan();
             titleSpan.setAttribute("aria-expanded", "true");
 
             if (onFinished) {
@@ -114,7 +114,7 @@ class FolderElement extends NodeElement {
             });
         };
 
-        const ul = this.getUl();
+        const ul = this._getUl();
 
         if (slide) {
             slideDown(ul, animationSpeed, doOpen);
@@ -124,7 +124,7 @@ class FolderElement extends NodeElement {
         }
     }
 
-    protected mustShowBorderDropHint(position: Position): boolean {
+    protected _mustShowBorderDropHint(position: Position): boolean {
         return !this.node.is_open && position === "inside";
     }
 

package/src/nodeElement/index.ts

@@ -33,7 +33,7 @@ class NodeElement {
     }
 
     public addDropHint(position: Position): DropHint {
-        if (this.mustShowBorderDropHint(position)) {
+        if (this._mustShowBorderDropHint(position)) {
             return new BorderDropHint(this.element, this._getScrollLeft());
         } else {
             return new GhostDropHint(this.node, this.element, position);
@@ -43,7 +43,7 @@ class NodeElement {
     public deselect(): void {
         this.element.classList.remove("jqtree-selected");
 
-        const titleSpan = this.getTitleSpan();
+        const titleSpan = this._getTitleSpan();
         titleSpan.removeAttribute("tabindex");
         titleSpan.setAttribute("aria-selected", "false");
 
@@ -61,7 +61,7 @@ class NodeElement {
     public select(mustSetFocus: boolean): void {
         this.element.classList.add("jqtree-selected");
 
-        const titleSpan = this.getTitleSpan();
+        const titleSpan = this._getTitleSpan();
         const tabIndex = this._tabIndex;
 
         // Check for null or undefined
@@ -76,17 +76,17 @@ class NodeElement {
         }
     }
 
-    protected getTitleSpan(): HTMLSpanElement {
+    protected _getTitleSpan(): HTMLSpanElement {
         return this.element.querySelector(
             ":scope > .jqtree-element > span.jqtree-title",
         ) as HTMLSpanElement;
     }
 
-    protected getUl(): HTMLUListElement {
+    protected _getUl(): HTMLUListElement {
         return this.element.querySelector(":scope > ul") as HTMLUListElement;
     }
 
-    protected mustShowBorderDropHint(position: Position): boolean {
+    protected _mustShowBorderDropHint(position: Position): boolean {
         return position === "inside";
     }
 }

package/src/scrollHandler/containerScrollParent.ts

@@ -14,15 +14,15 @@ export default class ContainerScrollParent extends ScrollParent {
     public stopScrolling() {
         super.stopScrolling();
 
-        this.horizontalScrollDirection = undefined;
-        this.verticalScrollDirection = undefined;
+        this._horizontalScrollDirection = undefined;
+        this._verticalScrollDirection = undefined;
     }
 
-    protected getNewHorizontalScrollDirection(
+    protected _getNewHorizontalScrollDirection(
         pageX: number,
     ): HorizontalScrollDirection | undefined {
-        const scrollParentOffset = getElementPosition(this.container);
-        const containerWidth = this.container.getBoundingClientRect().width;
+        const scrollParentOffset = getElementPosition(this._container);
+        const containerWidth = this._container.getBoundingClientRect().width;
 
         const rightEdge = scrollParentOffset.left + containerWidth;
         const leftEdge = scrollParentOffset.left;
@@ -38,7 +38,7 @@ export default class ContainerScrollParent extends ScrollParent {
         return undefined;
     }
 
-    protected getNewVerticalScrollDirection(
+    protected _getNewVerticalScrollDirection(
         pageY: number,
     ): undefined | VerticalScrollDirection {
         if (pageY < this._getScrollParentTop()) {
@@ -55,7 +55,7 @@ export default class ContainerScrollParent extends ScrollParent {
     private _getScrollParentBottom() {
         if (this._scrollParentBottom == null) {
             const containerHeight =
-                this.container.getBoundingClientRect().height;
+                this._container.getBoundingClientRect().height;
             this._scrollParentBottom =
                 this._getScrollParentTop() + containerHeight;
         }
@@ -64,7 +64,7 @@ export default class ContainerScrollParent extends ScrollParent {
     }
 
     private _getScrollParentTop() {
-        this._scrollParentTop ??= getOffsetTop(this.container);
+        this._scrollParentTop ??= getOffsetTop(this._container);
 
         return this._scrollParentTop;
     }

package/src/scrollHandler/documentScrollParent.ts

@@ -35,10 +35,10 @@ export default class DocumentScrollParent extends ScrollParent {
         this._documentScrollWidth = undefined;
     }
 
-    protected getNewHorizontalScrollDirection(
+    protected _getNewHorizontalScrollDirection(
         pageX: number,
     ): HorizontalScrollDirection | undefined {
-        const scrollLeft = this.container.scrollLeft;
+        const scrollLeft = this._container.scrollLeft;
         const windowWidth = window.innerWidth;
 
         const isNearRightEdge = pageX > windowWidth - 20;
@@ -55,10 +55,10 @@ export default class DocumentScrollParent extends ScrollParent {
         return undefined;
     }
 
-    protected getNewVerticalScrollDirection(
+    protected _getNewVerticalScrollDirection(
         pageY: number,
     ): undefined | VerticalScrollDirection {
-        const scrollTop = this.container.scrollTop;
+        const scrollTop = this._container.scrollTop;
         const distanceTop = pageY - scrollTop;
 
         if (distanceTop < 20) {
@@ -76,28 +76,28 @@ export default class DocumentScrollParent extends ScrollParent {
 
     private _canScrollDown() {
         return (
-            this.container.scrollTop + this.container.clientHeight <
+            this._container.scrollTop + this._container.clientHeight <
             this._getDocumentScrollHeight()
         );
     }
 
     private _canScrollRight() {
         return (
-            this.container.scrollLeft + this.container.clientWidth <
+            this._container.scrollLeft + this._container.clientWidth <
             this._getDocumentScrollWidth()
         );
     }
 
     private _getDocumentScrollHeight() {
         // Store the original scroll height because the scroll height can increase when the drag element is moved beyond the scroll height.
-        this._documentScrollHeight ??= this.container.scrollHeight;
+        this._documentScrollHeight ??= this._container.scrollHeight;
 
         return this._documentScrollHeight;
     }
 
     private _getDocumentScrollWidth() {
         // Store the original scroll width because the scroll width can increase when the drag element is moved beyond the scroll width.
-        this._documentScrollWidth ??= this.container.scrollWidth;
+        this._documentScrollWidth ??= this._container.scrollWidth;
 
         return this._documentScrollWidth;
     }

package/src/scrollHandler/scrollParent.ts

@@ -7,33 +7,33 @@ interface ConstructorParams {
 }
 
 export abstract class ScrollParent {
-    protected container: HTMLElement;
-    protected horizontalScrollDirection?: HorizontalScrollDirection;
-    protected horizontalScrollTimeout?: number;
+    protected _container: HTMLElement;
+    protected _horizontalScrollDirection?: HorizontalScrollDirection;
+    protected _horizontalScrollTimeout?: number;
 
-    protected refreshHitAreas: () => void;
-    protected verticalScrollDirection?: VerticalScrollDirection;
-    protected verticalScrollTimeout?: number;
+    protected _refreshHitAreas: () => void;
+    protected _verticalScrollDirection?: VerticalScrollDirection;
+    protected _verticalScrollTimeout?: number;
 
     constructor({ container, refreshHitAreas }: ConstructorParams) {
-        this.container = container;
-        this.refreshHitAreas = refreshHitAreas;
+        this._container = container;
+        this._refreshHitAreas = refreshHitAreas;
     }
 
     public checkHorizontalScrolling(pageX: number): void {
         const newHorizontalScrollDirection =
-            this.getNewHorizontalScrollDirection(pageX);
+            this._getNewHorizontalScrollDirection(pageX);
 
-        if (this.horizontalScrollDirection !== newHorizontalScrollDirection) {
-            this.horizontalScrollDirection = newHorizontalScrollDirection;
+        if (this._horizontalScrollDirection !== newHorizontalScrollDirection) {
+            this._horizontalScrollDirection = newHorizontalScrollDirection;
 
-            if (this.horizontalScrollTimeout != null) {
-                window.clearTimeout(this.horizontalScrollTimeout);
+            if (this._horizontalScrollTimeout != null) {
+                window.clearTimeout(this._horizontalScrollTimeout);
             }
 
             if (newHorizontalScrollDirection) {
-                this.horizontalScrollTimeout = window.setTimeout(
-                    this.scrollHorizontally.bind(this),
+                this._horizontalScrollTimeout = window.setTimeout(
+                    this._scrollHorizontally.bind(this),
                     40,
                 );
             }
@@ -42,19 +42,19 @@ export abstract class ScrollParent {
 
     public checkVerticalScrolling(pageY: number) {
         const newVerticalScrollDirection =
-            this.getNewVerticalScrollDirection(pageY);
+            this._getNewVerticalScrollDirection(pageY);
 
-        if (this.verticalScrollDirection !== newVerticalScrollDirection) {
-            this.verticalScrollDirection = newVerticalScrollDirection;
+        if (this._verticalScrollDirection !== newVerticalScrollDirection) {
+            this._verticalScrollDirection = newVerticalScrollDirection;
 
-            if (this.verticalScrollTimeout != null) {
-                window.clearTimeout(this.verticalScrollTimeout);
-                this.verticalScrollTimeout = undefined;
+            if (this._verticalScrollTimeout != null) {
+                window.clearTimeout(this._verticalScrollTimeout);
+                this._verticalScrollTimeout = undefined;
             }
 
             if (newVerticalScrollDirection) {
-                this.verticalScrollTimeout = window.setTimeout(
-                    this.scrollVertically.bind(this),
+                this._verticalScrollTimeout = window.setTimeout(
+                    this._scrollVertically.bind(this),
                     40,
                 );
             }
@@ -62,56 +62,56 @@ export abstract class ScrollParent {
     }
 
     public getScrollLeft(): number {
-        return this.container.scrollLeft;
+        return this._container.scrollLeft;
     }
 
     public scrollToY(top: number): void {
-        this.container.scrollTop = top;
+        this._container.scrollTop = top;
     }
 
     public stopScrolling() {
-        this.horizontalScrollDirection = undefined;
-        this.verticalScrollDirection = undefined;
+        this._horizontalScrollDirection = undefined;
+        this._verticalScrollDirection = undefined;
     }
 
-    protected abstract getNewHorizontalScrollDirection(
+    protected abstract _getNewHorizontalScrollDirection(
         pageX: number,
     ): HorizontalScrollDirection | undefined;
-    protected abstract getNewVerticalScrollDirection(
+    protected abstract _getNewVerticalScrollDirection(
         pageY: number,
     ): undefined | VerticalScrollDirection;
 
-    protected scrollHorizontally() {
-        if (!this.horizontalScrollDirection) {
+    protected _scrollHorizontally() {
+        if (!this._horizontalScrollDirection) {
             return;
         }
 
-        const distance = this.horizontalScrollDirection === "left" ? -20 : 20;
-        this.container.scrollBy({
+        const distance = this._horizontalScrollDirection === "left" ? -20 : 20;
+        this._container.scrollBy({
             behavior: "instant",
             left: distance,
             top: 0,
         });
 
-        this.refreshHitAreas();
+        this._refreshHitAreas();
 
-        setTimeout(this.scrollHorizontally.bind(this), 40);
+        setTimeout(this._scrollHorizontally.bind(this), 40);
     }
 
-    protected scrollVertically() {
-        if (!this.verticalScrollDirection) {
+    protected _scrollVertically() {
+        if (!this._verticalScrollDirection) {
             return;
         }
 
-        const distance = this.verticalScrollDirection === "top" ? -20 : 20;
-        this.container.scrollBy({
+        const distance = this._verticalScrollDirection === "top" ? -20 : 20;
+        this._container.scrollBy({
             behavior: "instant",
             left: 0,
             top: distance,
         });
 
-        this.refreshHitAreas();
+        this._refreshHitAreas();
 
-        setTimeout(this.scrollVertically.bind(this), 40);
+        setTimeout(this._scrollVertically.bind(this), 40);
     }
 }

package/src/tree.jquery.d.ts

@@ -24,8 +24,8 @@ interface IJQTreeOptions {
     onCanMove?: (node: INode) => boolean;
     onCanSelectNode?: (node: INode) => boolean;
     onCreateLi?: (node: INode, el: JQuery, isSelected: boolean) => void;
-    onDragMove?: (node: INode, event: JQuery.Event | Touch) => void;
-    onDragStop?: (node: INode, event: JQuery.Event | Touch) => void;
+    onDragMove?: (node: INode, event: Event | Touch) => void;
+    onDragStop?: (node: INode, event: Event | Touch) => void;
     onGetStateFromStorage?: () => string;
     onIsMoveHandle?: (el: JQuery) => boolean;
     onLoadFailed?: (response: JQuery.jqXHR) => void;
@@ -79,6 +79,7 @@ interface IJQTreePlugin {
     (behavior: "getStateFromStorage"): INode | null;
     (behavior: "getTree"): INode;
     (behavior: "getVersion"): string;
+    (behavior: "isDragging"): boolean;
     (behavior: "isNodeSelected", node: INode): boolean;
     (behavior: "loadData", data: NodeData[], parentNode?: INode): JQuery;
     (