[TASK] Migrate icon-element to @lit/task
@lit/task graduated from a 2-year @lit-labs phase, to an officially supported helper library for lit elements. We want to start using it as it embraces a tri-state model for async operations: * pending * complete * error …which is important in order to always handle error cases instead of silently ignoring them. Also existing or currently running requests are aborted and/or ignored when a newer state has already been activated. The icon-library deduplication caching strategy is refactored into a reusable helper instance that is a bit more generic and allows to abort (ref counted across duplicate requests) async operations. Commands executed: npm install --save @lit/task npm run build Resolves: #102308 Releases: main, 12.4 Change-Id: If068145a8bc87ae386af38aa6a63e4f19062345f Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/81683 Tested-by:core-ci <typo3@b13.com> Reviewed-by:
Benjamin Franzke <ben@bnf.dev> Tested-by:
Benjamin Franzke <ben@bnf.dev>
Showing
- Build/Gruntfile.js 1 addition, 0 deletionsBuild/Gruntfile.js
- Build/Sources/TypeScript/backend/element/icon-element.ts 21 additions, 13 deletionsBuild/Sources/TypeScript/backend/element/icon-element.ts
- Build/Sources/TypeScript/backend/icons.ts 30 additions, 77 deletionsBuild/Sources/TypeScript/backend/icons.ts
- Build/Sources/TypeScript/backend/tests/icons-test.ts 0 additions, 16 deletionsBuild/Sources/TypeScript/backend/tests/icons-test.ts
- Build/Sources/TypeScript/core/cache/dedupe-async-task.ts 95 additions, 0 deletionsBuild/Sources/TypeScript/core/cache/dedupe-async-task.ts
- Build/Sources/TypeScript/core/tests/cache/dedupe-async-task-test.ts 152 additions, 0 deletions...ces/TypeScript/core/tests/cache/dedupe-async-task-test.ts
- Build/package-lock.json 9 additions, 0 deletionsBuild/package-lock.json
- Build/package.json 1 addition, 0 deletionsBuild/package.json
- Build/web-test-runner.config.mjs 2 additions, 0 deletionsBuild/web-test-runner.config.mjs
- typo3/sysext/backend/Resources/Public/JavaScript/element/icon-element.js 9 additions, 5 deletions...ckend/Resources/Public/JavaScript/element/icon-element.js
- typo3/sysext/backend/Resources/Public/JavaScript/icons.js 2 additions, 2 deletionstypo3/sysext/backend/Resources/Public/JavaScript/icons.js
- typo3/sysext/core/Configuration/JavaScriptModules.php 2 additions, 0 deletionstypo3/sysext/core/Configuration/JavaScriptModules.php
- typo3/sysext/core/Resources/Public/JavaScript/Contrib/@lit/task/deep-equals.js 6 additions, 0 deletions...ources/Public/JavaScript/Contrib/@lit/task/deep-equals.js
- typo3/sysext/core/Resources/Public/JavaScript/Contrib/@lit/task/index.js 1 addition, 0 deletions...re/Resources/Public/JavaScript/Contrib/@lit/task/index.js
- typo3/sysext/core/Resources/Public/JavaScript/Contrib/@lit/task/task.js 6 additions, 0 deletions...ore/Resources/Public/JavaScript/Contrib/@lit/task/task.js
- typo3/sysext/core/Resources/Public/JavaScript/cache/dedupe-async-task.js 13 additions, 0 deletions...re/Resources/Public/JavaScript/cache/dedupe-async-task.js
Please register or sign in to comment