@ionic/core 7.5.7-dev.11701201960.104b3a1f → 7.5.7-dev.11701203423.1b6a8829

package/dist/collection/components/picker/picker.md.css

@@ -54,61 +54,61 @@
  * }
  */
 :host {
-  display: flex;
-  position: relative;
-  align-items: center;
-  justify-content: center;
-  width: 100%;
-  height: 200px;
   /**
-   * Picker columns should display
-   * in the order in which developers
-   * added them and should ignore
-   * LTR vs RTL directions.
+   * @prop --background: Background of the picker
+   * @prop --background-rgb: Background of the picker in rgb format
+   *
+   * @prop --border-radius: Border radius of the picker
+   * @prop --border-color: Border color of the picker
+   * @prop --border-width: Border width of the picker
+   * @prop --border-style: Border style of the picker
+   *
+   * @prop --min-width: Minimum width of the picker
+   * @prop --width: Width of the picker
+   * @prop --max-width: Maximum width of the picker
+   *
+   * @prop --min-height: Minimum height of the picker
+   * @prop --height: Height of the picker
+   * @prop --max-height: Maximum height of the picker
+   *
+   * @prop --backdrop-opacity: Opacity of the backdrop
    */
-  direction: ltr;
-  /**
-   * This is required otherwise the
-   * highlight will appear behind
-   * the picker when used inline.
-   */
-  z-index: 0;
-}
-
-:host .picker-before,
-:host .picker-after {
+  --border-radius: 0;
+  --border-style: solid;
+  --min-width: auto;
+  --width: 100%;
+  --max-width: 500px;
+  --min-height: auto;
+  --max-height: auto;
+  -moz-osx-font-smoothing: grayscale;
+  -webkit-font-smoothing: antialiased;
+  top: 0;
+  display: block;
   position: absolute;
   width: 100%;
-  /**
-   * The transform and z-index
-   * are needed for WebKit otherwise
-   * the fade will appear underneath the picker.
-   */
-  transform: translateZ(0);
-  z-index: 1;
-  pointer-events: none;
-}
-
-:host .picker-before {
-  top: 0;
-  height: 83px;
+  height: 100%;
+  outline: none;
+  font-family: var(--ion-font-family, inherit);
+  contain: strict;
+  user-select: none;
+  z-index: 1001;
 }
 @supports (inset-inline-start: 0) {
-  :host .picker-before {
+  :host {
     inset-inline-start: 0;
   }
 }
 @supports not (inset-inline-start: 0) {
-  :host .picker-before {
+  :host {
     left: 0;
   }
-  :host-context([dir=rtl]) .picker-before {
+  :host-context([dir=rtl]) {
     left: unset;
     right: unset;
     right: 0;
   }
   @supports selector(:dir(rtl)) {
-    :host(:dir(rtl)) .picker-before {
+    :host(:dir(rtl)) {
       left: unset;
       right: unset;
       right: 0;
@@ -116,82 +116,69 @@
   }
 }
 
-:host .picker-after {
-  top: 116px;
-  height: 84px;
-}
-@supports (inset-inline-start: 0) {
-  :host .picker-after {
-    inset-inline-start: 0;
-  }
-}
-@supports not (inset-inline-start: 0) {
-  :host .picker-after {
-    left: 0;
-  }
-  :host-context([dir=rtl]) .picker-after {
-    left: unset;
-    right: unset;
-    right: 0;
-  }
-  @supports selector(:dir(rtl)) {
-    :host(:dir(rtl)) .picker-after {
-      left: unset;
-      right: unset;
-      right: 0;
-    }
-  }
+:host(.overlay-hidden) {
+  display: none;
 }
 
-:host .picker-highlight {
-  border-radius: 8px;
+.picker-wrapper {
+  border-radius: var(--border-radius);
   left: 0;
   right: 0;
-  top: 50%;
   bottom: 0;
   -webkit-margin-start: auto;
   margin-inline-start: auto;
   -webkit-margin-end: auto;
   margin-inline-end: auto;
-  margin-top: 0;
-  margin-bottom: 0;
+  margin-top: auto;
+  margin-bottom: auto;
+  transform: translate3d(0,  100%,  0);
+  display: flex;
   position: absolute;
-  width: calc(100% - 16px);
-  height: 34px;
-  transform: translateY(-50%);
-  background: var(--wheel-highlight-background);
-  z-index: -1;
+  flex-direction: column;
+  width: var(--width);
+  min-width: var(--min-width);
+  max-width: var(--max-width);
+  height: var(--height);
+  min-height: var(--min-height);
+  max-height: var(--max-height);
+  border-width: var(--border-width);
+  border-style: var(--border-style);
+  border-color: var(--border-color);
+  background: var(--background);
+  contain: strict;
+  overflow: hidden;
+  z-index: 10;
 }
 
-:host input {
-  position: absolute;
-  top: 0;
-  left: 0;
-  right: 0;
-  bottom: 0;
+.picker-toolbar {
   width: 100%;
-  height: 100%;
-  margin: 0;
-  padding: 0;
+  background: transparent;
+  contain: strict;
+  z-index: 1;
+}
+
+.picker-button {
   border: 0;
-  outline: 0;
-  clip: rect(0 0 0 0);
-  opacity: 0;
-  overflow: hidden;
-  -webkit-appearance: none;
-  -moz-appearance: none;
+  font-family: inherit;
 }
 
-:host ::slotted(ion-picker-column:first-of-type) {
-  text-align: start;
+.picker-button:active, .picker-button:focus {
+  outline: none;
 }
 
-:host ::slotted(ion-picker-column:last-of-type) {
-  text-align: end;
+.picker-columns {
+  display: flex;
+  position: relative;
+  justify-content: center;
+  margin-bottom: var(--ion-safe-area-bottom, 0);
+  contain: strict;
+  overflow: hidden;
 }
 
-:host ::slotted(ion-picker-column:only-child) {
-  text-align: center;
+.picker-above-highlight,
+.picker-below-highlight {
+  display: none;
+  pointer-events: none;
 }
 
 /**
@@ -249,10 +236,119 @@
  *   }
  * }
  */
-:host .picker-before {
-  background: linear-gradient(to bottom, rgba(var(--wheel-fade-background-rgb, var(--background-rgb, var(--ion-background-color-rgb, 255, 255, 255))), 1) 20%, rgba(var(--wheel-fade-background-rgb, var(--background-rgb, var(--ion-background-color-rgb, 255, 255, 255))), 0) 90%);
+:host {
+  --background: var(--ion-background-color, #fff);
+  --border-width: 0.55px 0 0;
+  --border-color: var(--ion-item-border-color, var(--ion-border-color, var(--ion-color-step-150, rgba(0, 0, 0, 0.13))));
+  --height: 260px;
+  --backdrop-opacity: var(--ion-backdrop-opacity, 0.26);
+  color: var(--ion-item-color, var(--ion-text-color, #000));
+}
+
+.picker-toolbar {
+  display: flex;
+  justify-content: flex-end;
+  height: 44px;
 }
 
-:host .picker-after {
-  background: linear-gradient(to top, rgba(var(--wheel-fade-background-rgb, var(--background-rgb, var(--ion-background-color-rgb, 255, 255, 255))), 1) 30%, rgba(var(--wheel-fade-background-rgb, var(--background-rgb, var(--ion-background-color-rgb, 255, 255, 255))), 0) 90%);
+.picker-button,
+.picker-button.ion-activated {
+  margin-left: 0;
+  margin-right: 0;
+  margin-top: 0;
+  margin-bottom: 0;
+  -webkit-padding-start: 1.1em;
+  padding-inline-start: 1.1em;
+  -webkit-padding-end: 1.1em;
+  padding-inline-end: 1.1em;
+  padding-top: 0;
+  padding-bottom: 0;
+  height: 44px;
+  background: transparent;
+  color: var(--ion-color-primary, #3880ff);
+  font-size: 14px;
+  font-weight: 500;
+  text-transform: uppercase;
+  box-shadow: none;
+}
+
+.picker-columns {
+  height: 216px;
+  perspective: 1800px;
+}
+
+.picker-above-highlight {
+  top: 0;
+  transform: translate3d(0,  0,  90px);
+  position: absolute;
+  width: 100%;
+  height: 81px;
+  border-bottom: 1px solid var(--ion-item-border-color, var(--ion-border-color, var(--ion-color-step-150, rgba(0, 0, 0, 0.13))));
+  background: linear-gradient(to bottom, var(--ion-background-color, #fff) 20%, rgba(var(--ion-background-color-rgb, 255, 255, 255), 0.8) 100%);
+  z-index: 10;
+}
+@supports (inset-inline-start: 0) {
+  .picker-above-highlight {
+    inset-inline-start: 0;
+  }
+}
+@supports not (inset-inline-start: 0) {
+  .picker-above-highlight {
+    left: 0;
+  }
+  :host-context([dir=rtl]) .picker-above-highlight {
+    left: unset;
+    right: unset;
+    right: 0;
+  }
+  [dir=rtl] .picker-above-highlight {
+    left: unset;
+    right: unset;
+    right: 0;
+  }
+  @supports selector(:dir(rtl)) {
+    .picker-above-highlight:dir(rtl) {
+      left: unset;
+      right: unset;
+      right: 0;
+    }
+  }
+}
+
+.picker-below-highlight {
+  top: 115px;
+  transform: translate3d(0,  0,  90px);
+  position: absolute;
+  width: 100%;
+  height: 119px;
+  border-top: 1px solid var(--ion-item-border-color, var(--ion-border-color, var(--ion-color-step-150, rgba(0, 0, 0, 0.13))));
+  background: linear-gradient(to top, var(--ion-background-color, #fff) 30%, rgba(var(--ion-background-color-rgb, 255, 255, 255), 0.8) 100%);
+  z-index: 11;
+}
+@supports (inset-inline-start: 0) {
+  .picker-below-highlight {
+    inset-inline-start: 0;
+  }
+}
+@supports not (inset-inline-start: 0) {
+  .picker-below-highlight {
+    left: 0;
+  }
+  :host-context([dir=rtl]) .picker-below-highlight {
+    left: unset;
+    right: unset;
+    right: 0;
+  }
+  [dir=rtl] .picker-below-highlight {
+    left: unset;
+    right: unset;
+    right: 0;
+  }
+  @supports selector(:dir(rtl)) {
+    .picker-below-highlight:dir(rtl) {
+      left: unset;
+      right: unset;
+      right: 0;
+    }
+  }
 }

package/dist/collection/components/picker/test/basic/picker.e2e.js

@@ -3,116 +3,23 @@
  */
 import { expect } from "@playwright/test";
 import { configs, test } from "../../../../utils/test/playwright/index";
-/**
- * This behavior does not vary across directions.
- */
-configs({ directions: ['ltr'] }).forEach(({ title, screenshot, config }) => {
-  test.describe(title('picker: rendering'), () => {
-    test('inline pickers should not have visual regression', async ({ page }) => {
-      await page.goto(`/src/components/picker/test/basic`, config);
-      const fullStack = page.locator('#inline button[data-value="full-stack"]');
-      const onion = page.locator('#inline button[data-value="onion"]');
-      await expect(fullStack).toHaveClass(/picker-item-active/);
-      await expect(onion).toHaveClass(/picker-item-active/);
-      await page.waitForChanges();
-      await expect(page.locator('#inline')).toHaveScreenshot(screenshot(`picker-inline-diff`));
-    });
-  });
-  test.describe(title('picker: overlay rendering'), () => {
-    test('popover: should not have visual regression', async ({ page }) => {
-      await page.goto(`/src/components/picker/test/basic`, config);
-      const button = page.locator('#popover');
-      const didPresent = await page.spyOnEvent('ionPopoverDidPresent');
-      const picker = page.locator('ion-popover ion-picker');
-      await button.click();
-      await didPresent.next();
-      await expect(picker).toBeVisible();
-      const popoverContent = page.locator('ion-popover .ion-delegate-host');
-      await expect(popoverContent).toHaveScreenshot(screenshot(`picker-popover-diff`), {
-        /**
-         * Animations must be enabled to capture the screenshot.
-         * By default, animations are disabled with toHaveScreenshot,
-         * and when capturing the screenshot will call animation.finish().
-         * This will cause the popover to close and the screenshot capture
-         * to be invalid.
-         */
-        animations: 'allow',
-      });
-    });
-    test('modal: should not have visual regression', async ({ page }) => {
+configs().forEach(({ title, screenshot, config }) => {
+  test.describe(title('picker: basic'), () => {
+    test('should not have visual regressions', async ({ page }) => {
       await page.goto('/src/components/picker/test/basic', config);
-      const button = page.locator('#modal');
-      const didPresent = await page.spyOnEvent('ionModalDidPresent');
-      const picker = page.locator('ion-modal ion-picker');
-      await button.click();
+      const didPresent = await page.spyOnEvent('ionPickerDidPresent');
+      const didDismiss = await page.spyOnEvent('ionPickerDidDismiss');
+      await page.click('#basic');
       await didPresent.next();
-      await expect(picker).toBeVisible();
-      const modalContent = page.locator('ion-modal .ion-delegate-host');
-      await expect(modalContent).toHaveScreenshot(screenshot(`picker-modal-diff`), {
-        /**
-         * Animations must be enabled to capture the screenshot.
-         * By default, animations are disabled with toHaveScreenshot,
-         * and when capturing the screenshot will call animation.finish().
-         * This will cause the modal to close and the screenshot capture
-         * to be invalid.
-         */
-        animations: 'allow',
-      });
-    });
-  });
-});
-/**
- * This behavior does not vary across modes/directions.
- */
-configs({ modes: ['ios'], directions: ['ltr'] }).forEach(({ title, config }) => {
-  test.describe(title('picker: focus'), () => {
-    test.beforeEach(async ({ page }) => {
-      await page.setContent(`
-        <ion-picker>
-          <ion-picker-column value="full-stack" id="first"></ion-picker-column>
-          <ion-picker-column value="onion" id="second"></ion-picker-column>
-        </ion-picker>
-
-        <script>
-          const columns = document.querySelectorAll('ion-picker-column');
-          columns[0].items = [
-            { text: 'Minified', value: 'minified' },
-            { text: 'Responsive', value: 'responsive' },
-            { text: 'Full Stack', value: 'full-stack' },
-            { text: 'Mobile First', value: 'mobile-first' },
-            { text: 'Serverless', value: 'serverless' },
-          ]
-
-          columns[1].items = [
-            { text: 'Tomato', value: 'tomato' },
-            { text: 'Avocado', value: 'avocado' },
-            { text: 'Onion', value: 'onion' },
-            { text: 'Potato', value: 'potato' },
-            { text: 'Artichoke', value: 'artichoke' },
-          ];
-        </script>
-      `, config);
-    });
-    test('tabbing should correctly move focus between columns', async ({ page }) => {
-      const firstColumn = page.locator('ion-picker-column#first');
-      const secondColumn = page.locator('ion-picker-column#second');
-      // Focus first column
-      await page.keyboard.press('Tab');
-      await expect(firstColumn).toBeFocused();
       await page.waitForChanges();
-      // Focus second column
-      await page.keyboard.press('Tab');
-      await expect(secondColumn).toBeFocused();
-    });
-    test('tabbing should correctly move focus back', async ({ page }) => {
-      const firstColumn = page.locator('ion-picker-column#first');
-      const secondColumn = page.locator('ion-picker-column#second');
-      await secondColumn.focus();
-      await expect(secondColumn).toBeFocused();
+      await expect(page).toHaveScreenshot(screenshot(`picker-basic`));
+      await page.click('.picker-opt:nth-child(2)');
+      await page.click('ion-picker .save-btn');
+      await didDismiss.next();
+      await page.click('#basic');
+      await didPresent.next();
       await page.waitForChanges();
-      // Focus first column
-      await page.keyboard.press('Shift+Tab');
-      await expect(firstColumn).toBeFocused();
+      await expect(page).toHaveScreenshot(screenshot(`picker-value-selected`));
     });
   });
 });

package/dist/collection/components/{picker-legacy → picker}/test/is-open/picker.e2e.js

@@ -9,10 +9,10 @@ import { configs, test } from "../../../../utils/test/playwright/index";
 configs({ modes: ['ios'], directions: ['ltr'] }).forEach(({ title, config }) => {
   test.describe(title('picker: isOpen'), () => {
     test('should open and close the picker', async ({ page }) => {
-      await page.goto('/src/components/picker-legacy/test/is-open', config);
+      await page.goto('/src/components/picker/test/is-open', config);
       const ionPickerDidPresent = await page.spyOnEvent('ionPickerDidPresent');
       const ionPickerDidDismiss = await page.spyOnEvent('ionPickerDidDismiss');
-      const picker = page.locator('ion-picker-legacy');
+      const picker = page.locator('ion-picker');
       await page.click('#default');
       await ionPickerDidPresent.next();
       await expect(picker).toBeVisible();
@@ -21,8 +21,8 @@ configs({ modes: ['ios'], directions: ['ltr'] }).forEach(({ title, config }) =>
       await expect(picker).toBeHidden();
     });
     test('should open if isOpen is true on load', async ({ page }) => {
-      await page.setContent('<ion-picker-legacy is-open="true"></ion-picker-legacy>', config);
-      await expect(page.locator('ion-picker-legacy')).toBeVisible();
+      await page.setContent('<ion-picker is-open="true"></ion-picker>', config);
+      await expect(page.locator('ion-picker')).toBeVisible();
     });
   });
 });

package/dist/collection/components/{picker-legacy → picker}/test/picker-id.spec.js

@@ -6,27 +6,27 @@ import { Picker } from "../picker";
 it('picker should be assigned an incrementing id', async () => {
   const page = await newSpecPage({
     components: [Picker],
-    html: `<ion-picker-legacy is-open="true"></ion-picker-legacy>`,
+    html: `<ion-picker is-open="true"></ion-picker>`,
   });
   let picker;
-  picker = page.body.querySelector('ion-picker-legacy');
+  picker = page.body.querySelector('ion-picker');
   expect(picker).not.toBe(null);
   expect(picker.getAttribute('id')).toBe('ion-overlay-1');
   // Remove the picker from the DOM
   picker.remove();
   await page.waitForChanges();
   // Create a new picker to verify the id is incremented
-  picker = document.createElement('ion-picker-legacy');
+  picker = document.createElement('ion-picker');
   picker.isOpen = true;
   page.body.appendChild(picker);
   await page.waitForChanges();
-  picker = page.body.querySelector('ion-picker-legacy');
+  picker = page.body.querySelector('ion-picker');
   expect(picker.getAttribute('id')).toBe('ion-overlay-2');
   // Presenting the same picker again should reuse the existing id
   picker.isOpen = false;
   await page.waitForChanges();
   picker.isOpen = true;
   await page.waitForChanges();
-  picker = page.body.querySelector('ion-picker-legacy');
+  picker = page.body.querySelector('ion-picker');
   expect(picker.getAttribute('id')).toBe('ion-overlay-2');
 });

package/dist/collection/components/{picker-legacy → picker}/test/trigger/picker.e2e.js

@@ -9,7 +9,7 @@ import { configs, test } from "../../../../utils/test/playwright/index";
 configs({ modes: ['ios'], directions: ['ltr'] }).forEach(({ title, config }) => {
   test.describe(title('picker: trigger'), () => {
     test.beforeEach(async ({ page }) => {
-      await page.goto('/src/components/picker-legacy/test/trigger', config);
+      await page.goto('/src/components/picker/test/trigger', config);
     });
     test('should open the picker', async ({ page }) => {
       const ionPickerDidPresent = await page.spyOnEvent('ionPickerDidPresent');