@qtoggle/qui 1.18.3 → 1.19.1

package/.github/workflows/main.yml

@@ -2,6 +2,10 @@ name: Main
 
 on: push
 
+permissions:
+  id-token: write
+  contents: write
+
 jobs:
   lint:
     name: Lint
@@ -24,6 +28,10 @@ jobs:
         uses: actions/checkout@v4
       - name: Replace source version
         uses: qtoggle/actions-common/actions/replace-source-version@v1
+      - name: Setup NodeJS
+        uses: actions/setup-node@v4
+        with:
+          node-version: '18'
       - name: Install
         run: |
           npm install && rm -rf docs/* js/lib/* && npx jsdoc -c jsdoc.conf.json
@@ -55,19 +63,17 @@ jobs:
       - name: Setup NodeJS
         uses: actions/setup-node@v4
         with:
-          node-version: '18'
+          node-version: '20'
       - name: Publish to NPM
         run: |
-          echo '//registry.npmjs.org/:_authToken=${NPM_TOKEN}' > ~/.npmrc
+          npm install -g npm@latest &&
           npm publish
-        env:
-          NPM_TOKEN: ${{ secrets.NPM_TOKEN }}
       - name: Fetch python dist folder
         uses: actions/download-artifact@v4
         with:
           name: python-package-dist
           path: dist/
-      - name: Publish package
+      - name: Publish to PyPI
         uses: pypa/gh-action-pypi-publish@release/v1
         with:
           user: __token__

package/js/forms/form.js

@@ -4,6 +4,7 @@ import $ from '$qui/lib/jquery.module.js'
 import {AssertionError}      from '$qui/base/errors.js'
 import {mix}                 from '$qui/base/mixwith.js'
 import * as Theme            from '$qui/theme.js'
+import Debouncer             from '$qui/utils/debouncer.js'
 import {asap}                from '$qui/utils/misc.js'
 import * as ObjectUtils      from '$qui/utils/object.js'
 import {ProgressViewMixin}   from '$qui/views/common-views/common-views.js'
@@ -78,7 +79,7 @@ class Form extends mix().with(ViewMixin, StructuredViewMixin, ProgressViewMixin)
         /* Last known validity state */
         this._isValid = null
         this._validationCache = {}
-        this._updateValidationStateASAPHandle = null
+        this._updateValidationStateDebouncer = new Debouncer(() => this.updateValidationState())
 
         this._fieldsByName = {}
 
@@ -737,16 +738,7 @@ class Form extends mix().with(ViewMixin, StructuredViewMixin, ProgressViewMixin)
      * Calls {@link qui.forms.Form#updateValidationState} asap, preventing multiple unnecessary successive calls.
      */
     updateValidationStateASAP() {
-        if (this._updateValidationStateASAPHandle) {
-            clearTimeout(this._updateValidationStateASAPHandle)
-        }
-
-        this._updateValidationStateASAPHandle = asap(function () {
-
-            this._updateValidationStateASAPHandle = null
-            this.updateValidationState()
-
-        }.bind(this))
+        this._updateValidationStateDebouncer.call()
     }
 
     /**

package/js/index.js

@@ -44,6 +44,8 @@ const logger = Logger.get('qui')
  */
 export let whenReady = new ConditionVariable()
 
+let globalErrorMessageForm = null
+
 
 function initConfig() {
     /* Look for the main script name */
@@ -177,6 +179,11 @@ function configureGlobalErrorHandling() {
             return
         }
 
+        /* Don't show another global error message on top of an existing one */
+        if (globalErrorMessageForm != null) {
+            return
+        }
+
         logger.error(`unhandled promise rejection: ${e.reason || '<unspecified reason>'}`)
         if (e.reason != null) {
             logger.error(e.reason)
@@ -186,8 +193,16 @@ function configureGlobalErrorHandling() {
         let msg = gettext('An unexpected error occurred.')
         msg += '<br>'
         msg += gettext('Application reloading is recommended.')
-        new StickySimpleMessageForm({type: 'error', message: msg}).show()
-        e.preventDefault()
+        globalErrorMessageForm = new StickySimpleMessageForm(
+            {
+                type: 'error',
+                message: msg,
+                onClose: function () {
+                    globalErrorMessageForm = null
+                }
+            }
+        )
+        globalErrorMessageForm.show()
     })
 }
 

package/js/lists/common-items/icon-label-list-item.js

@@ -7,9 +7,6 @@ import ListItem   from '../list-item.js'
 import * as Lists from '../lists.js'
 
 
-const PHRASE_SPLIT_REGEX = /[^a-zA-Z0-9]/
-
-
 /**
  * A list item made of an icon and a label.
  * @alias qui.lists.commonitems.IconLabelListItem
@@ -43,13 +40,9 @@ class IconLabelListItem extends mix(ListItem).with(IconLabelViewMixin) {
 
     getMatchPhrase() {
         if (this._matchPhrase == null) {
-            this._matchPhrase =
-            this._matchPhrase = [
-                ...this.getLabel().split(PHRASE_SPLIT_REGEX),
-                ...this.getSubLabel().split(PHRASE_SPLIT_REGEX)
-            ]
+            this._matchPhrase = []
 
-            /* Also consider the entire label as is */
+            /* Consider the entire label as is */
             if (this.getLabel()) {
                 this._matchPhrase.push(this.getLabel().toLowerCase())
             }

package/js/utils/debouncer.js

@@ -0,0 +1,44 @@
+
+/**
+ * A class that debounces function calls, ensuring that a function is not called more often than a specified delay.
+ * @alias qui.utils.Debouncer
+ */
+class Debouncer {
+
+    /**
+     * @constructs
+     * @param {Function} func the function to debounce
+     * @param {Number} [delay] the debouncing delay, in milliseconds (defaults to `0`)
+     */
+    constructor(func, delay = 0) {
+        this._func = func
+        this._delay = delay
+        this._timeoutHandle = null
+    }
+
+    /**
+     * Call function ensuring debouncing condition. Any previous pending call is cancelled.
+     * Arguments are passed to the function when called.
+     */
+    call(...args) {
+        if (this._timeoutHandle !== null) {
+            clearTimeout(this._timeoutHandle)
+        }
+
+        this._timeoutHandle = setTimeout(function () {
+            this._timeoutHandle = null
+            this._func(...args)
+        }.bind(this), this._delay)
+    }
+
+    /**
+     * Tell if there is a pending call.
+     * @return {Boolean}
+     */
+    isPending() {
+        return this._timeoutHandle != null
+    }
+
+}
+
+export default Debouncer

package/less/all.less

@@ -0,0 +1,39 @@
+@import "base.less";
+@import "main-ui.less";
+@import "no-effects.less";
+@import "visibility-manager.less";
+@import "icons.less";
+
+@import "widgets/check-button.less";
+@import "widgets/choice-buttons.less";
+@import "widgets/color-combo.less";
+@import "widgets/combo.less";
+@import "widgets/common.less";
+@import "widgets/common-buttons.less";
+@import "widgets/labels.less";
+@import "widgets/progress-disk.less";
+@import "widgets/slider.less";
+@import "widgets/input.less";
+@import "widgets/updown.less";
+@import "widgets/various.less";
+
+@import "messages.less";
+@import "global-glass.less";
+@import "structured-view.less";
+@import "progress-view.less";
+@import "icon-label-view.less";
+
+@import "pages/breadcrumbs.less";
+@import "pages/common-pages.less";
+@import "pages/page.less";
+
+@import "forms/common-fields.less";
+@import "forms/common-forms.less";
+@import "forms/form.less";
+@import "forms/form-button.less";
+@import "forms/form-field.less";
+
+@import "lists.less";
+@import "tables.less";
+
+@import "theme.less";

package/package.json

@@ -1,11 +1,14 @@
 {
   "name": "@qtoggle/qui",
   "description": "A JavaScript UI library with batteries included.",
-  "version": "1.18.3",
+  "version": "1.19.1",
   "author": {
     "name": "Calin Crisan",
     "email": "ccrisan@gmail.com"
   },
+  "repository": {
+    "url": "https://github.com/qtoggle/qui.git"
+  },
   "keywords": [],
   "license": "Apache-2.0",
   "dependencies": {
@@ -16,30 +19,30 @@
     "pepjs": "*"
   },
   "devDependencies": {
-    "@babel/core": "^7",
-    "@babel/plugin-proposal-class-properties": "^7",
-    "@babel/preset-env": "^7",
+    "@babel/core": "*",
+    "@babel/plugin-proposal-class-properties": "*",
+    "@babel/preset-env": "*",
     "@babel/eslint-parser": "*",
     "babel-loader": "^8",
     "css-loader": "^3",
     "eslint": "^9",
-    "expose-loader": "^0",
-    "file-loader": "^6",
-    "glob": "^7",
+    "expose-loader": "^1",
+    "file-loader": "*",
+    "glob": "*",
     "jsdoc": "^3",
-    "jsdoc-export-default-interop": "^0",
-    "jsdoc-typeof-plugin": "^1",
-    "less": "^3",
+    "jsdoc-export-default-interop": "*",
+    "jsdoc-typeof-plugin": "*",
+    "less": "*",
     "less-loader": "^6",
-    "mini-css-extract-plugin": "^0",
-    "optimize-css-assets-webpack-plugin": "^5",
+    "mini-css-extract-plugin": "^1",
+    "optimize-css-assets-webpack-plugin": "*",
     "string-replace-loader": "^2",
     "terser-webpack-plugin": "^3",
     "webpack": "^4",
     "webpack-cli": "^3",
-    "webpack-fix-style-only-entries": "^0",
-    "webpack-inject-plugin": "^1",
-    "webpack-shell-plugin": "^0"
+    "webpack-fix-style-only-entries": "*",
+    "webpack-inject-plugin": "*",
+    "webpack-shell-plugin": "*"
   },
   "scripts": {
     "postinstall": "scripts/postinstall.sh"

package/pyproject.toml

@@ -1,11 +1,11 @@
 [project]
 name = "qui-server"
-version = "1.18.3"
+version = "1.19.1"
 description = "A fully fledged qToggle implementation written in Python"
 authors = [
     {name = "Calin Crisan", email = "ccrisan@gmail.com"},
 ]
-requires-python = "==3.10.*"
+requires-python = ">=3.11"
 readme = "README.md"
 license = {text = "Apache 2.0"}
 dependencies = [
@@ -28,7 +28,6 @@ package = true
 
 [tool.ruff]
 line-length = 120
-target-version = "py310"
 lint.extend-select = ["I", "RUF022"]
 lint.isort.lines-after-imports = 2
 lint.isort.lines-between-types = 1

package/webpack/webpack-common.js

@@ -125,7 +125,7 @@ function makeLessRule({type, theme, isProduction, appName, appFullPath, quiFullP
     if (isProduction) {
         loaders = [
             MiniCssExtractPlugin.loader,
-            cssLoader,
+            cssLoader
         ]
     }
     else { /* Development mode */
@@ -232,8 +232,8 @@ function makeConfig({theme, isProduction, appName, appFullPath, extraAliases, cs
 
     let excludeLessRegex = new RegExp('theme-(' + THEMES.join('|') + ')\\.less', 'i')
     let cssRequirements = [
-        ...requireFromDir(LESS_REGEX, quiLessPath, excludeLessRegex),
-        ...requireFromDir(LESS_REGEX, appLessPath, excludeLessRegex)
+        quiLessPath + '/all.less',
+        appLessPath + '/all.less'
     ]
 
     /* This is needed because FixStyleOnlyEntriesPlugin() needs resources for theme entries to be different */