Files
cunningham/packages/react/src/components/Forms/DatePicker/CalendarCell.tsx
Lebaud Antoine 219d08c82e 🐛(react) prevent ENTER event being triggered twice on calendar grid-cell
iThe Button component is not utilizing the headless UI Button from react-aria.
When pressing ENTER on calendar grid cells, the onClick event handler is
being called twice. This issue remained unnoticed when the date picker only
selected a single date and closed. However, when selecting a range of dates,
this double triggering becomes problematic.
2023-07-10 22:05:33 +02:00

96 lines
2.8 KiB
TypeScript

import React, { useRef } from "react";
import classNames from "classnames";
import { useCalendarCell } from "@react-aria/calendar";
import {
CalendarDate,
getLocalTimeZone,
isSameDay,
isToday,
} from "@internationalized/date";
import { CalendarState, RangeCalendarState } from "@react-stately/calendar";
import { Button } from ":/components/Button";
interface CalendarCellProps {
state: CalendarState | RangeCalendarState;
date: CalendarDate;
}
const isRangeCalendar = (object: any): object is RangeCalendarState => {
return object?.highlightedRange;
};
export const CalendarCell = ({ state, date }: CalendarCellProps) => {
const ref = useRef<HTMLButtonElement>(null);
const {
cellProps,
buttonProps,
isSelected,
formattedDate,
isOutsideVisibleRange,
isDisabled,
} = useCalendarCell({ date }, state, ref);
const isSelectionEnd =
isRangeCalendar(state) && isSameDay(date, state?.highlightedRange?.end);
const isSelectionStart =
isRangeCalendar(state) && isSameDay(date, state?.highlightedRange?.start);
const isWithinHighlightedRange =
isRangeCalendar(state) &&
state?.highlightedRange?.start <= date &&
state?.highlightedRange?.end >= date;
return (
<td {...cellProps}>
<div
hidden={isOutsideVisibleRange}
className={classNames({
"c__calendar__wrapper__grid__week-row__background--range--disabled":
isWithinHighlightedRange && isDisabled,
"c__calendar__wrapper__grid__week-row__background--range":
isWithinHighlightedRange,
"c__calendar__wrapper__grid__week-row__background--range--end":
isSelectionEnd,
"c__calendar__wrapper__grid__week-row__background--range--start":
isSelectionStart,
})}
>
<Button
size="small"
color={
(
isRangeCalendar(state)
? isSelectionStart || isSelectionEnd
: isSelected
)
? "primary"
: "tertiary"
}
className={classNames(
"c__calendar__wrapper__grid__week-row__button",
{
"c__calendar__wrapper__grid__week-row__button--selected":
isSelected,
"c__calendar__wrapper__grid__week-row__button--today": isToday(
date,
getLocalTimeZone()
),
}
)}
disabled={isDisabled}
{...buttonProps}
// The keyboard's ENTER event triggers the button twice.
// We could prevent this behavior using their headless ui
// button component.
onKeyDown={(e) => e.preventDefault()}
onKeyUp={(e) => e.preventDefault()}
ref={ref}
>
{formattedDate}
</Button>
</div>
</td>
);
};