Skip to content

Commit

Permalink
fix(datetime): display today's date and time when value is an empty s…
Browse files Browse the repository at this point in the history
…tring (#29839)

Issue number: resolves #29669

---------

## What is the current behavior?

Setting `value` to an empty string on `<ion-datetime>` renders a May
2021 calendar:

```html
<ion-datetime value=""></ion-datetime>
```

## What is the new behavior?

Show the month and time for today's date when value is an empty string.
This matches how a native `input` with `type="datetime-local"` works.

## Does this introduce a breaking change?

- [ ] Yes
- [x] No

## Other information

This can be tested by removing my fix in `datetime.tsx` and running the
e2e test for Datetime:

```bash
npm run test.e2e src/components/datetime/test/basic/datetime.e2e.ts
```

The `should display today's date and time when value is an empty string`
test should fail.

Alternatively, you can add a datetime with `value=""` and see the
calendar before & after my fix.

---------

Co-authored-by: Tanner Reits <[email protected]>
  • Loading branch information
brandyscarney and tanner-reits committed Sep 6, 2024
1 parent ac3959a commit f1d50c0
Show file tree
Hide file tree
Showing 2 changed files with 34 additions and 1 deletion.
3 changes: 2 additions & 1 deletion core/src/components/datetime/datetime.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -1234,7 +1234,8 @@ export class Datetime implements ComponentInterface {
}

private processValue = (value?: string | string[] | null) => {
const hasValue = value !== null && value !== undefined && (!Array.isArray(value) || value.length > 0);
const hasValue =
value !== null && value !== undefined && value !== '' && (!Array.isArray(value) || value.length > 0);
const valueToProcess = hasValue ? parseDate(value) : this.defaultParts;

const { minParts, maxParts, workingParts, el } = this;
Expand Down
32 changes: 32 additions & 0 deletions core/src/components/datetime/test/basic/datetime.e2e.ts
Original file line number Diff line number Diff line change
Expand Up @@ -121,6 +121,38 @@ configs({ modes: ['ios'], directions: ['ltr'] }).forEach(({ title, config }) =>

await expect(datetime).toHaveJSProperty('value', '2022-10-01T16:22:00');
});

test("should display today's date and time when value is an empty string", async ({ page }) => {
await page.setContent(
`
<ion-datetime locale="en-US" presentation="date-time" value=""></ion-datetime>
<script>
const mockToday = '2024-07-24T16:22';
Date = class extends Date {
constructor(...args) {
if (args.length === 0) {
super(mockToday)
} else {
super(...args);
}
}
}
</script>
`,
config
);

await page.locator('.datetime-ready').waitFor();

// July 24, 2024
const todayButton = page.locator('.calendar-day[data-day="24"][data-month="7"][data-year="2024"]');
await expect(todayButton).toHaveClass(/calendar-day-today/);

// 4:22 PM
const timeBody = page.locator('ion-datetime .time-body');
await expect(timeBody).toHaveText('4:22 PM');
});
});
});

Expand Down

0 comments on commit f1d50c0

Please sign in to comment.