@ionic/core 7.4.4-dev.11699474548.1f806781 → 7.4.4-dev.11699475060.1eee2115

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

@@ -1,36 +1,3 @@
-/**
- * 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.
- */
 :host {
   /**
    * @prop --background: The primary background of the datetime component.
@@ -253,18 +220,6 @@ ion-picker-column-internal {
   justify-content: space-between;
 }
 
-/**
- * The confirm and clear buttons are grouped in a
- * container so that they appear on the end opposite
- * of the cancel button.
- * We use display: flex so that the
- * wrapper only takes up as much
- * height as it needs.
- */
-.datetime-action-buttons .datetime-action-buttons-container {
-  display: flex;
-}
-
 /**
  * Date/Year button should be on
  * the opposite side of the component
@@ -300,20 +255,6 @@ ion-picker-column-internal {
   text-align: center;
 }
 
-.calendar-days-of-week .day-of-week {
-  /**
-   * Center the days of the week within each cell
-   * if the container size is changed as a result
-   * of Dynamic Type.
-   */
-  -webkit-margin-start: auto;
-  margin-inline-start: auto;
-  -webkit-margin-end: auto;
-  margin-inline-end: auto;
-  margin-top: 0;
-  margin-bottom: 0;
-}
-
 :host .calendar-body {
   /**
    * Show all calendar months inline
@@ -388,7 +329,7 @@ ion-picker-column-internal {
  * Center the day text vertically
  * and horizontally within its grid cell.
  */
-.calendar-day {
+:host .calendar-day {
   border-radius: 50%;
   -webkit-padding-start: 0px;
   padding-inline-start: 0px;
@@ -477,39 +418,6 @@ ion-picker-column-internal {
   --color: var(--ion-color-base);
 }
 
-/**
- * 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.
- */
 :host {
   --background: var(--ion-color-light, #ffffff);
   --background-rgb: var(--ion-color-light-rgb);
@@ -530,7 +438,7 @@ ion-picker-column-internal {
   padding-top: 16px;
   padding-bottom: 16px;
   border-bottom: 0.55px solid var(--ion-color-step-200, #cccccc);
-  font-size: min(0.875rem, 22.4px);
+  font-size: 14px;
 }
 
 :host .datetime-header .datetime-title {
@@ -545,7 +453,7 @@ ion-picker-column-internal {
   --padding-start: 16px;
   --background-hover: transparent;
   --background-activated: transparent;
-  font-size: min(1rem, 25.6px);
+  font-size: 16px;
   font-weight: 600;
 }
 
@@ -576,111 +484,12 @@ ion-picker-column-internal {
   padding-top: 0;
   padding-bottom: 0;
   color: var(--ion-color-step-300, #b3b3b3);
-  font-size: min(0.75rem, 19.2px);
+  font-size: 12px;
   font-weight: 600;
   line-height: 24px;
   text-transform: uppercase;
 }
 
-@supports (border-radius: mod(1px, 1px)) {
-  .calendar-days-of-week .day-of-week {
-    /**
-     * When the computed font size is 24px
-     * we want the days of the week to only
-     * show the first letter of each day.
-     *
-     * We use the modulus operator to ensure
-     * that any font size less than 24px is
-     * unchanged (i.e. 16 % 24 = 16). If the
-     * font size is 24px then the remainder
-     * will be 0.
-     */
-    /**
-     * From here, we scale the remainder to get
-     * a width value larger than 20px.
-     * We then clamp the width to have a max width
-     * of 100% so that each element does not appear
-     * too wide if $width is a very large number.
-     *
-     * When the font size is >=24px then $remainder
-     * will be 0px. This results in $width being 0px
-     * since 0px * 10 is still 0px. We set a minimum
-     * width of 20px below so that the days of the
-     * week still show when the font size is >=24px
-     * and have a fixed width of 20px.
-     *
-     * The days of the week text on iOS have a maximum font size
-     * that is no larger than 20px, so setting a minimum width
-     * of 20px ensures that the text will always fit. (See font-size
-     * declaration in the .calendar-days-of-week block).
-     */
-    width: clamp(20px, calc(mod(min(1rem, 24px), 24px) * 10), 100%);
-    /**
-     * We show the first letter of the week
-     * by setting the width small such that all other
-     * letters wrap to the next line. While we do have
-     * overflow: hidden to visually hide the overflowing
-     * text, we need to explicitly set the height of the
-     * container too otherwise it will increase as the text wraps.
-     */
-    height: 24px;
-    /**
-     * The above width will cause
-     * all the other letters to overflow
-     * to other lines. The below code hides
-     * those additional letters so only
-     * the first letter is shown.
-     */
-    overflow: hidden;
-  }
-  .calendar-day {
-    /**
-     * We use the modulus operator to ensure
-     * that any font size less than 24px is
-     * unchanged (i.e. 16 % 24 = 16). If the
-     * font size is 24px then the remainder
-     * will be 0.
-     */
-    /**
-     * From here, we scale the remainder. The purpose of
-     * scaling the remainder is to get a border radius
-     * large enough such that the element's
-     * shape is a circle.
-     *
-     * Note that when the font size is 24px
-     * then the remainder will be 0, so $radius
-     * will still be zero.
-     */
-    /**
-     * In the event that $radius is 0
-     * we use max() to ensure a minimum
-     * border radius of 8px which will
-     * give the day button the rounded
-     * rectangle appearance.
-     * For font sizes smaller than 24px
-     * the border radius will give a circle appearance.
-     */
-    border-radius: max(8px, mod(min(1rem, 24px), 24px) * 10);
-  }
-}
-/**
- * iOS <17 has a bug where the result of mod() is
- * used incorrectly if used inside of another CSS function such
- * as calc(). The first set of @support checks accounts for iOS <17.
- * We check -webkit-named-image to ensure that non-WebKit browsers are ignored.
- * The second set of @support checks account for all other browsers that
- * do not support mod() yet.
- */
-@supports ((border-radius: mod(1px, 1px)) and (background: -webkit-named-image(apple-pay-logo-black)) and (not (contain-intrinsic-size: none))) or (not (border-radius: mod(1px, 1px))) {
-  .calendar-days-of-week .day-of-week {
-    width: auto;
-    height: auto;
-    overflow: initial;
-  }
-  .calendar-day {
-    border-radius: 32px;
-  }
-}
 :host .calendar-body .calendar-month .calendar-month-grid {
   /**
    * We need to apply the padding to
@@ -707,14 +516,14 @@ ion-picker-column-internal {
   padding-top: 4px;
   padding-bottom: 4px;
   height: 0;
-  min-height: 1rem;
+  min-height: 16px;
 }
 
 :host .calendar-day {
   width: 40px;
   min-width: 40px;
   height: 40px;
-  font-size: min(1.25rem, 32px);
+  font-size: 20px;
 }
 
 .calendar-day.calendar-day-active {
@@ -754,7 +563,7 @@ ion-picker-column-internal {
   padding-inline-end: 16px;
   padding-top: 8px;
   padding-bottom: 16px;
-  font-size: min(1rem, 25.6px);
+  font-size: 16px;
 }
 
 :host .datetime-time .time-header {

package/dist/collection/components/datetime/datetime.js

@@ -12,7 +12,7 @@ import { getIonMode } from "../../global/ionic-global";
 import { isSameDay, warnIfValueOutOfBounds, isBefore, isAfter } from "./utils/comparison";
 import { generateMonths, getDaysOfMonth, getDaysOfWeek, getToday, getMonthColumnData, getDayColumnData, getYearColumnData, getTimeColumnsData, getCombinedDateColumnData, } from "./utils/data";
 import { formatValue, getLocalizedTime, getMonthAndDay, getMonthAndYear } from "./utils/format";
-import { isLocaleDayPeriodRTL, isMonthFirstLocale, getNumDaysInMonth, getHourCycle } from "./utils/helpers";
+import { is24Hour, isLocaleDayPeriodRTL, isMonthFirstLocale, getNumDaysInMonth } from "./utils/helpers";
 import { calculateHourFromAMPM, convertDataToISO, getClosestValidDate, getEndOfWeek, getNextDay, getNextMonth, getNextWeek, getNextYear, getPreviousDay, getPreviousMonth, getPreviousWeek, getPreviousYear, getStartOfWeek, validateParts, } from "./utils/manipulation";
 import { clampDate, convertToArrayOfNumbers, getPartsFromCalendarDay, parseAmPm, parseDate, parseMaxParts, parseMinParts, } from "./utils/parse";
 import { getCalendarDayState, getHighlightStyles, isDayDisabled, isMonthDisabled, isNextMonthDisabled, isPrevMonthDisabled, } from "./utils/state";
@@ -933,7 +933,7 @@ export class Datetime {
     return (h("div", { class: "datetime-footer" }, h("div", { class: "datetime-buttons" }, h("div", { class: {
         ['datetime-action-buttons']: true,
         ['has-clear-button']: this.showClearButton,
-      } }, h("slot", { name: "buttons" }, h("ion-buttons", null, showDefaultButtons && (h("ion-button", { id: "cancel-button", color: this.color, onClick: () => this.cancel(true) }, this.cancelText)), h("div", { class: "datetime-action-buttons-container" }, showClearButton && (h("ion-button", { id: "clear-button", color: this.color, onClick: () => clearButtonClick() }, this.clearText)), showDefaultButtons && (h("ion-button", { id: "confirm-button", color: this.color, onClick: () => this.confirm(true) }, this.doneText)))))))));
+      } }, h("slot", { name: "buttons" }, h("ion-buttons", null, showDefaultButtons && (h("ion-button", { id: "cancel-button", color: this.color, onClick: () => this.cancel(true) }, this.cancelText)), h("div", null, showClearButton && (h("ion-button", { id: "clear-button", color: this.color, onClick: () => clearButtonClick() }, this.clearText)), showDefaultButtons && (h("ion-button", { id: "confirm-button", color: this.color, onClick: () => this.confirm(true) }, this.doneText)))))))));
   }
   /**
    * Wheel picker render methods
@@ -1370,7 +1370,7 @@ export class Datetime {
   }
   renderTimeOverlay() {
     const { hourCycle, isTimePopoverOpen, locale } = this;
-    const computedHourCycle = getHourCycle(locale, hourCycle);
+    const use24Hour = is24Hour(locale, hourCycle);
     const activePart = this.getActivePartsWithFallback();
     return [
       h("div", { class: "time-header" }, this.renderTimeLabel()),
@@ -1389,7 +1389,7 @@ export class Datetime {
             await popoverRef.onWillDismiss();
             this.isTimePopoverOpen = false;
           }
-        } }, getLocalizedTime(locale, activePart, computedHourCycle)),
+        } }, getLocalizedTime(locale, activePart, use24Hour)),
       h("ion-popover", { alignment: "center", translucent: true, overlayIndex: 1, arrow: false, onWillPresent: (ev) => {
           /**
            * Intersection Observers do not consistently fire between Blink and Webkit
@@ -2029,15 +2029,9 @@ export class Datetime {
         "type": "string",
         "mutable": false,
         "complexType": {
-          "original": "DatetimeHourCycle",
-          "resolved": "\"h11\" | \"h12\" | \"h23\" | \"h24\" | undefined",
-          "references": {
-            "DatetimeHourCycle": {
-              "location": "import",
-              "path": "./datetime-interface",
-              "id": "src/components/datetime/datetime-interface.ts::DatetimeHourCycle"
-            }
-          }
+          "original": "'h23' | 'h12'",
+          "resolved": "\"h12\" | \"h23\" | undefined",
+          "references": {}
         },
         "required": false,
         "optional": true,

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

@@ -1,36 +1,3 @@
-/**
- * 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.
- */
 :host {
   /**
    * @prop --background: The primary background of the datetime component.
@@ -253,18 +220,6 @@ ion-picker-column-internal {
   justify-content: space-between;
 }
 
-/**
- * The confirm and clear buttons are grouped in a
- * container so that they appear on the end opposite
- * of the cancel button.
- * We use display: flex so that the
- * wrapper only takes up as much
- * height as it needs.
- */
-.datetime-action-buttons .datetime-action-buttons-container {
-  display: flex;
-}
-
 /**
  * Date/Year button should be on
  * the opposite side of the component
@@ -300,20 +255,6 @@ ion-picker-column-internal {
   text-align: center;
 }
 
-.calendar-days-of-week .day-of-week {
-  /**
-   * Center the days of the week within each cell
-   * if the container size is changed as a result
-   * of Dynamic Type.
-   */
-  -webkit-margin-start: auto;
-  margin-inline-start: auto;
-  -webkit-margin-end: auto;
-  margin-inline-end: auto;
-  margin-top: 0;
-  margin-bottom: 0;
-}
-
 :host .calendar-body {
   /**
    * Show all calendar months inline
@@ -388,7 +329,7 @@ ion-picker-column-internal {
  * Center the day text vertically
  * and horizontally within its grid cell.
  */
-.calendar-day {
+:host .calendar-day {
   border-radius: 50%;
   -webkit-padding-start: 0px;
   padding-inline-start: 0px;
@@ -477,39 +418,6 @@ ion-picker-column-internal {
   --color: var(--ion-color-base);
 }
 
-/**
- * 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.
- */
 :host {
   --background: var(--ion-color-step-100, #ffffff);
   --title-color: var(--ion-color-contrast);
@@ -527,13 +435,13 @@ ion-picker-column-internal {
 }
 
 :host .datetime-header .datetime-title {
-  font-size: 0.75rem;
+  font-size: 12px;
   text-transform: uppercase;
 }
 
 :host .datetime-header .datetime-selected-date {
   margin-top: 30px;
-  font-size: 2.125rem;
+  font-size: 34px;
 }
 
 :host .datetime-calendar .calendar-action-buttons ion-item {
@@ -553,7 +461,7 @@ ion-picker-column-internal {
   padding-top: 0px;
   padding-bottom: 0px;
   color: var(--ion-color-step-500, gray);
-  font-size: 0.875rem;
+  font-size: 14px;
   line-height: 36px;
 }
 
@@ -578,7 +486,7 @@ ion-picker-column-internal {
   width: 42px;
   min-width: 42px;
   height: 42px;
-  font-size: 0.875rem;
+  font-size: 14px;
 }
 
 /**

package/dist/collection/components/datetime/test/hour-cycle/datetime.e2e.js

@@ -19,19 +19,5 @@ configs({ modes: ['ios'], directions: ['ltr'] }).forEach(({ title, config }) =>
       const timeButton = page.locator('ion-datetime .time-body');
       await expect(timeButton).toHaveText('4:30 PM');
     });
-    test('should set the h11 hour cycle correctly', async ({ page }) => {
-      await page.setContent(`
-        <ion-datetime hour-cycle="h11" value="2022-01-01T00:30:00"></ion-datetime>
-      `, config);
-      const timeButton = page.locator('ion-datetime .time-body');
-      await expect(timeButton).toHaveText('0:30 AM');
-    });
-    test('should set the h24 hour cycle correctly', async ({ page }) => {
-      await page.setContent(`
-        <ion-datetime hour-cycle="h24" value="2022-01-01T00:30:00"></ion-datetime>
-      `, config);
-      const timeButton = page.locator('ion-datetime .time-body');
-      await expect(timeButton).toHaveText('24:30');
-    });
   });
 });

package/dist/collection/components/datetime/utils/data.js

@@ -3,7 +3,7 @@
  */
 import { isAfter, isBefore, isSameDay } from "./comparison";
 import { getLocalizedDayPeriod, removeDateTzOffset, getFormattedHour, addTimePadding, getTodayLabel, getYear, } from "./format";
-import { getNumDaysInMonth, is24Hour, getHourCycle } from "./helpers";
+import { getNumDaysInMonth, is24Hour } from "./helpers";
 import { getNextMonth, getPreviousMonth, getInternalHourValue } from "./manipulation";
 /**
  * Returns the current date as
@@ -34,14 +34,8 @@ const minutes = [
   0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13, 14, 15, 16, 17, 18, 19, 20, 21, 22, 23, 24, 25, 26, 27, 28, 29, 30, 31,
   32, 33, 34, 35, 36, 37, 38, 39, 40, 41, 42, 43, 44, 45, 46, 47, 48, 49, 50, 51, 52, 53, 54, 55, 56, 57, 58, 59,
 ];
-// h11 hour system uses 0-11. Midnight starts at 0:00am.
-const hour11 = [0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11];
-// h12 hour system uses 0-12. Midnight starts at 12:00am.
 const hour12 = [0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11];
-// h23 hour system uses 0-23. Midnight starts at 0:00.
 const hour23 = [0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13, 14, 15, 16, 17, 18, 19, 20, 21, 22, 23];
-// h24 hour system uses 1-24. Midnight starts at 24:00.
-const hour24 = [1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13, 14, 15, 16, 17, 18, 19, 20, 21, 22, 23, 0];
 /**
  * Given a locale and a mode,
  * return an array with formatted days
@@ -110,33 +104,14 @@ export const getDaysOfMonth = (month, year, firstDayOfWeek) => {
   }
   return days;
 };
-/**
- * Returns an array of pre-defined hour
- * values based on the provided hourCycle.
- */
-const getHourData = (hourCycle) => {
-  switch (hourCycle) {
-    case 'h11':
-      return hour11;
-    case 'h12':
-      return hour12;
-    case 'h23':
-      return hour23;
-    case 'h24':
-      return hour24;
-    default:
-      throw new Error(`Invalid hour cycle "${hourCycle}"`);
-  }
-};
 /**
  * Given a local, reference datetime parts and option
  * max/min bound datetime parts, calculate the acceptable
  * hour and minute values according to the bounds and locale.
  */
-export const generateTime = (locale, refParts, hourCycle = 'h12', minParts, maxParts, hourValues, minuteValues) => {
-  const computedHourCycle = getHourCycle(locale, hourCycle);
-  const use24Hour = is24Hour(computedHourCycle);
-  let processedHours = getHourData(computedHourCycle);
+export const generateTime = (refParts, hourCycle = 'h12', minParts, maxParts, hourValues, minuteValues) => {
+  const use24Hour = hourCycle === 'h23';
+  let processedHours = use24Hour ? hour23 : hour12;
   let processedMinutes = minutes;
   let isAMAllowed = true;
   let isPMAllowed = true;
@@ -468,12 +443,11 @@ export const getCombinedDateColumnData = (locale, todayParts, minParts, maxParts
   };
 };
 export const getTimeColumnsData = (locale, refParts, hourCycle, minParts, maxParts, allowedHourValues, allowedMinuteValues) => {
-  const computedHourCycle = getHourCycle(locale, hourCycle);
-  const use24Hour = is24Hour(computedHourCycle);
-  const { hours, minutes, am, pm } = generateTime(locale, refParts, computedHourCycle, minParts, maxParts, allowedHourValues, allowedMinuteValues);
+  const use24Hour = is24Hour(locale, hourCycle);
+  const { hours, minutes, am, pm } = generateTime(refParts, use24Hour ? 'h23' : 'h12', minParts, maxParts, allowedHourValues, allowedMinuteValues);
   const hoursItems = hours.map((hour) => {
     return {
-      text: getFormattedHour(hour, computedHourCycle),
+      text: getFormattedHour(hour, use24Hour),
       value: getInternalHourValue(hour, use24Hour, refParts.ampm),
     };
   });

package/dist/collection/components/datetime/utils/format.js

@@ -1,7 +1,6 @@
 /*!
  * (C) Ionic http://ionicframework.com - MIT License
  */
-import { is24Hour } from "./helpers";
 import { convertDataToISO } from "./manipulation";
 const getFormattedDayPeriod = (dayPeriod) => {
   if (dayPeriod === undefined) {
@@ -9,7 +8,7 @@ const getFormattedDayPeriod = (dayPeriod) => {
   }
   return dayPeriod.toUpperCase();
 };
-export const getLocalizedTime = (locale, refParts, hourCycle) => {
+export const getLocalizedTime = (locale, refParts, use24Hour) => {
   const timeParts = {
     hour: refParts.hour,
     minute: refParts.minute,
@@ -31,7 +30,7 @@ export const getLocalizedTime = (locale, refParts, hourCycle) => {
      * We use hourCycle here instead of hour12 due to:
      * https://bugs.chromium.org/p/chromium/issues/detail?id=1347316&q=hour12&can=2
      */
-    hourCycle,
+    hourCycle: use24Hour ? 'h23' : 'h12',
     /**
      * Setting Z at the end indicates that this
      * date string is in the UTC time zone. This
@@ -70,33 +69,17 @@ export const addTimePadding = (value) => {
  * 12 hour times it ensures that
  * hour 0 is formatted as '12'.
  */
-export const getFormattedHour = (hour, hourCycle) => {
-  /**
-   * Midnight for h11 starts at 0:00am
-   * Midnight for h12 starts at 12:00am
-   * Midnight for h23 starts at 00:00
-   * Midnight for h24 starts at 24:00
-   */
-  if (hour === 0) {
-    switch (hourCycle) {
-      case 'h11':
-        return '0';
-      case 'h12':
-        return '12';
-      case 'h23':
-        return '00';
-      case 'h24':
-        return '24';
-      default:
-        throw new Error(`Invalid hour cycle "${hourCycle}"`);
-    }
+export const getFormattedHour = (hour, use24Hour) => {
+  if (use24Hour) {
+    return addTimePadding(hour);
   }
-  const use24Hour = is24Hour(hourCycle);
   /**
-   * h23 and h24 use 24 hour times.
+   * If using 12 hour
+   * format, make sure hour
+   * 0 is formatted as '12'.
    */
-  if (use24Hour) {
-    return addTimePadding(hour);
+  if (hour === 0) {
+    return '12';
   }
   return hour.toString();
 };