Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

fix(datePicker): prevent year, month change in showMonthAndYearPickers mode #3088

Merged
merged 7 commits into from
Jul 8, 2024
Merged

fix(datePicker): prevent year, month change in showMonthAndYearPickers mode #3088

Changes from 1 commit
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
30e4f3c
fix: prevent year, month change error in showMonthAndYearPickers mode
KumJungMin May 27, 2024
c13401b
docs: add changeset
KumJungMin May 27, 2024
4ef3a89
docs: edit changeset
KumJungMin May 27, 2024
ac31e00
test: add test code about date clear issue in date picker
KumJungMin May 27, 2024
77ce798
test: update test code
KumJungMin May 27, 2024
56207f4
test: check application aria is same with current date
KumJungMin May 27, 2024
339a7b9
chore(changeset): revise package and add issue number
wingkwong May 27, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 2 additions & 0 deletions packages/components/calendar/src/use-calendar-picker.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -115,6 +115,8 @@ export function useCalendarPicker(props: CalendarPickerProps) {

// scroll to the selected month/year when the component is mounted/opened/closed
useEffect(() => {
jrgarciadev marked this conversation as resolved.
Show resolved Hide resolved
if (!isHeaderExpanded) return;

scrollTo(date.month, "months", false);
scrollTo(date.year, "years", false);
}, [isHeaderExpanded]);
Expand Down