@ionic/core 7.6.7-dev.11706571117.1bcbc36b → 7.6.7-dev.11706824245.18d35376

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

@@ -170,10 +170,10 @@ ion-spinner {
  * }
  */
 :host {
-  --background: var(--ion-overlay-background-color, var(--ion-color-step-100, #f9f9f9));
+  --background: var(--ion-overlay-background-color, var(--ion-color-step-100, var(--ion-background-color-step-100, #f9f9f9)));
   --max-width: 270px;
   --max-height: 90%;
-  --spinner-color: var(--ion-color-step-600, #666666);
+  --spinner-color: var(--ion-color-step-600, var(--ion-text-color-step-400, #666666));
   --backdrop-opacity: var(--ion-backdrop-opacity, 0.3);
   color: var(--ion-text-color, #000);
   font-size: 0.875rem;

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

@@ -170,12 +170,12 @@ ion-spinner {
  * }
  */
 :host {
-  --background: var(--ion-color-step-50, #f2f2f2);
+  --background: var(--ion-color-step-50, var(--ion-background-color-step-50, #f2f2f2));
   --max-width: 280px;
   --max-height: 90%;
-  --spinner-color: var(--ion-color-primary, #3880ff);
+  --spinner-color: var(--ion-color-primary, #0054e9);
   --backdrop-opacity: var(--ion-backdrop-opacity, 0.32);
-  color: var(--ion-color-step-850, #262626);
+  color: var(--ion-color-step-850, var(--ion-text-color-step-150, #262626));
   font-size: 0.875rem;
 }
 

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

@@ -4,20 +4,36 @@
 import AxeBuilder from "@axe-core/playwright";
 import { expect } from "@playwright/test";
 import { configs, test } from "../../../../utils/test/playwright/index";
-configs({ modes: ['ios'], directions: ['ltr'] }).forEach(({ title, config }) => {
+configs({ modes: ['ios'], directions: ['ltr'], themes: ['light', 'dark'] }).forEach(({ title, config }) => {
     test.describe(title('loading: a11y'), () => {
         test('should set aria-labelledby with a message', async ({ page }) => {
-            await page.goto('/src/components/loading/test/a11y', config);
+            await page.setContent(`
+          <ion-loading></ion-loading>
+
+          <script>
+            const loading = document.querySelector('ion-loading');
+            loading.message = 'Loading';
+          </script>
+        `, config);
             const ionLoadingDidPresent = await page.spyOnEvent('ionLoadingDidPresent');
-            await page.click('#open-message-loading');
+            const loading = page.locator('ion-loading');
+            await loading.evaluate((el) => el.present());
             await ionLoadingDidPresent.next();
             const results = await new AxeBuilder({ page }).analyze();
             expect(results.violations).toEqual([]);
         });
         test('should set aria-labelledby with a label', async ({ page }) => {
-            await page.goto('/src/components/loading/test/a11y', config);
+            await page.setContent(`
+          <ion-loading aria-label="Loading"></ion-loading>
+
+          <script>
+            const loading = document.querySelector('ion-loading');
+            loading.spinner = 'bubbles';
+          </script>
+        `, config);
             const ionLoadingDidPresent = await page.spyOnEvent('ionLoadingDidPresent');
-            await page.click('#open-label-loading');
+            const loading = page.locator('ion-loading');
+            await loading.evaluate((el) => el.present());
             await ionLoadingDidPresent.next();
             const results = await new AxeBuilder({ page }).analyze();
             expect(results.violations).toEqual([]);

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

@@ -295,10 +295,49 @@ ion-backdrop {
   cursor: pointer;
 }
 
+/**
+ * The split pane styles for menu are defined
+ * in the menu stylesheets instead in the split pane
+ * stylesheets with ::slotted to allow for menus
+ * to be wrapped in custom components.
+ * If we used ::slotted to target the menu
+ * then menus wrapped in components would never
+ * receive these styles because they are not
+ * children of the split pane.
+ */
+/**
+ * Do not pass CSS Variables down on larger
+ * screens as we want them to affect the outer
+ * `ion-menu` rather than the inner content
+ */
 :host(.menu-pane-visible) {
-  width: var(--width);
-  min-width: var(--min-width);
-  max-width: var(--max-width);
+  flex: 0 1 auto;
+  width: var(--side-width, var(--width));
+  min-width: var(--side-min-width, var(--min-width));
+  max-width: var(--side-max-width, var(--max-width));
+}
+
+:host(.menu-pane-visible.split-pane-side) {
+  left: 0;
+  right: 0;
+  top: 0;
+  bottom: 0;
+  position: relative;
+  box-shadow: none;
+  z-index: 0;
+}
+
+:host(.menu-pane-visible.split-pane-side.menu-enabled) {
+  display: flex;
+  flex-shrink: 0;
+}
+
+:host(.menu-pane-visible.split-pane-side) {
+  order: -1;
+}
+
+:host(.menu-pane-visible.split-pane-side[side=end]) {
+  order: 1;
 }
 
 :host(.menu-pane-visible) .menu-inner {
@@ -314,6 +353,28 @@ ion-backdrop {
   display: hidden !important;
 }
 
+:host(.menu-pane-visible.split-pane-side) {
+  -webkit-border-start: 0;
+  border-inline-start: 0;
+  -webkit-border-end: var(--border);
+  border-inline-end: var(--border);
+  border-top: 0;
+  border-bottom: 0;
+  min-width: var(--side-min-width);
+  max-width: var(--side-max-width);
+}
+
+:host(.menu-pane-visible.split-pane-side[side=end]) {
+  -webkit-border-start: var(--border);
+  border-inline-start: var(--border);
+  -webkit-border-end: 0;
+  border-inline-end: 0;
+  border-top: 0;
+  border-bottom: 0;
+  min-width: var(--side-min-width);
+  max-width: var(--side-max-width);
+}
+
 /**
  * Convert a font size to a dynamic font size.
  * Fonts that participate in Dynamic Type should use

package/dist/collection/components/menu/menu.js

@@ -7,6 +7,7 @@ import { GESTURE_CONTROLLER } from "../../utils/gesture/index";
 import { inheritAriaAttributes, assert, clamp, isEndSide as isEnd } from "../../utils/helpers";
 import { menuController } from "../../utils/menu-controller/index";
 import { getPresentedOverlay } from "../../utils/overlays";
+import { hostContext } from "../../utils/theme";
 import { config } from "../../global/config";
 import { getIonMode } from "../../global/ionic-global";
 const iosEasing = 'cubic-bezier(0.32,0.72,0,1)';
@@ -138,6 +139,19 @@ export class Menu {
     }
     async componentDidLoad() {
         this.didLoad = true;
+        /**
+         * A menu inside of a split pane is assumed
+         * to be a side pane.
+         *
+         * When the menu is loaded it needs to
+         * see if it should be considered visible inside
+         * of the split pane. If the split pane is
+         * hidden then the menu should be too.
+         */
+        const splitPane = this.el.closest('ion-split-pane');
+        if (splitPane !== null) {
+            this.isPaneVisible = await splitPane.isVisible();
+        }
         this.menuChanged();
         this.updateState();
     }
@@ -173,20 +187,11 @@ export class Menu {
         this.contentEl = undefined;
     }
     onSplitPaneChanged(ev) {
-        const { target } = ev;
         const closestSplitPane = this.el.closest('ion-split-pane');
-        /**
-         * Menu listens on the body for "ionSplitPaneVisible".
-         * However, this means the callback will run any time
-         * a SplitPane changes visibility. As a result, we only want
-         * Menu's visibility state to update if its parent SplitPane
-         * changes visibility.
-         */
-        if (target !== closestSplitPane) {
-            return;
+        if (closestSplitPane !== null && closestSplitPane === ev.target) {
+            this.isPaneVisible = ev.detail.visible;
+            this.updateState();
         }
-        this.isPaneVisible = ev.detail.isPane(this.el);
-        this.updateState();
     }
     onBackdropClick(ev) {
         // TODO(FW-2832): type (CustomEvent triggers errors which should be sorted)
@@ -593,7 +598,7 @@ export class Menu {
         }
     }
     render() {
-        const { type, disabled, isPaneVisible, inheritedAttributes, side } = this;
+        const { type, disabled, el, isPaneVisible, inheritedAttributes, side } = this;
         const mode = getIonMode(this);
         return (h(Host, { role: "navigation", "aria-label": inheritedAttributes['aria-label'] || 'menu', class: {
                 [mode]: true,
@@ -601,6 +606,7 @@ export class Menu {
                 'menu-enabled': !disabled,
                 [`menu-side-${side}`]: true,
                 'menu-pane-visible': isPaneVisible,
+                'split-pane-side': hostContext('ion-split-pane', el),
             } }, h("div", { class: "menu-inner", part: "container", ref: (el) => (this.menuInnerEl = el) }, h("slot", null)), h("ion-backdrop", { ref: (el) => (this.backdropEl = el), class: "menu-backdrop", tappable: false, stopPropagation: false, part: "backdrop" })));
     }
     static get is() { return "ion-menu"; }

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

@@ -295,10 +295,49 @@ ion-backdrop {
   cursor: pointer;
 }
 
+/**
+ * The split pane styles for menu are defined
+ * in the menu stylesheets instead in the split pane
+ * stylesheets with ::slotted to allow for menus
+ * to be wrapped in custom components.
+ * If we used ::slotted to target the menu
+ * then menus wrapped in components would never
+ * receive these styles because they are not
+ * children of the split pane.
+ */
+/**
+ * Do not pass CSS Variables down on larger
+ * screens as we want them to affect the outer
+ * `ion-menu` rather than the inner content
+ */
 :host(.menu-pane-visible) {
-  width: var(--width);
-  min-width: var(--min-width);
-  max-width: var(--max-width);
+  flex: 0 1 auto;
+  width: var(--side-width, var(--width));
+  min-width: var(--side-min-width, var(--min-width));
+  max-width: var(--side-max-width, var(--max-width));
+}
+
+:host(.menu-pane-visible.split-pane-side) {
+  left: 0;
+  right: 0;
+  top: 0;
+  bottom: 0;
+  position: relative;
+  box-shadow: none;
+  z-index: 0;
+}
+
+:host(.menu-pane-visible.split-pane-side.menu-enabled) {
+  display: flex;
+  flex-shrink: 0;
+}
+
+:host(.menu-pane-visible.split-pane-side) {
+  order: -1;
+}
+
+:host(.menu-pane-visible.split-pane-side[side=end]) {
+  order: 1;
 }
 
 :host(.menu-pane-visible) .menu-inner {
@@ -314,6 +353,28 @@ ion-backdrop {
   display: hidden !important;
 }
 
+:host(.menu-pane-visible.split-pane-side) {
+  -webkit-border-start: 0;
+  border-inline-start: 0;
+  -webkit-border-end: var(--border);
+  border-inline-end: var(--border);
+  border-top: 0;
+  border-bottom: 0;
+  min-width: var(--side-min-width);
+  max-width: var(--side-max-width);
+}
+
+:host(.menu-pane-visible.split-pane-side[side=end]) {
+  -webkit-border-start: var(--border);
+  border-inline-start: var(--border);
+  -webkit-border-end: 0;
+  border-inline-end: 0;
+  border-top: 0;
+  border-bottom: 0;
+  min-width: var(--side-min-width);
+  max-width: var(--side-max-width);
+}
+
 /**
  * Convert a font size to a dynamic font size.
  * Fonts that participate in Dynamic Type should use

package/dist/collection/components/menu-button/menu-button.ios.css

@@ -261,7 +261,7 @@ ion-icon {
   --background-focused: currentColor;
   --background-focused-opacity: .1;
   --border-radius: 4px;
-  --color: var(--ion-color-primary, #3880ff);
+  --color: var(--ion-color-primary, #0054e9);
   --padding-start: 5px;
   --padding-end: 5px;
   min-height: 32px;

package/dist/collection/components/menu-button/test/a11y/menu-button.e2e.js

@@ -4,6 +4,20 @@
 import AxeBuilder from "@axe-core/playwright";
 import { expect } from "@playwright/test";
 import { configs, test } from "../../../../utils/test/playwright/index";
+/**
+ * Only ios mode uses ion-color() for the menu button
+ */
+configs({ directions: ['ltr'], modes: ['ios'], themes: ['light', 'dark'] }).forEach(({ title, config }) => {
+    test.describe(title('menu-button: a11y for ion-color()'), () => {
+        test('should not have accessibility violations', async ({ page }) => {
+            await page.setContent(`
+        <ion-menu-button auto-hide="false"></ion-menu-button>
+      `, config);
+            const results = await new AxeBuilder({ page }).analyze();
+            expect(results.violations).toEqual([]);
+        });
+    });
+});
 configs({ directions: ['ltr'] }).forEach(({ title, screenshot, config }) => {
     test.describe(title('menu-button: a11y'), () => {
         test('should not have accessibility violations', async ({ page }) => {

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

@@ -165,7 +165,7 @@ ion-backdrop {
    */
   transform: translateZ(0);
   border: 0;
-  background: var(--ion-color-step-350, #c0c0be);
+  background: var(--ion-color-step-350, var(--ion-background-color-step-350, #c0c0be));
   cursor: pointer;
   z-index: 11;
 }

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

@@ -165,7 +165,7 @@ ion-backdrop {
    */
   transform: translateZ(0);
   border: 0;
-  background: var(--ion-color-step-350, #c0c0be);
+  background: var(--ion-color-step-350, var(--ion-background-color-step-350, #c0c0be));
   cursor: pointer;
   z-index: 11;
 }

package/dist/collection/components/modal/test/standalone/modal.e2e.js

@@ -13,7 +13,7 @@ configs({ modes: ['ios'], directions: ['ltr'] }).forEach(({ title, config }) =>
             const modal = page.locator('ion-modal');
             await modal.evaluate((el) => el.dismiss());
             await ionModalDidDismiss.next();
-            await page.locator('ion-modal').waitFor({ state: 'detached' });
+            await page.waitForSelector('ion-modal', { state: 'detached' });
         });
     });
 });

package/dist/collection/components/nav/nav.js

@@ -320,8 +320,11 @@ export class Nav {
     async getPrevious(view) {
         return this.getPreviousSync(view);
     }
-    getLength() {
-        return this.views.length;
+    /**
+     * Returns the number of views in the stack.
+     */
+    async getLength() {
+        return Promise.resolve(this.views.length);
     }
     getActiveSync() {
         return this.views[this.views.length - 1];
@@ -1763,6 +1766,23 @@ export class Nav {
                             "text": "view The view to get."
                         }]
                 }
+            },
+            "getLength": {
+                "complexType": {
+                    "signature": "() => Promise<number>",
+                    "parameters": [],
+                    "references": {
+                        "Promise": {
+                            "location": "global",
+                            "id": "global::Promise"
+                        }
+                    },
+                    "return": "Promise<number>"
+                },
+                "docs": {
+                    "text": "Returns the number of views in the stack.",
+                    "tags": []
+                }
             }
         };
     }