Merge branch 'main' into element/ak-spinner

* main:
  Accepted suggestion from @GirlBossRush
  Update vitest.config.js
  Update src/ak-button/ak-button.component.ts
  Update src/ak-button/ak-button.template.ts

Author: kensternberg-authentik

src/ak-button/ak-button.component.ts

@@ -150,7 +150,7 @@ export class Button extends LitElement implements IButton {
     #internals = InternalsController.of(this, { role: "button" });
 
     @property({ reflect: true, type: Boolean })
-    disabled = false;
+    public disabled = false;
 
     get #disabled() {
         return this.matches(":disabled") || this.disabled;

src/ak-button/ak-button.template.ts

@@ -17,12 +17,16 @@ type ButtonLinkProps = Pick<ButtonProps, "disabled" | "onClick" | "onKeydown"> &
     download?: string;
 };
 
+/**
+ * @remarks
+ * The ARIA rules for disabled links specifies that "to communicate a link as ‘disabled,’ remove
+ * the `href` attribute and style accordingly." Hence the `href=${ifDefined(...)}`.
+ *
+ * @see {@link https://www.w3.org/TR/html-aria/#example-communicate-a-disabled-link-with-aria | ARIA Disabled Link Rules}
+ */
+
 export function linkTemplate(props: ButtonLinkProps) {
     const { href, target, disabled, download, rel, onClick, onKeydown } = props;
-
-    // The ARIA rules for disabled links specifies that "to communicate a link as ‘disabled,’ remove
-    // the `href` attribute and style accordingly." Hence the `href=${ifDefined(...)}`. See:
-    // https://www.w3.org/TR/html-aria/#example-communicate-a-disabled-link-with-aria
     return html`<a
         id="main"
         href=${ifDefined(disabled ? null : href)}

vitest.config.js

@@ -1,4 +1,4 @@
-import path, { dirname } from "node:path";
+import { dirname, resolve } from "node:path";
 import { fileURLToPath } from "node:url";
 
 import { playwright } from "@vitest/browser-playwright";
@@ -8,7 +8,7 @@ const __dirname = dirname(fileURLToPath(import.meta.url));
 
 const $ = ({ page, provider }, selector) => {
     if (provider.name !== "playwright") {
-        throw new Error(`provider ${provider.name} is not supported`);
+        throw new Error(`Provider "${provider.name}" is not supported`);
     }
     return page.locator(selector);
 };
@@ -29,7 +29,7 @@ export default defineConfig(({ mode }) => ({
     },
     test: {
         alias: {
-            "@goauthentik/elements": path.resolve(__dirname, "./dist"),
+            "@goauthentik/elements": resolve(__dirname, "./dist"),
         },
         dir: "./dist",
         include: ["**/*.test.js"],