@stackoverflow/stacks 1.10.3 → 1.10.5

package/lib/atomic/typography.less

@@ -198,6 +198,10 @@ p {
     overflow-wrap: normal !important;
 }
 
+.ow-anywhere {
+    overflow-wrap: anywhere !important;
+}
+
 .ow-break-word {
     overflow-wrap: break-word !important;
 }

package/lib/components/expandable/expandable.a11y.test.ts

@@ -0,0 +1,27 @@
+import { html } from "@open-wc/testing";
+import { runComponentTests } from "../../test/test-utils";
+import "../../index";
+
+describe("expandable", () => {
+    runComponentTests({
+        type: "a11y",
+        baseClass: "s-expandable",
+        modifiers: {
+            global: ["is-expanded"],
+        },
+        children: {
+            default: `
+                <div class="s-expandable--content">
+                    <p class="mb8">Expandable: Lorem ipsum dolor sit amet, ex iudicabit necessitatibus usu, cetero laboramus concludaturque mel no, facilisis posidonium cu cum. Pro ex senserit dissentiunt, imperdiet intellegam at sed, ex pri dicit eruditi convenire. Est harum movet gubergren ei, errem dictas evertitur ea sit, at mei oratio eligendi. Ad vis legere possit, vis ne ipsum democritum appellantur. Nullam ancillae iudicabit his ad.</p>
+                </div>
+            `,
+        },
+        template: ({ component, testid }) => html`
+            <div class="ws2 p8" data-testid="${testid}">
+                <div><p>Before expandable content</p></div>
+                ${component}
+                <div><p>After expandable content</p></div>
+            </div>
+        `,
+    });
+});

package/lib/components/expandable/expandable.visual.test.ts

@@ -0,0 +1,27 @@
+import { html } from "@open-wc/testing";
+import { runComponentTests } from "../../test/test-utils";
+import "../../index";
+
+describe("expandable", () => {
+    runComponentTests({
+        type: "visual",
+        baseClass: "s-expandable",
+        modifiers: {
+            global: ["is-expanded"],
+        },
+        children: {
+            default: `
+                <div class="s-expandable--content">
+                    <p class="mb8">Expandable: Lorem ipsum dolor sit amet, ex iudicabit necessitatibus usu, cetero laboramus concludaturque mel no, facilisis posidonium cu cum. Pro ex senserit dissentiunt, imperdiet intellegam at sed, ex pri dicit eruditi convenire. Est harum movet gubergren ei, errem dictas evertitur ea sit, at mei oratio eligendi. Ad vis legere possit, vis ne ipsum democritum appellantur. Nullam ancillae iudicabit his ad.</p>
+                </div>
+            `,
+        },
+        template: ({ component, testid }) => html`
+            <div class="ws2 p8" data-testid="${testid}">
+                <div><p>Before expandable content</p></div>
+                ${component}
+                <div><p>After expandable content</p></div>
+            </div>
+        `,
+    });
+});

package/lib/components/link-preview/link-preview.a11y.test.ts

@@ -0,0 +1,55 @@
+import { runComponentTests } from "../../test/test-utils";
+import "../../index";
+
+const getChild = (child?: string): string => {
+    return `
+        <div class="s-link-preview--header">
+            <div class="s-link-preview--icon">👋</div>
+            <div>
+                <a href="#" class="s-link-preview--title">
+                    Using hooks for a simple fetch request and breaking the rules of hooks, unsure how?
+                </a>
+                <div class="s-link-preview--details">
+                    Issue submitted by Ricky Otero on <relative-time datetime="2019-08-12T04:05:22Z" title="Aug 12, 2019, 12:05 AM EDT">Aug 12, 2019</relative-time> • <strong>RESOLVED</strong>
+                </div>
+            </div>
+        </div>
+        ${
+            child
+                ? child
+                : `
+            <div class="s-link-preview--body">
+                <p>I'm trying to create a simple fetch with hooks from an AWS database. At the moment it errors out and the only reason I can see is because it breaks the rules of hooks but I'm not sure how. It's at the top level of this functional component and it's not called inside an event handler.</p>
+                <p>The result of this call (an array of user data), needs to be exported as a function and called in another file.</p>
+                <p>If anyone can spot something I have missed and can highlighted how I'm breaking the rules of hooks I'd be grateful!</p>
+                <p>Thanks!</p>
+            </div>
+        `
+        }
+        <div class="s-link-preview--footer">
+            <a href="#" class="s-link-preview--url">https://stackoverflow.atlassian.net/projects/SREREQ/queues/custom/1</a>
+            <a href="#" class="s-link-preview--misc">Privacy notice</a>
+        </div>
+    `;
+};
+describe("link preview", () => {
+    runComponentTests({
+        type: "a11y",
+        baseClass: "s-link-preview",
+        children: {
+            default: getChild(),
+            code: getChild(`
+                <div class="s-link-preview--code">
+                    <pre class="language-js s-code-block" tabindex="0"><code class="language-js s-code-block"><span class="hljs-meta">'use strict'</span>;</code></pre>
+                </div>
+            `),
+        },
+        // TODO: Most of those skipped tests should be fixed by the new Stacks 2.0 palette
+        skippedTestids: [
+            "s-link-preview-dark",
+            "s-link-preview-dark-code",
+            "s-link-preview-light",
+            "s-link-preview-light-code",
+        ],
+    });
+});

package/lib/components/link-preview/link-preview.visual.test.ts

@@ -0,0 +1,52 @@
+import { html } from "@open-wc/testing";
+import { runComponentTests } from "../../test/test-utils";
+import "../../index";
+
+const getChild = (child?: string): string => {
+    return `
+        <div class="s-link-preview--header">
+            <div class="s-link-preview--icon">👋</div>
+            <div>
+                <a href="#" class="s-link-preview--title">
+                    Using hooks for a simple fetch request and breaking the rules of hooks, unsure how?
+                </a>
+                <div class="s-link-preview--details">
+                    Issue submitted by Ricky Otero on <relative-time datetime="2019-08-12T04:05:22Z" title="Aug 12, 2019, 12:05 AM EDT">Aug 12, 2019</relative-time> • <strong>RESOLVED</strong>
+                </div>
+            </div>
+        </div>
+        ${
+            child
+                ? child
+                : `
+            <div class="s-link-preview--body">
+                <p>I'm trying to create a simple fetch with hooks from an AWS database. At the moment it errors out and the only reason I can see is because it breaks the rules of hooks but I'm not sure how. It's at the top level of this functional component and it's not called inside an event handler.</p>
+                <p>The result of this call (an array of user data), needs to be exported as a function and called in another file.</p>
+                <p>If anyone can spot something I have missed and can highlighted how I'm breaking the rules of hooks I'd be grateful!</p>
+                <p>Thanks!</p>
+            </div>
+        `
+        }
+        <div class="s-link-preview--footer">
+            <a href="#" class="s-link-preview--url">https://stackoverflow.atlassian.net/projects/SREREQ/queues/custom/1</a>
+            <a href="#" class="s-link-preview--misc">Privacy notice</a>
+        </div>
+    `;
+};
+describe("link preview", () => {
+    runComponentTests({
+        type: "visual",
+        baseClass: "s-link-preview",
+        children: {
+            default: getChild(),
+            code: getChild(`
+                <div class="s-link-preview--code">
+                    <pre class="language-js s-code-block" tabindex="0"><code class="language-js s-code-block"><span class="hljs-meta">'use strict'</span>;</code></pre>
+                </div>
+            `),
+        },
+        template: ({ component, testid }) => html`
+            <div class="ws8 p8" data-testid="${testid}">${component}</div>
+        `,
+    });
+});

package/lib/components/notice/notice.a11y.test.ts

@@ -0,0 +1,22 @@
+import { runComponentTests } from "../../test/test-utils";
+import "../../index";
+
+describe("notice", () => {
+    runComponentTests({
+        type: "a11y",
+        baseClass: "s-notice",
+        variants: ["info", "success", "warning", "danger"],
+        modifiers: {
+            primary: ["important"],
+        },
+        children: {
+            default: `Test notice`,
+        },
+        tag: "aside",
+        skippedTestids: [
+            /s-notice-dark/,
+            "s-notice-light-success-important",
+            "s-notice-light-warning-important",
+        ],
+    });
+});

package/lib/components/notice/notice.visual.test.ts

@@ -0,0 +1,26 @@
+import { runComponentTests } from "../../test/test-utils";
+import { html } from "@open-wc/testing";
+import "../../index";
+
+describe("notice", () => {
+    runComponentTests({
+        type: "visual",
+        baseClass: "s-notice",
+        variants: ["info", "success", "warning", "danger"],
+        modifiers: {
+            primary: ["important"],
+        },
+        attributes: {
+            ariaHidden: "false",
+        },
+        children: {
+            default: `Test notice`,
+        },
+        tag: "aside",
+        template: ({ component, testid }) => html`
+            <div class="d-inline-block p8" data-testid="${testid}">
+                ${component}
+            </div>
+        `,
+    });
+});

package/lib/components/pagination/pagination.a11y.test.ts

@@ -0,0 +1,22 @@
+import { runComponentTests } from "../../test/test-utils";
+import "../../index";
+
+describe("pagination", () => {
+    runComponentTests({
+        type: "a11y",
+        baseClass: "s-pagination",
+        children: {
+            default: `
+                <a class="s-pagination--item" href="#">Prev</a>
+                <a class="s-pagination--item" href="#">1</a>
+                <span class="s-pagination--item is-selected" aria-current="page">2</span>
+                <a class="s-pagination--item" href="#">3</a>
+                <span class="s-pagination--item s-pagination--item__clear">…</span>
+                <a class="s-pagination--item" href="#">100</a>
+                <a class="s-pagination--item" href="#">Next</a>
+            `,
+        },
+        // TODO: Most of those skipped tests should be fixed by the new Stacks 2.0 palette
+        skippedTestids: ["s-pagination-dark", "s-pagination-light"],
+    });
+});

package/lib/components/pagination/pagination.visual.test.ts

@@ -0,0 +1,26 @@
+import { html } from "@open-wc/testing";
+import { runComponentTests } from "../../test/test-utils";
+import "../../index";
+
+describe("pagination", () => {
+    runComponentTests({
+        type: "visual",
+        baseClass: "s-pagination",
+        children: {
+            default: `
+                <a class="s-pagination--item" href="#">Prev</a>
+                <a class="s-pagination--item" href="#">1</a>
+                <span class="s-pagination--item is-selected" aria-current="page">2</span>
+                <a class="s-pagination--item" href="#">3</a>
+                <span class="s-pagination--item s-pagination--item__clear">…</span>
+                <a class="s-pagination--item" href="#">100</a>
+                <a class="s-pagination--item" href="#">Next</a>
+            `,
+        },
+        template: ({ component, testid }) => html`
+            <div class="d-inline-block p8" data-testid="${testid}">
+                ${component}
+            </div>
+        `,
+    });
+});

package/lib/components/spinner/spinner.a11y.test.ts

@@ -0,0 +1,15 @@
+import { runComponentTests } from "../../test/test-utils";
+import "../../index";
+
+describe("spinner", () => {
+    runComponentTests({
+        type: "a11y",
+        baseClass: "s-spinner",
+        modifiers: {
+            primary: ["xs", "sm", "md", "lg"],
+        },
+        children: {
+            default: `<div class="v-visible-sr">Loading…</div>`,
+        },
+    });
+});

package/lib/components/spinner/spinner.visual.test.ts

@@ -0,0 +1,43 @@
+import { html } from "@open-wc/testing";
+import { runComponentTests } from "../../test/test-utils";
+import "../../index";
+
+// eslint-disable-next-line @typescript-eslint/no-explicit-any
+const template = ({ component, testid }: any) => html`
+    <div class="d-inline-block p8" data-testid="${testid}">${component}</div>
+`;
+describe("spinner", () => {
+    // default, sizes
+    runComponentTests({
+        type: "visual",
+        baseClass: "s-spinner",
+        modifiers: {
+            primary: ["xs", "sm", "md", "lg"],
+        },
+        children: {
+            default: `<div class="v-visible-sr">Loading…</div>`,
+        },
+        template,
+    });
+    // applied font color
+    runComponentTests({
+        type: "visual",
+        baseClass: "s-spinner",
+        modifiers: {
+            global: ["fc-theme-primary"],
+        },
+        children: {
+            default: `<div class="v-visible-sr">Loading…</div>`,
+        },
+        template,
+    });
+    // .is-loading
+    runComponentTests({
+        type: "visual",
+        baseClass: "is-loading",
+        children: {
+            default: `Loading…`,
+        },
+        template,
+    });
+});

package/lib/components/toast/toast.a11y.test.ts

@@ -0,0 +1,35 @@
+import { runComponentTests } from "../../test/test-utils";
+import { html } from "@open-wc/testing";
+import "../../index";
+
+describe("toast > notice", () => {
+    // This is a test of notice component wrapped in a toast component
+    runComponentTests({
+        type: "a11y",
+        baseClass: "s-notice", // s-toast is a wrapper around s-notice
+        variants: ["info", "success", "warning", "danger"],
+        modifiers: {
+            primary: ["important"],
+        },
+        children: {
+            toast: `<span id="message">Test toast</span>`,
+        },
+        tag: "aside",
+        template: ({ component, testid }) => html`
+            <div
+                class="s-toast ps-static t0 l0"
+                role="alertdialog"
+                aria-hidden="false"
+                aria-labelledby="message"
+                data-testid="${testid}"
+            >
+                ${component}
+            </div>
+        `,
+        skippedTestids: [
+            /s-notice-dark.*-toast/,
+            "s-notice-light-success-important-toast",
+            "s-notice-light-warning-important-toast",
+        ],
+    });
+});

package/lib/components/toast/toast.visual.test.ts

@@ -11,16 +11,20 @@ describe("toast > notice", () => {
         modifiers: {
             primary: ["important"],
         },
-        attributes: {
-            ariaHidden: "false",
-        },
         children: {
-            toast: "Test toast",
+            toast: `<span id="message">Test toast</span>`,
         },
         tag: "aside",
         template: ({ component, testid }) => html`
-            <div data-testid="${testid}" class="s-toast" aria-hidden="false">
-                ${component}
+            <div class="d-inline-block p8" data-testid="${testid}">
+                <div
+                    class="s-toast ps-static t0 l0"
+                    role="alertdialog"
+                    aria-hidden="false"
+                    aria-labelledby="message"
+                >
+                    ${component}
+                </div>
             </div>
         `,
     });

package/lib/components/toggle-switch/toggle-switch.less

@@ -102,6 +102,7 @@
         background-repeat: no-repeat;
         border-radius: 1000px;
         cursor: pointer;
+        flex-shrink: 0;
         height: var(--su-static24);
         margin: 0; // guard against production adding 5px of margin to these
         transition: background-position 0.2s ease;

package/package.json

@@ -5,7 +5,7 @@
         "type": "git",
         "url": "https://github.com/StackExchange/Stacks.git"
     },
-    "version": "1.10.3",
+    "version": "1.10.5",
     "files": [
         "dist",
         "lib"
@@ -50,38 +50,38 @@
         "@stackoverflow/stacks-icons": "^5.5.0",
         "@testing-library/dom": "^9.3.1",
         "@testing-library/user-event": "^14.4.3",
-        "@typescript-eslint/eslint-plugin": "^6.4.0",
-        "@typescript-eslint/parser": "^6.4.0",
+        "@typescript-eslint/eslint-plugin": "^6.5.0",
+        "@typescript-eslint/parser": "^6.6.0",
         "@web/dev-server-esbuild": "^0.4.1",
         "@web/dev-server-rollup": "^0.5.2",
-        "@web/test-runner": "^0.17.0",
+        "@web/test-runner": "^0.17.1",
         "@web/test-runner-playwright": "^0.10.1",
         "@web/test-runner-visual-regression": "^0.8.2",
         "apca-w3": "^0.1.9",
-        "concurrently": "^8.2.0",
+        "concurrently": "^8.2.1",
         "css-loader": "^6.8.1",
         "cssnano": "^6.0.1",
         "docsearch.js": "^2.6.3",
         "eleventy-plugin-highlightjs": "^1.1.0",
         "eleventy-plugin-nesting-toc": "^1.3.0",
-        "eslint": "^8.47.0",
+        "eslint": "^8.48.0",
         "eslint-config-prettier": "^9.0.0",
         "eslint-plugin-no-unsanitized": "^4.0.2",
-        "jquery": "^3.7.0",
+        "jquery": "^3.7.1",
         "less-loader": "^11.1.3",
         "list.js": "^2.3.1",
         "markdown-it": "^13.0.1",
         "mini-css-extract-plugin": "^2.7.6",
         "postcss-less": "^6.0.0",
         "postcss-loader": "^7.3.3",
-        "prettier": "^3.0.2",
+        "prettier": "^3.0.3",
         "rollup-plugin-postcss": "^4.0.2",
         "stylelint": "^15.10.3",
         "stylelint-config-recommended": "^13.0.0",
         "stylelint-config-standard": "^34.0.0",
         "terser-webpack-plugin": "^5.3.9",
         "ts-loader": "^9.4.4",
-        "typescript": "^5.1.6",
+        "typescript": "^5.2.2",
         "webpack": "^5.88.2",
         "webpack-cli": "^5.1.4",
         "webpack-merge": "^5.9.0"