Skip to content

feat(ui-date-input): add feature to disable dates and access the input's ref#1875

Merged
balzss merged 1 commit intomasterfrom
feat/dateinput2-disabled-dates
Mar 3, 2025
Merged

feat(ui-date-input): add feature to disable dates and access the input's ref#1875
balzss merged 1 commit intomasterfrom
feat/dateinput2-disabled-dates

Conversation

@balzss
Copy link
Contributor

@balzss balzss commented Feb 20, 2025

INSTUI-4430

test plan:

  • add disabledDates prop to the component as a list of strings (['2025-02-20', '2025-02-21', '2025-02-22']) and open the calendar to check if dates are disabled
  • add disabledDate prop with an array and see if the dates are disabled ((date) => date.startsWith('2025-02'))
  • add inputRef prop as a function and see if it returns the inputref
  • call that inputref with the .focus() fn and see if it focuses the input

@balzss balzss self-assigned this Feb 20, 2025
@github-actions
Copy link

github-actions bot commented Feb 20, 2025

PR Preview Action v1.6.0
Preview removed because the pull request was closed.
2025-03-03 09:28 UTC

@balzss balzss force-pushed the feat/dateinput2-disabled-dates branch from 864b76d to db8a451 Compare February 25, 2025 09:29
@balzss balzss force-pushed the feat/dateinput2-disabled-dates branch from db8a451 to cf65997 Compare February 25, 2025 10:47
@balzss balzss marked this pull request as ready for review February 25, 2025 10:48
@balzss balzss requested review from HerrTopi, ToMESSKa, git-nandor and joyenjoyer and removed request for HerrTopi and ToMESSKa February 25, 2025 10:53
Copy link
Contributor

@joyenjoyer joyenjoyer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice work

@balzss balzss merged commit 411219e into master Mar 3, 2025
11 checks passed
@balzss balzss deleted the feat/dateinput2-disabled-dates branch March 3, 2025 09:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants