@ionic/core 7.5.8-dev.11701876182.1e71e7a0 → 7.5.8-dev.11702056888.11188f48

package/dist/collection/components/{picker-column-internal/test/basic/picker-column-internal.e2e.js → picker-column/test/basic/picker-column.e2e.js}

@@ -7,20 +7,20 @@ import { configs, test } from "../../../../utils/test/playwright/index";
  * This behavior does not vary across modes/directions.
  */
 configs({ modes: ['ios'], directions: ['ltr'] }).forEach(({ title, config }) => {
-    test.describe(title('picker-column-internal'), () => {
+    test.describe(title('picker-column'), () => {
         test.beforeEach(async ({ page }) => {
-            await page.goto('/src/components/picker-column-internal/test/basic', config);
+            await page.goto('/src/components/picker-column/test/basic', config);
         });
         test('should render a picker item for each item', async ({ page }) => {
-            const columns = page.locator('ion-picker-column-internal .picker-item:not(.picker-item-empty)');
+            const columns = page.locator('ion-picker-column ion-picker-column-option');
             await expect(columns).toHaveCount(24);
         });
         test('should render 6 empty picker items', async ({ page }) => {
-            const columns = page.locator('ion-picker-column-internal .picker-item-empty');
+            const columns = page.locator('ion-picker-column .picker-item-empty');
             await expect(columns).toHaveCount(6);
         });
         test('should not have an active item when value is not set', async ({ page }) => {
-            const activeColumn = page.locator('ion-picker-column-internal .picker-item-active');
+            const activeColumn = page.locator('ion-picker-column ion-picker-column-option.option-active');
             await expect(activeColumn).toHaveCount(0);
         });
         test('should have an active item when value is set', async ({ page }) => {
@@ -28,7 +28,7 @@ configs({ modes: ['ios'], directions: ['ltr'] }).forEach(({ title, config }) =>
                 el.value = '12';
             });
             await page.waitForChanges();
-            const activeColumn = page.locator('ion-picker-column-internal .picker-item-active');
+            const activeColumn = page.locator('ion-picker-column ion-picker-column-option.option-active');
             expect(activeColumn).not.toBeNull();
         });
         // TODO FW-3616
@@ -38,8 +38,8 @@ configs({ modes: ['ios'], directions: ['ltr'] }).forEach(({ title, config }) =>
                 el.scrollTop = 801;
             });
             await page.waitForChanges();
-            const activeColumn = page.locator('ion-picker-column-internal .picker-item-active');
-            expect(await (activeColumn === null || activeColumn === void 0 ? void 0 : activeColumn.innerText())).toEqual('23');
+            const activeColumn = page.locator('ion-picker-column ion-picker-column-option.option-active');
+            await expect(activeColumn).toHaveJSProperty('value', 23);
         });
         test('should not emit ionChange when the value is modified externally', async ({ page, skip }) => {
             skip.browser('firefox', 'https://bugzilla.mozilla.org/show_bug.cgi?id=1766890');

package/dist/collection/components/picker-column/test/disabled/picker-column.e2e.js

@@ -0,0 +1,91 @@
+/*!
+ * (C) Ionic http://ionicframework.com - MIT License
+ */
+import { expect } from "@playwright/test";
+import { configs, test } from "../../../../utils/test/playwright/index";
+/**
+ * This behavior does not vary across modes/directions.
+ */
+configs({ modes: ['ios'], directions: ['ltr'] }).forEach(({ title, config }) => {
+    test.describe(title('picker-column: disabled items'), () => {
+        test('disabled picker item should not be considered active', async ({ page }) => {
+            await page.setContent(`
+        <ion-picker>
+          <ion-picker-column value="b">
+            <ion-picker-column-option value="a">A</ion-picker-column-option>
+            <ion-picker-column-option value="b" disabled="true">B</ion-picker-column-option>
+            <ion-picker-column-option value="c">C</ion-picker-column-option>
+          </ion-picker-column>
+        </ion-picker>
+      `, config);
+            const disabledItem = page.locator('ion-picker-column-option').nth(1);
+            await expect(disabledItem).not.toHaveClass(/option-active/);
+        });
+        test('setting the value to a disabled item should not cause that item to be active', async ({ page }) => {
+            await page.setContent(`
+        <ion-picker>
+          <ion-picker-column>
+            <ion-picker-column-option value="a">A</ion-picker-column-option>
+            <ion-picker-column-option value="b" disabled="true">B</ion-picker-column-option>
+            <ion-picker-column-option value="c">C</ion-picker-column-option>
+          </ion-picker-column>
+        </ion-picker>
+      `, config);
+            const pickerColumn = page.locator('ion-picker-column');
+            await pickerColumn.evaluate((el) => (el.value = 'b'));
+            await page.waitForChanges();
+            const disabledItem = page.locator('ion-picker-column ion-picker-column-option').nth(1);
+            await expect(disabledItem).not.toHaveClass(/option-active/);
+        });
+        test('defaulting the value to a disabled item should not cause that item to be active', async ({ page }) => {
+            await page.setContent(`
+        <ion-picker>
+          <ion-picker-column value="b">
+            <ion-picker-column-option value="a">A</ion-picker-column-option>
+            <ion-picker-column-option value="b" disabled="true">B</ion-picker-column-option>
+            <ion-picker-column-option value="c">C</ion-picker-column-option>
+          </ion-picker-column>
+        </ion-picker>
+      `, config);
+            const disabledItem = page.locator('ion-picker-column ion-picker-column-option').nth(1);
+            await expect(disabledItem).not.toHaveClass(/option-active/);
+        });
+    });
+});
+/**
+ * This behavior does not vary across directions.
+ */
+configs({ directions: ['ltr'] }).forEach(({ title, screenshot, config }) => {
+    test.describe(title('picker-column: disabled column rendering'), () => {
+        test.beforeEach(async ({ page }) => {
+            await page.goto('/src/components/picker-column/test/disabled', config);
+        });
+        test('disabled column should not have visual regressions', async ({ page }) => {
+            const disabledColumn = page.locator('#column-disabled');
+            await page.waitForChanges();
+            await expect(disabledColumn).toHaveScreenshot(screenshot('picker-disabled-column'));
+        });
+    });
+});
+/**
+ * This behavior does not vary across modes/directions.
+ */
+configs({ modes: ['ios'], directions: ['ltr'] }).forEach(({ title, config }) => {
+    test.describe(title('picker-column: disabled column'), () => {
+        test.beforeEach(async ({ page }) => {
+            await page.goto('/src/components/picker-column/test/disabled', config);
+        });
+        test('clicking option in disabled column should not change value', async ({ page }) => {
+            const column = page.locator('#column-disabled');
+            const option = column.locator('ion-picker-column-option').nth(1);
+            await expect(column).toHaveJSProperty('value', 11);
+            /**
+             * Options do not receive pointer events
+             * so we need to forcibly click the element.
+             */
+            await option.click({ force: true });
+            await page.waitForChanges();
+            await expect(column).toHaveJSProperty('value', 11);
+        });
+    });
+});

package/dist/collection/components/picker-column/test/slots/picker-column.e2e.js

@@ -0,0 +1,39 @@
+/*!
+ * (C) Ionic http://ionicframework.com - MIT License
+ */
+import { expect } from "@playwright/test";
+import { configs, test } from "../../../../utils/test/playwright/index";
+configs({ directions: ['ltr'] }).forEach(({ title, screenshot, config }) => {
+    test.describe(title('picker-column: slots'), () => {
+        test('should not have visual regressions with prefix and suffix', async ({ page }) => {
+            await page.setContent(`
+        <ion-picker>
+          <ion-picker-column value="b">
+            <ion-picker-column-option value="a">A</ion-picker-column-option>
+            <ion-picker-column-option value="b">B</ion-picker-column-option>
+            <ion-picker-column-option value="c">C</ion-picker-column-option>
+            <div slot="prefix">Prefix</div>
+            <div slot="suffix">Suffix</div>
+          </ion-picker-column>
+        </ion-picker>
+      `, config);
+            const picker = page.locator('ion-picker');
+            await expect(picker).toHaveScreenshot(screenshot(`picker-prefix-suffix`));
+        });
+        test('should not have visual regressions with a long prefix and suffix', async ({ page }) => {
+            await page.setContent(`
+        <ion-picker>
+          <ion-picker-column value="b">
+            <ion-picker-column-option value="a">A</ion-picker-column-option>
+            <ion-picker-column-option value="b">B</ion-picker-column-option>
+            <ion-picker-column-option value="c">C</ion-picker-column-option>
+            <div slot="prefix">Long prefix long prefix long prefix</div>
+            <div slot="suffix">Long suffix long suffix long suffix</div>
+          </ion-picker-column>
+        </ion-picker>
+      `, config);
+            const picker = page.locator('ion-picker');
+            await expect(picker).toHaveScreenshot(screenshot(`picker-long-prefix-suffix`));
+        });
+    });
+});

package/dist/collection/components/picker-column-option/picker-column-option.ios.css

@@ -0,0 +1,87 @@
+/**
+ * Convert a font size to a dynamic font size.
+ * Fonts that participate in Dynamic Type should use
+ * dynamic font sizes.
+ * @param size - The initial font size including the unit (i.e. px or pt)
+ * @param unit (optional) - The unit to convert to. Use this if you want to
+ * convert to a unit other than $baselineUnit.
+ */
+/**
+ * Convert a font size to a dynamic font size but impose
+ * a maximum font size.
+ * @param size - The initial font size including the unit (i.e. px or pt)
+ * @param maxScale - The maximum scale of the font (i.e. 2.5 for a maximum 250% scale).
+ * @param unit (optional) - The unit to convert the initial font size to. Use this if you want to
+ * convert to a unit other than $baselineUnit.
+ */
+/**
+ * Convert a font size to a dynamic font size but impose
+ * a minimum font size.
+ * @param size - The initial font size including the unit (i.e. px or pt)
+ * @param minScale - The minimum scale of the font (i.e. 0.8 for a minimum 80% scale).
+ * @param unit (optional) - The unit to convert the initial font size to. Use this if you want to
+ * convert to a unit other than $baselineUnit.
+ */
+/**
+ * Convert a font size to a dynamic font size but impose
+ * maximum and minimum font sizes.
+ * @param size - The initial font size including the unit (i.e. px or pt)
+ * @param minScale - The minimum scale of the font (i.e. 0.8 for a minimum 80% scale).
+ * @param maxScale - The maximum scale of the font (i.e. 2.5 for a maximum 250% scale).
+ * @param unit (optional) - The unit to convert the initial font size to. Use this if you want to
+ * convert to a unit other than $baselineUnit.
+ */
+/**
+ * A heuristic that applies CSS to tablet
+ * viewports.
+ *
+ * Usage:
+ * @include tablet-viewport() {
+ *   :host {
+ *     background-color: green;
+ *   }
+ * }
+ */
+/**
+ * A heuristic that applies CSS to mobile
+ * viewports (i.e. phones, not tablets).
+ *
+ * Usage:
+ * @include mobile-viewport() {
+ *   :host {
+ *     background-color: blue;
+ *   }
+ * }
+ */
+button {
+  padding-left: 0;
+  padding-right: 0;
+  padding-top: 0;
+  padding-bottom: 0;
+  margin-left: 0;
+  margin-right: 0;
+  margin-top: 0;
+  margin-bottom: 0;
+  width: 100%;
+  height: 34px;
+  border: 0px;
+  outline: none;
+  background: transparent;
+  color: inherit;
+  font-family: var(--ion-font-family, inherit);
+  font-size: inherit;
+  line-height: 34px;
+  text-align: inherit;
+  text-overflow: ellipsis;
+  white-space: nowrap;
+  cursor: pointer;
+  overflow: hidden;
+}
+
+:host(.option-disabled) {
+  opacity: 0.4;
+}
+
+:host(.option-disabled) button {
+  cursor: default;
+}

package/dist/collection/components/picker-column-option/picker-column-option.js

@@ -0,0 +1,173 @@
+/*!
+ * (C) Ionic http://ionicframework.com - MIT License
+ */
+import { Host, h } from "@stencil/core";
+import { inheritAttributes } from "../../utils/helpers";
+import { createColorClasses } from "../../utils/theme";
+import { getIonMode } from "../../global/ionic-global";
+export class PickerColumnOption {
+    constructor() {
+        /**
+         * We keep track of the parent picker column
+         * so we can update the value of it when
+         * clicking an enable option.
+         */
+        this.pickerColumn = null;
+        this.ariaLabel = null;
+        this.disabled = false;
+        this.value = undefined;
+        this.color = 'primary';
+    }
+    /**
+     * The aria-label of the option has changed after the
+     * first render and needs to be updated within the component.
+     *
+     * @param ariaLbl The new aria-label value.
+     */
+    onAriaLabelChange(ariaLbl) {
+        this.ariaLabel = ariaLbl;
+    }
+    componentWillLoad() {
+        const inheritedAttributes = inheritAttributes(this.el, ['aria-label']);
+        /**
+         * The initial value of `aria-label` needs to be set for
+         * the first render.
+    
+         */
+        this.ariaLabel = inheritedAttributes['aria-label'] || null;
+    }
+    connectedCallback() {
+        this.pickerColumn = this.el.closest('ion-picker-column');
+    }
+    disconnectedCallback() {
+        this.pickerColumn = null;
+    }
+    /**
+     * The column options can load at any time
+     * so the options needs to tell the
+     * parent picker column when it is loaded
+     * so the picker column can ensure it is
+     * centered in the view.
+     *
+     * We intentionally run this for every
+     * option. If we only ran this from
+     * the selected option then if the newly
+     * loaded options were not selected then
+     * scrollActiveItemIntoView would not be called.
+     */
+    componentDidLoad() {
+        const { pickerColumn } = this;
+        if (pickerColumn !== null) {
+            pickerColumn.scrollActiveItemIntoView();
+        }
+    }
+    /**
+     * When an option is clicked, update the
+     * parent picker column value. This
+     * component will handle centering the option
+     * in the column view.
+     */
+    onClick() {
+        const { pickerColumn } = this;
+        if (pickerColumn !== null) {
+            pickerColumn.setValue(this.value);
+        }
+    }
+    render() {
+        const { color, disabled, ariaLabel } = this;
+        const mode = getIonMode(this);
+        return (h(Host, { class: createColorClasses(color, {
+                [mode]: true,
+                ['option-disabled']: disabled,
+            }) }, h("button", { tabindex: "-1", "aria-label": ariaLabel, disabled: disabled, onClick: () => this.onClick() }, h("slot", null))));
+    }
+    static get is() { return "ion-picker-column-option"; }
+    static get encapsulation() { return "shadow"; }
+    static get originalStyleUrls() {
+        return {
+            "ios": ["picker-column-option.ios.scss"],
+            "md": ["picker-column-option.md.scss"]
+        };
+    }
+    static get styleUrls() {
+        return {
+            "ios": ["picker-column-option.ios.css"],
+            "md": ["picker-column-option.md.css"]
+        };
+    }
+    static get properties() {
+        return {
+            "disabled": {
+                "type": "boolean",
+                "mutable": false,
+                "complexType": {
+                    "original": "boolean",
+                    "resolved": "boolean",
+                    "references": {}
+                },
+                "required": false,
+                "optional": false,
+                "docs": {
+                    "tags": [],
+                    "text": "If `true`, the user cannot interact with the picker column option."
+                },
+                "attribute": "disabled",
+                "reflect": false,
+                "defaultValue": "false"
+            },
+            "value": {
+                "type": "any",
+                "mutable": false,
+                "complexType": {
+                    "original": "any | null",
+                    "resolved": "any",
+                    "references": {}
+                },
+                "required": false,
+                "optional": true,
+                "docs": {
+                    "tags": [],
+                    "text": "The text value of the option."
+                },
+                "attribute": "value",
+                "reflect": false
+            },
+            "color": {
+                "type": "string",
+                "mutable": false,
+                "complexType": {
+                    "original": "Color",
+                    "resolved": "\"danger\" | \"dark\" | \"light\" | \"medium\" | \"primary\" | \"secondary\" | \"success\" | \"tertiary\" | \"warning\" | string & Record<never, never> | undefined",
+                    "references": {
+                        "Color": {
+                            "location": "import",
+                            "path": "../../interface",
+                            "id": "src/interface.d.ts::Color"
+                        }
+                    }
+                },
+                "required": false,
+                "optional": true,
+                "docs": {
+                    "tags": [],
+                    "text": "The color to use from your application's color palette.\nDefault options are: `\"primary\"`, `\"secondary\"`, `\"tertiary\"`, `\"success\"`, `\"warning\"`, `\"danger\"`, `\"light\"`, `\"medium\"`, and `\"dark\"`.\nFor more information on colors, see [theming](/docs/theming/basics)."
+                },
+                "attribute": "color",
+                "reflect": true,
+                "defaultValue": "'primary'"
+            }
+        };
+    }
+    static get states() {
+        return {
+            "ariaLabel": {}
+        };
+    }
+    static get elementRef() { return "el"; }
+    static get watchers() {
+        return [{
+                "propName": "aria-label",
+                "methodName": "onAriaLabelChange"
+            }];
+    }
+}

package/dist/collection/components/{picker-column-internal/picker-column-internal.ios.css → picker-column-option/picker-column-option.md.css}

@@ -53,35 +53,7 @@
  *   }
  * }
  */
-:host {
-  -webkit-padding-start: 16px;
-  padding-inline-start: 16px;
-  -webkit-padding-end: 16px;
-  padding-inline-end: 16px;
-  padding-top: 0px;
-  padding-bottom: 0px;
-  height: 200px;
-  outline: none;
-  font-size: 22px;
-  scroll-snap-type: y mandatory;
-  /**
-   * Need to explicitly set overflow-x: hidden
-   * for older implementations of scroll snapping.
-   */
-  overflow-x: hidden;
-  overflow-y: scroll;
-  scrollbar-width: none;
-  text-align: center;
-}
-
-/**
- * Hide scrollbars on Chrome and Safari
- */
-:host::-webkit-scrollbar {
-  display: none;
-}
-
-:host .picker-item {
+button {
   padding-left: 0;
   padding-right: 0;
   padding-top: 0;
@@ -90,7 +62,6 @@
   margin-right: 0;
   margin-top: 0;
   margin-bottom: 0;
-  display: block;
   width: 100%;
   height: 34px;
   border: 0px;
@@ -105,34 +76,16 @@
   white-space: nowrap;
   cursor: pointer;
   overflow: hidden;
-  scroll-snap-align: center;
 }
 
-:host .picker-item-empty,
-:host .picker-item[disabled] {
-  cursor: default;
-}
-
-:host .picker-item-empty,
-:host(:not([disabled])) .picker-item[disabled] {
-  scroll-snap-align: none;
-}
-
-:host([disabled]) {
-  overflow-y: hidden;
-}
-
-:host .picker-item[disabled] {
+:host(.option-disabled) {
   opacity: 0.4;
 }
 
-:host(.picker-column-active) .picker-item.picker-item-active {
-  color: var(--ion-color-base);
+:host(.option-disabled) button {
+  cursor: default;
 }
 
-@media (any-hover: hover) {
-  :host(:focus) {
-    outline: none;
-    background: rgba(var(--ion-color-base-rgb), 0.2);
-  }
+:host(.option-active) {
+  color: var(--ion-color-base);
 }

package/dist/collection/components/picker-column-option/test/a11y/picker-column-option.e2e.js

@@ -0,0 +1,18 @@
+/*!
+ * (C) Ionic http://ionicframework.com - MIT License
+ */
+import AxeBuilder from "@axe-core/playwright";
+import { expect } from "@playwright/test";
+import { configs, test } from "../../../../utils/test/playwright/index";
+/**
+ * This behavior does not vary across directions
+ */
+configs({ directions: ['ltr'] }).forEach(({ config, title }) => {
+    test.describe(title('picker column option: a11y'), () => {
+        test('should not have accessibility violations', async ({ page }) => {
+            await page.goto(`/src/components/picker-column-option/test/a11y`, config);
+            const results = await new AxeBuilder({ page }).analyze();
+            expect(results.violations).toEqual([]);
+        });
+    });
+});

package/dist/collection/components/picker-column-option/test/basic/picker-column-option.e2e.js

@@ -0,0 +1,37 @@
+/*!
+ * (C) Ionic http://ionicframework.com - MIT License
+ */
+import { expect } from "@playwright/test";
+import { configs, test } from "../../../../utils/test/playwright/index";
+configs({ directions: ['ltr'] }).forEach(({ title, screenshot, config }) => {
+    test.describe(title('picker-column-option: rendering'), () => {
+        test('picker option should not have visual regressions', async ({ page }) => {
+            await page.setContent(`
+        <ion-picker-column-option value="option">My Option</ion-picker-column-option>
+      `, config);
+            const option = page.locator('ion-picker-column-option');
+            await expect(option).toHaveScreenshot(screenshot('picker-column-option'));
+        });
+        test('disabled picker option should not have visual regressions', async ({ page }) => {
+            await page.setContent(`
+        <ion-picker-column-option disabled="true" value="option">My Option</ion-picker-column-option>
+      `, config);
+            const option = page.locator('ion-picker-column-option');
+            await expect(option).toHaveScreenshot(screenshot('disabled-picker-column-option'));
+        });
+        test('active picker option should not have visual regressions', async ({ page }) => {
+            await page.setContent(`
+        <ion-picker-column-option class="option-active" value="option">My Option</ion-picker-column-option>
+      `, config);
+            const option = page.locator('ion-picker-column-option');
+            await expect(option).toHaveScreenshot(screenshot('active-picker-column-option'));
+        });
+        test('disabled active picker option should not have visual regressions', async ({ page }) => {
+            await page.setContent(`
+        <ion-picker-column-option class="option-active" disabled="true" value="option">My Option</ion-picker-column-option>
+      `, config);
+            const option = page.locator('ion-picker-column-option');
+            await expect(option).toHaveScreenshot(screenshot('disabled-active-picker-column-option'));
+        });
+    });
+});

package/dist/collection/components/picker-column-option/test/picker-column-option.spec.js

@@ -0,0 +1,29 @@
+/*!
+ * (C) Ionic http://ionicframework.com - MIT License
+ */
+import { newSpecPage } from "@stencil/core/testing";
+import { PickerColumnOption } from "../picker-column-option";
+describe('picker column option', () => {
+    it('button should be enabled by default', async () => {
+        const page = await newSpecPage({
+            components: [PickerColumnOption],
+            html: `
+        <ion-picker-column-option value="test">Test</ion-picker-column-otion>
+      `,
+        });
+        const option = page.body.querySelector('ion-picker-column-option');
+        const button = option.shadowRoot.querySelector('button');
+        await expect(button.hasAttribute('disabled')).toEqual(false);
+    });
+    it('button should be disabled if specified', async () => {
+        const page = await newSpecPage({
+            components: [PickerColumnOption],
+            html: `
+        <ion-picker-column-option disabled="true" value="test">Test</ion-picker-column-otion>
+      `,
+        });
+        const option = page.body.querySelector('ion-picker-column-option');
+        const button = option.shadowRoot.querySelector('button');
+        await expect(button.hasAttribute('disabled')).toEqual(true);
+    });
+});