Merge branch 'main' into fix/tile/remove-base

.changeset/chilly-plums-shake.md

@@ -0,0 +1,6 @@
+---
+"@patternfly/eslint-config-elements": major
+"@patternfly/eslint-plugin-elements": major
+---
+
+Provide ESLint flat config. Upgrade to ESLint 9.0 to use.

.changeset/deep-mammals-rescue.md

@@ -0,0 +1,4 @@
+---
+"@patternfly/pfe-tools": patch
+---
+Dev Server: update icon and theme colours

.changeset/eleven-llamas-follow.md

@@ -0,0 +1,4 @@
+---
+"@patternfly/elements": patch
+---
+`<pf-select>`: fixed computed button label when the placeholder attribute is present

.changeset/label-close-remove.md

@@ -0,0 +1,17 @@
+---
+"@patternfly/elements": major
+---
+`<pf-label>`: when clicking close button, `close` event is fired.
+Now, if that event is not cancelled, the label will remove itself from the document.
+
+To restore previous behaviour:
+
+```js
+import { LabelCloseEvent } from '@patternfly/elements/pf-label/pf-label.js';
+label.addEventListener('close', function(event) {
+  if (event instanceof LabelCloseEvent) {
+    event.preventDefault();
+    return false;
+  }
+});
+```

.changeset/remove-base-clipboard-copy.md

@@ -0,0 +1,35 @@
+---
+"@patternfly/elements": major
+---
+`<pf-clipboard-copy>`: Removed `BaseClipboardCopy` class.
+Reimplement (recommended) or extend `PfClipboardCopy`.
+Renames `AvatarLoadEvent` to `PfAvatarLoadEvent` and moves it to `pf-avatar.js`.
+
+**Before**:
+
+```js
+import {
+  ClipboardCopyCopiedEvent
+} from '@patternfly/elements/pf-clipboard-copy/BaseClipboardCopy.js';
+
+addEventListener('copy', function(event) {
+  if (event instanceof ClipboardCopyCopiedEvent) {
+    // ...
+  }
+});
+```
+
+**After**:
+
+```js
+import {
+  PfClipboardCopyCopiedEvent
+} from '@patternfly/elements/pf-clipboard-copy/pf-clipboard-copy.js';
+
+addEventListener('copy', function(event) {
+  if (event instanceof PfClipboardCopyCopiedEvent) {
+    // ...
+  }
+});
+```
+

.changeset/remove-base-label.md

@@ -0,0 +1,4 @@
+---
+"@patternfly/elements": major
+---
+`<pf-label>`: Removed `BaseLabel` class. Reimplement (recommended) or extend `PfLabel`.

.changeset/remove-base-switch.md

@@ -0,0 +1,4 @@
+---
+"@patternfly/elements": major
+---
+`<pf-switch>`: Removed `BaseSwitch` class. Reimplement (recommended) or extend `PfSwitch`.

.changeset/remove-baseavatar.md

@@ -0,0 +1,29 @@
+---
+"@patternfly/elements": major
+---
+`<pf-avatar>`: Removed `BaseAvatar` class. Reimplement (recommended) or extend `PfAvatar`.
+Renames `AvatarLoadEvent` to `PfAvatarLoadEvent` and moves it to `pf-avatar.js`.
+
+**Before**:
+
+```js
+import { AvatarLoadEvent } from '@patternfly/elements/pf-avatar/BaseAvatar.js';
+
+addEventListener('load', function(event) {
+  if (event instanceof AvatarLoadEvent) {
+    // ...
+  }
+});
+```
+
+**After**:
+
+```js
+import { PfAvatarLoadEvent } from '@patternfly/elements/pf-avatar/pf-avatar.js';
+
+addEventListener('load', function(event) {
+  if (event instanceof PfAvatarLoadEvent) {
+    // ...
+  }
+});
+```

.changeset/remove-basebadge.md

@@ -0,0 +1,4 @@
+---
+"@patternfly/elements": major
+---
+`<pf-badge>`: Removed `BaseBadge` class. Reimplement (recommended) or extend `PfBadge`.

.changeset/remove-basecodeblock.md

@@ -0,0 +1,4 @@
+---
+"@patternfly/elements": major
+---
+`<pf-code-block>`: Removed `BaseCodeBlock` class. Reimplement (recommended) or extend `PfCodeBlock`.

.changeset/remove-basespinner.md

@@ -0,0 +1,4 @@
+---
+"@patternfly/elements": major
+---
+`<pf-spinner>`: Removed `BaseSpinner` class. Reimplement (recommended) or extend `PfSpinner`.

.changeset/rude-kiwis-live.md

@@ -0,0 +1,4 @@
+---
+"@patternfly/pfe-tools": patch
+---
+Update typescript types

.commitlintrc.cjs

@@ -1,5 +1,5 @@
-const fs = require("fs");
-const path = require("path");
+const fs = require('fs');
+const path = require('path');
 const normalizeWorkspace = x =>
   fs.readdirSync(path.join(__dirname, x)).map(x => x.replace('pf-', ''));
 
@@ -23,5 +23,5 @@ module.exports = {
       ...normalizeWorkspace('core'),
       ...normalizeWorkspace('tools'),
     ]],
-  }
+  },
 };

.github/workflows/bundle.yml

@@ -25,7 +25,7 @@ jobs:
       - uses: actions/checkout@v3
       - uses: actions/setup-node@v3
         with:
-          node-version: '18'
+          node-version: '20'
           cache: npm
 
       - name: Bundle

.github/workflows/preview.yml

@@ -13,7 +13,7 @@ jobs:
         uses: actions/checkout@v3
       - uses: actions/setup-node@v3
         with:
-          node-version: '18'
+          node-version: '20'
           cache: npm
 
       - name: Verify JSPM URL

.github/workflows/release.yml

@@ -14,7 +14,7 @@ jobs:
       - uses: actions/checkout@v3
       - uses: actions/setup-node@v3
         with:
-          node-version: '18'
+          node-version: '20'
           cache: npm
 
       - run: npm ci --prefer-offline

.github/workflows/tests.yml

@@ -40,9 +40,8 @@ jobs:
   #   steps:
   #     - name: Debugging context variables
   #       run: echo "$GITHUB_CONTEXT"
-
-  test:
-    name: Run test suite (Web Test Runner)
+  lint:
+    name: Lint files
     runs-on: ubuntu-latest
     steps:
       - name: Checkout repository
@@ -52,7 +51,7 @@ jobs:
       - name: Configure node version
         uses: actions/setup-node@v3
         with:
-          node-version: '18'
+          node-version: '20'
           cache: npm
 
       - name: Install dependencies
@@ -62,13 +61,32 @@ jobs:
         id: lint
         run: npm run lint
 
+
+  test:
+    name: Run test suite (Web Test Runner)
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout repository
+        uses: actions/checkout@v3
+
+      # Configures the node version used on GitHub-hosted runners
+      - name: Configure node version
+        uses: actions/setup-node@v3
+        with:
+          node-version: '20'
+          cache: npm
+
+      - name: Install dependencies
+        run: npm ci --prefer-offline
+
+      - name: install playwright
+        run: npx playwright install
+
       - name: Run tests
         run: npm test
-        if: ${{ always() }}
 
       - name: JUnit Report Action
         uses: mikepenz/[email protected]
-        if: ${{ always() }}
         with:
           report_paths: test-results/test-results.xml
           fail_on_failure: true # fail the actions run if the tests failed
@@ -79,10 +97,8 @@ jobs:
     strategy:
       matrix:
         node:
-          - '18'
-          - '19'
-          # https://github.com/TypeStrong/ts-node/issues/1997
-          # - '20'
+          - '20'
+          - '21'
     if: |
       github.event_name == 'workflow_dispatch'
       ||   github.event_name == 'push'
@@ -126,8 +142,9 @@ jobs:
   validate:
     name: Validate successful build on main
     needs:
-      - build
+      - lint
       - test
+      - build
     if: ${{ always() }}
     runs-on: ubuntu-latest
     steps:

.github/workflows/visual-regression.yml

@@ -29,7 +29,7 @@ jobs:
       - uses: actions/checkout@v3
       - uses: actions/setup-node@v3
         with:
-          node-version: '18'
+          node-version: '20'
           cache: npm
       - run: npm ci --prefer-offline
 

.gitignore

@@ -65,12 +65,12 @@ tools/*/test/**/*.png
 !core/*/demo/*
 !tools/*/demo/*
 
-elements/pf-icon/icons
 !elements/pf-icon/demo/icons/**/*.js
 
 !scripts/**/*.js
 
 !tools/eslint-plugin/index.js
+!tools/eslint-config/**/*.js
 
 *.tgz
 custom-elements.json

.husky/commit-msg

@@ -1,4 +1 @@
-#!/bin/sh
-. "$(dirname "$0")/_/husky.sh"
-
 npx --no -- commitlint --edit "$1"

.husky/pre-commit

@@ -1,2 +1 @@
-#!/bin/sh
 npx lint-staged

.nvmrc

@@ -1 +1 @@
-v18.12.1
+v20.10.0