@ionic/core 7.5.8 → 7.6.1-dev.11701921257.1d512d3c

package/dist/collection/components/label/label.ios.css

@@ -63,17 +63,24 @@
   font-family: var(--ion-font-family, inherit);
   font-size: inherit;
   text-overflow: ellipsis;
+  box-sizing: border-box;
+}
+
+:host-context(.item-legacy) {
   white-space: nowrap;
   overflow: hidden;
-  box-sizing: border-box;
+}
+
+:host-context(.item:not(.item-input):not(.item-legacy)) {
+  flex-grow: 1;
 }
 
 :host(.ion-color) {
   color: var(--ion-color-base);
 }
 
-:host(.ion-text-wrap) {
-  white-space: normal;
+:host(.ion-text-nowrap) {
+  overflow: hidden;
 }
 
 :host-context(.item-interactive-disabled:not(.item-multiple-inputs)) {
@@ -92,6 +99,10 @@
   align-self: baseline;
 }
 
+:host-context(.item-skeleton-text) {
+  overflow: hidden;
+}
+
 :host(.label-fixed) {
   flex: 0 0 100px;
   width: 100px;

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

@@ -63,17 +63,24 @@
   font-family: var(--ion-font-family, inherit);
   font-size: inherit;
   text-overflow: ellipsis;
+  box-sizing: border-box;
+}
+
+:host-context(.item-legacy) {
   white-space: nowrap;
   overflow: hidden;
-  box-sizing: border-box;
+}
+
+:host-context(.item:not(.item-input):not(.item-legacy)) {
+  flex-grow: 1;
 }
 
 :host(.ion-color) {
   color: var(--ion-color-base);
 }
 
-:host(.ion-text-wrap) {
-  white-space: normal;
+:host(.ion-text-nowrap) {
+  overflow: hidden;
 }
 
 :host-context(.item-interactive-disabled:not(.item-multiple-inputs)) {
@@ -92,6 +99,10 @@
   align-self: baseline;
 }
 
+:host-context(.item-skeleton-text) {
+  overflow: hidden;
+}
+
 :host(.label-fixed) {
   flex: 0 0 100px;
   width: 100px;

package/dist/collection/components/modal/gestures/swipe-to-close.js

@@ -5,6 +5,7 @@ import { getTimeGivenProgression } from "../../../utils/animation/cubic-bezier";
 import { isIonContent, findClosestIonContent, disableContentScrollY, resetContentScrollY } from "../../../utils/content/index";
 import { createGesture } from "../../../utils/gesture/index";
 import { clamp, getElementRoot } from "../../../utils/helpers";
+import { OVERLAY_GESTURE_PRIORITY } from "../../../utils/overlays";
 import { setCardStatusBarDark, setCardStatusBarDefault } from "../utils";
 import { calculateSpringStep, handleCanDismiss } from "./utils";
 // Defaults for the card swipe animation
@@ -250,7 +251,7 @@ export const createSwipeToCloseGesture = (el, animation, statusBarStyle, onDismi
     const gesture = createGesture({
         el,
         gestureName: 'modalSwipeToClose',
-        gesturePriority: 39,
+        gesturePriority: OVERLAY_GESTURE_PRIORITY,
         direction: 'y',
         threshold: 10,
         canStart,

package/dist/collection/components/note/test/a11y/note.e2e.js

@@ -48,5 +48,48 @@ configs({ directions: ['ltr'] }).forEach(({ title, screenshot, config }) => {
             const note = page.locator('ion-note');
             await expect(note).toHaveScreenshot(screenshot(`note-item-divider-scale`));
         });
+        test('should wrap label to the next line without truncating', async ({ page }) => {
+            await page.setContent(`
+        <style>
+          html {
+            font-size: 310%;
+          }
+        </style>
+
+        <ion-list>
+          <ion-item>
+            <ion-note slot="start">Really really long note</ion-note>
+            <ion-label>Label</ion-label>
+          </ion-item>
+
+          <ion-item>
+            <ion-label>Label</ion-label>
+            <ion-note slot="end">Really really long note</ion-note>
+          </ion-item>
+
+          <ion-item>
+            <ion-note slot="start">Note</ion-note>
+            <ion-label>Really really long label</ion-label>
+          </ion-item>
+
+          <ion-item>
+            <ion-label>Really really long label</ion-label>
+            <ion-note slot="end">Note</ion-note>
+          </ion-item>
+
+          <ion-item>
+            <ion-note slot="start">Really really long note</ion-note>
+            <ion-label>Really really long label</ion-label>
+          </ion-item>
+
+          <ion-item>
+            <ion-label>Really really long label</ion-label>
+            <ion-note slot="end">Really really long note</ion-note>
+          </ion-item>
+        </ion-list>
+      `, config);
+            const list = page.locator('ion-list');
+            await expect(list).toHaveScreenshot(screenshot(`note-wrapping-label-scale`));
+        });
     });
 });

package/dist/collection/components/radio/radio.ios.css

@@ -186,12 +186,6 @@ input {
 }
 
 .label-text-wrapper {
-  /**
-   * This ensures that double tapping this text
-   * clicks the <label> and focuses the radio
-   * when a screen reader is enabled.
-   */
-  pointer-events: none;
   text-overflow: ellipsis;
   white-space: nowrap;
   overflow: hidden;

package/dist/collection/components/radio/radio.js

@@ -2,7 +2,7 @@
  * (C) Ionic http://ionicframework.com - MIT License
  */
 import { Host, h } from "@stencil/core";
-import { createLegacyFormController } from "../../utils/forms/index";
+import { createLegacyFormController, isOptionSelected } from "../../utils/forms/index";
 import { addEventListener, getAriaLabel, removeEventListener } from "../../utils/helpers";
 import { printIonWarning } from "../../utils/logging/index";
 import { createColorClasses, hostContext } from "../../utils/theme";
@@ -13,6 +13,7 @@ import { getIonMode } from "../../global/ionic-global";
  * @slot - The label text to associate with the radio. Use the "labelPlacement" property to control where the label is placed relative to the radio.
  *
  * @part container - The container for the radio mark.
+ * @part label - The label text describing the radio.
  * @part mark - The checkmark or dot used to indicate the checked state.
  */
 export class Radio {
@@ -23,7 +24,8 @@ export class Radio {
         this.hasLoggedDeprecationWarning = false;
         this.updateState = () => {
             if (this.radioGroup) {
-                this.checked = this.radioGroup.value === this.value;
+                const { compareWith, value: radioGroupValue } = this.radioGroup;
+                this.checked = isOptionSelected(radioGroupValue, this.value, compareWith);
             }
         };
         this.onClick = () => {
@@ -120,6 +122,8 @@ export class Radio {
     emitStyle() {
         const style = {
             'interactive-disabled': this.disabled,
+            // TODO(FW-3125): remove this
+            legacy: !!this.legacy,
         };
         if (this.legacyFormController.hasLegacyControl()) {
             style['radio-checked'] = this.checked;
@@ -154,7 +158,7 @@ export class Radio {
             }), role: "radio", "aria-checked": checked ? 'true' : 'false', "aria-disabled": disabled ? 'true' : null, tabindex: buttonTabindex }, h("label", { class: "radio-wrapper" }, h("div", { class: {
                 'label-text-wrapper': true,
                 'label-text-wrapper-hidden': !hasLabel,
-            } }, h("slot", null)), h("div", { class: "native-wrapper" }, this.renderRadioControl()))));
+            }, part: "label" }, h("slot", null)), h("div", { class: "native-wrapper" }, this.renderRadioControl()))));
     }
     renderLegacyRadio() {
         if (!this.hasLoggedDeprecationWarning) {

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

@@ -186,12 +186,6 @@ input {
 }
 
 .label-text-wrapper {
-  /**
-   * This ensures that double tapping this text
-   * clicks the <label> and focuses the radio
-   * when a screen reader is enabled.
-   */
-  pointer-events: none;
   text-overflow: ellipsis;
   white-space: nowrap;
   overflow: hidden;

package/dist/collection/components/radio/test/radio.spec.js

@@ -25,6 +25,20 @@ describe('ion-radio', () => {
         await page.waitForChanges();
         expect(radio.classList.contains('radio-checked')).toBe(true);
     });
+    it('should render the radio with shadow parts', async () => {
+        const page = await newSpecPage({
+            components: [Radio, RadioGroup],
+            html: `
+        <ion-radio-group>
+          <ion-radio value="value"></ion-radio>
+        </ion-radio-group>
+      `,
+        });
+        const radio = page.body.querySelector('ion-radio');
+        expect(radio).toHaveShadowPart('container');
+        expect(radio).toHaveShadowPart('label');
+        expect(radio).toHaveShadowPart('mark');
+    });
 });
 describe('ion-radio: disabled', () => {
     it('clicking disabled radio should not set checked state', async () => {

package/dist/collection/components/radio-group/radio-group.js

@@ -47,6 +47,7 @@ export class RadioGroup {
             }
         };
         this.allowEmptySelection = false;
+        this.compareWith = undefined;
         this.name = this.inputId;
         this.value = undefined;
     }
@@ -167,6 +168,29 @@ export class RadioGroup {
                 "reflect": false,
                 "defaultValue": "false"
             },
+            "compareWith": {
+                "type": "string",
+                "mutable": false,
+                "complexType": {
+                    "original": "string | RadioGroupCompareFn | null",
+                    "resolved": "((currentValue: any, compareValue: any) => boolean) | null | string | undefined",
+                    "references": {
+                        "RadioGroupCompareFn": {
+                            "location": "import",
+                            "path": "./radio-group-interface",
+                            "id": "src/components/radio-group/radio-group-interface.ts::RadioGroupCompareFn"
+                        }
+                    }
+                },
+                "required": false,
+                "optional": true,
+                "docs": {
+                    "tags": [],
+                    "text": "This property allows developers to specify a custom function or property\nname for comparing objects when determining the selected option in the\nion-radio-group. When not specified, the default behavior will use strict\nequality (===) for comparison."
+                },
+                "attribute": "compare-with",
+                "reflect": false
+            },
             "name": {
                 "type": "string",
                 "mutable": false,

package/dist/collection/components/radio-group/test/radio-group.spec.js

@@ -0,0 +1,69 @@
+/*!
+ * (C) Ionic http://ionicframework.com - MIT License
+ */
+import { h } from "@stencil/core";
+import { newSpecPage } from "@stencil/core/testing";
+import { Radio } from "../../radio/radio";
+import { RadioGroup } from "../radio-group";
+describe('ion-radio-group', () => {
+    it('should correctly set value when using compareWith string', async () => {
+        const page = await newSpecPage({
+            components: [Radio, RadioGroup],
+            template: () => (h("ion-radio-group", { compareWith: "value", value: { label: 'Blue', value: 'blue' } }, h("ion-radio", { value: { label: 'Red', value: 'red' } }, "Red"), h("ion-radio", { value: { label: 'Blue', value: 'blue' } }, "Blue"), h("ion-radio", { value: { label: 'Green', value: 'green' } }, "Green"))),
+        });
+        const radioGroup = page.body.querySelector('ion-radio-group');
+        const radios = document.querySelectorAll('ion-radio');
+        await radios[2].click();
+        await page.waitForChanges();
+        expect(radios[2].getAttribute('aria-checked')).toBe('true');
+        expect(radioGroup.value).toEqual({
+            label: 'Green',
+            value: 'green',
+        });
+    });
+    it('should correctly set value when using compareWith function', async () => {
+        const page = await newSpecPage({
+            components: [Radio, RadioGroup],
+            template: () => (h("ion-radio-group", { value: { label: 'Blue', value: 'blue' } }, h("ion-radio", { value: { label: 'Red', value: 'red' } }, "Red"), h("ion-radio", { value: { label: 'Blue', value: 'blue' } }, "Blue"), h("ion-radio", { value: { label: 'Green', value: 'green' } }, "Green"))),
+        });
+        const radioGroup = page.body.querySelector('ion-radio-group');
+        const radios = document.querySelectorAll('ion-radio');
+        radioGroup.compareWith = (a, b) => a.value === b.value;
+        await radios[2].click();
+        await page.waitForChanges();
+        expect(radios[2].getAttribute('aria-checked')).toBe('true');
+        expect(radioGroup.value).toEqual({
+            label: 'Green',
+            value: 'green',
+        });
+    });
+    it('should correctly set value when using compareWith null', async () => {
+        const page = await newSpecPage({
+            components: [RadioGroup, Radio],
+            template: () => (h("ion-radio-group", { compareWith: null, value: "blue" }, h("ion-radio", { value: "red" }, "Red"), h("br", null), h("ion-radio", { value: "blue" }, "Blue"), h("br", null), h("ion-radio", { value: "green" }, "Green"))),
+        });
+        const radioGroup = page.body.querySelector('ion-radio-group');
+        const radios = document.querySelectorAll('ion-radio');
+        await radios[2].click();
+        await page.waitForChanges();
+        expect(radios[2].getAttribute('aria-checked')).toBe('true');
+        expect(radioGroup.value).toEqual('green');
+    });
+    it('should work with different parameter types', async () => {
+        const page = await newSpecPage({
+            components: [Radio, RadioGroup],
+            template: () => (h("ion-radio-group", { value: 2 }, h("ion-radio", { value: 1 }, "Option #1"), h("ion-radio", { value: 2 }, "Option #2"), h("ion-radio", { value: 3 }, "Option #3"))),
+        });
+        const radioGroup = page.body.querySelector('ion-radio-group');
+        radioGroup.compareWith = (val1, val2) => {
+            // convert val1 to a number
+            return +val1 === val2;
+        };
+        const radios = document.querySelectorAll('ion-radio');
+        await expect(radios[1].getAttribute('aria-checked')).toBe('true');
+        await radios[2].click();
+        await page.waitForChanges();
+        expect(radios[2].getAttribute('aria-checked')).toBe('true');
+        expect(radioGroup.value).toEqual(3);
+    });
+});

package/dist/collection/components/range/range.ios.css

@@ -325,12 +325,6 @@
    * would not be able to see the range.
    */
   max-width: 200px;
-  /**
-   * This ensures that double tapping this text
-   * clicks the <label> and focuses the range
-   * when a screen reader is enabled.
-   */
-  pointer-events: none;
   text-overflow: ellipsis;
   white-space: nowrap;
   overflow: hidden;

package/dist/collection/components/range/range.js

@@ -24,6 +24,7 @@ import { roundToMaxDecimalPlaces } from "../../utils/floating-point";
  * @part knob - The handle that is used to drag the range.
  * @part bar - The inactive part of the bar.
  * @part bar-active - The active part of the bar.
+ * @part label - The label text describing the range.
  */
 export class Range {
     constructor() {
@@ -226,6 +227,8 @@ export class Range {
             this.ionStyle.emit({
                 interactive: true,
                 'interactive-disabled': this.disabled,
+                // TODO(FW-2997): remove this
+                legacy: !!this.legacy,
             });
         }
     }
@@ -414,7 +417,7 @@ Developers can dismiss this warning by removing their usage of the "legacy" prop
             }) }, h("label", { class: "range-wrapper", id: "range-label" }, h("div", { class: {
                 'label-text-wrapper': true,
                 'label-text-wrapper-hidden': !hasLabel,
-            } }, label !== undefined ? h("div", { class: "label-text" }, label) : h("slot", { name: "label" })), h("div", { class: "native-wrapper" }, h("slot", { name: "start" }), this.renderRangeSlider(), h("slot", { name: "end" })))));
+            }, part: "label" }, label !== undefined ? h("div", { class: "label-text" }, label) : h("slot", { name: "label" })), h("div", { class: "native-wrapper" }, h("slot", { name: "start" }), this.renderRangeSlider(), h("slot", { name: "end" })))));
     }
     get hasLabel() {
         return this.label !== undefined || this.el.querySelector('[slot="label"]') !== null;

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

@@ -325,12 +325,6 @@
    * would not be able to see the range.
    */
   max-width: 200px;
-  /**
-   * This ensures that double tapping this text
-   * clicks the <label> and focuses the range
-   * when a screen reader is enabled.
-   */
-  pointer-events: none;
   text-overflow: ellipsis;
   white-space: nowrap;
   overflow: hidden;

package/dist/collection/components/range/test/range.spec.js

@@ -202,4 +202,20 @@ describe('range: item adjustments', () => {
         expect(range.classList.contains('range-item-start-adjustment')).toBe(false);
         expect(range.classList.contains('range-item-end-adjustment')).toBe(false);
     });
+    describe('shadow parts', () => {
+        it('should have shadow parts', async () => {
+            const page = await newSpecPage({
+                components: [Range],
+                html: `<ion-range pin="true" snaps="true" value="50" label="Label"></ion-range>`,
+            });
+            const range = page.body.querySelector('ion-range');
+            expect(range).toHaveShadowPart('label');
+            expect(range).toHaveShadowPart('pin');
+            expect(range).toHaveShadowPart('knob');
+            expect(range).toHaveShadowPart('bar');
+            expect(range).toHaveShadowPart('bar-active');
+            expect(range).toHaveShadowPart('tick');
+            expect(range).toHaveShadowPart('tick-active');
+        });
+    });
 });

package/dist/collection/components/select/select.ios.css

@@ -132,6 +132,10 @@
   overflow: hidden;
 }
 
+:host(.in-item:not(.legacy-select)) {
+  flex: 1 1 0;
+}
+
 :host(.in-item.legacy-select) {
   position: static;
   max-width: 45%;
@@ -232,6 +236,12 @@ button {
   margin-top: 0;
   margin-bottom: 0;
   position: relative;
+  /**
+   * Prevent the icon from shrinking when the label and/or
+   * selected item text is long enough to fill the rest of
+   * the container.
+   */
+  flex-shrink: 0;
 }
 
 /**
@@ -315,6 +325,23 @@ button {
   transition: opacity 150ms cubic-bezier(0.4, 0, 0.2, 1);
 }
 
+.select-wrapper-inner {
+  display: flex;
+  align-items: center;
+  overflow: hidden;
+}
+
+:host(.select-label-placement-stacked) .select-wrapper-inner,
+:host(.select-label-placement-floating) .select-wrapper-inner {
+  /**
+   * When using a stacked/floating label, the inner wrapper is
+   * stacked vertically under the label container. This line
+   * ensures that the inner wrapper fills all the remaining height
+   * of the component.
+   */
+  flex-grow: 1;
+}
+
 :host(.ion-touched.ion-invalid) {
   --highlight-color: var(--highlight-color-invalid);
 }
@@ -555,11 +582,23 @@ button {
  * The placeholder should be hidden when the label
  * is on top of the select. This prevents the label
  * from overlapping any placeholder value.
+ *
+ * TODO(FW-5592): Remove :not(.label-floating) piece
  */
-:host(.select-label-placement-floating) .native-wrapper .select-placeholder {
+:host(.select-label-placement-floating:not(.label-floating)) .native-wrapper .select-placeholder {
   opacity: 0;
 }
 
+/**
+ * We don't use .label-floating here because that would
+ * also include the case where the label is floating due
+ * to content in the start/end slot. We want the opacity
+ * to remain at the default in this case, since the select
+ * isn't being actively interacted with.
+ *
+ * TODO(FW-5592): Change entire selector to:
+ * :host(.label-floating.select-label-placement-floating) .native-wrapper .select-placeholder
+ */
 :host(.select-expanded.select-label-placement-floating) .native-wrapper .select-placeholder,
 :host(.ion-focused.select-label-placement-floating) .native-wrapper .select-placeholder,
 :host(.has-value.select-label-placement-floating) .native-wrapper .select-placeholder {
@@ -569,10 +608,7 @@ button {
 /**
  * This makes the label sit above the input.
  */
-:host(.select-label-placement-stacked) .label-text-wrapper,
-:host(.select-expanded.select-label-placement-floating) .label-text-wrapper,
-:host(.ion-focused.select-label-placement-floating) .label-text-wrapper,
-:host(.has-value.select-label-placement-floating) .label-text-wrapper {
+:host(.label-floating) .label-text-wrapper {
   transform: translateY(50%) scale(0.75);
   /**
    * Label text should not extend
@@ -581,6 +617,23 @@ button {
   max-width: calc(100% / 0.75);
 }
 
+::slotted([slot=start]), ::slotted([slot=end]) {
+  /**
+   * Prevent the slots from shrinking when the label and/or
+   * selected item text is long enough to fill the rest of
+   * the container.
+   */
+  flex-shrink: 0;
+}
+
+::slotted([slot=start]) {
+  margin-inline-end: 16px;
+}
+
+::slotted([slot=end]) {
+  margin-inline-start: 16px;
+}
+
 /**
  * Convert a font size to a dynamic font size.
  * Fonts that participate in Dynamic Type should use
@@ -704,8 +757,8 @@ button {
   color: var(--ion-color-step-650, #595959);
 }
 
-:host(.select-label-placement-stacked) .native-wrapper,
-:host(.select-label-placement-floating) .native-wrapper {
+:host(.select-label-placement-stacked) .select-wrapper-inner,
+:host(.select-label-placement-floating) .select-wrapper-inner {
   width: calc(100% - 1.125rem - 4px);
 }