feat: add disabled dates API to date and date-time pickers#9691
Open
Artur- wants to merge 3 commits into
Open
Conversation
Add server-side disabled-dates support: setDisabledDates for a fixed list, setDisabledWeekdays, and a DisabledDatesProvider callback that is called per date. The connector composes these into the web component's disabledDatesProvider (static rules client-side, the provider via a @ClientCallable range request evaluated per date on the server) and the validator rejects disabled values. DateTimePicker delegates the same API to its date part. Part of #2867
Member
Author
com.vaadin.flow.component.datepicker.DatePicker// Added
public void setDisabledDates(Collection<LocalDate> dates) // fixed set of dates that cannot be selected
public Set<LocalDate> getDisabledDates()
public void setDisabledWeekdays(Set<DayOfWeek> weekdays) // e.g. disable weekends
public Set<DayOfWeek> getDisabledWeekdays()
public void setDisabledDatesProvider(DatePicker.DisabledDatesProvider provider) // per-date, server-evaluated
public DatePicker.DisabledDatesProvider getDisabledDatesProvider()com.vaadin.flow.component.datepicker.DatePicker.DisabledDatesProvider// Added
@FunctionalInterface
public interface DisabledDatesProvider extends Serializable
boolean isDisabled(LocalDate date) // true to disable the given datecom.vaadin.flow.component.datepicker.DatePicker.DatePickerI18n// Added
public String getDateDisabledErrorMessage()
public DatePicker.DatePickerI18n setDateDisabledErrorMessage(String errorMessage)com.vaadin.flow.component.datetimepicker.DateTimePicker// Added
public void setDisabledDates(Collection<LocalDate> dates) // delegates to the date part
public Set<LocalDate> getDisabledDates()
public void setDisabledWeekdays(Set<DayOfWeek> weekdays)
public Set<DayOfWeek> getDisabledWeekdays()
public void setDisabledDatesProvider(DatePicker.DisabledDatesProvider provider)
public DatePicker.DisabledDatesProvider getDisabledDatesProvider() |
11 tasks
The while loop mutated the loop variable via date.setDate(), which Sonar (rule S2189) reports as a potential infinite loop. Use a counted for loop instead.
Artur-
changed the title
Artur-
changed the title
Point every @vaadin NpmPackage annotation at the feature snapshot 25.3.0-dev.3a3c2d7d2a (dist-tag dev-disable-dates) to test the flow integration against the disabled dates web component changes.
vaadin-review-bot
previously requested changes
Jul 8, 2026
vaadin-review-bot
left a comment
vaadin-review-bot
left a comment
Collaborator
There was a problem hiding this comment.
This PR updates the @NpmPackage version. Because the default value of vaadin.npm.minimumFrontendPackageAgeDays is 1, I'm blocking it for now to avoid producing a failed snapshot for other projects.
No action needed — this is fully automated:
- I posted this block automatically when CI detected the
@NpmPackageversion bump. - 24 hours after this message, I will automatically dismiss this review, approve the PR, and enable auto-merge (squash) so it merges as soon as all required checks pass.
What affects the 24h timer:
- New commits / pushes do not reset it — the countdown is anchored to this message, so it keeps running no matter how many times you push.
- The timer only restarts if this block review is manually dismissed and a later CI run detects an
@NpmPackagebump again (that posts a fresh block). - If the PR is merged or closed before the 24 hours elapse, the auto-unblock simply does nothing.
|
vaadin-review-bot
dismissed
their stale review
July 9, 2026 09:58
24 hours have passed since the @NpmPackage block. Auto-unblocking and approving.
vaadin-review-bot
approved these changes
Jul 9, 2026
vaadin-review-bot
left a comment
vaadin-review-bot
left a comment
Collaborator
There was a problem hiding this comment.
Automatically approved 24 hours after the @NpmPackage version bump block.
vaadin-review-bot
enabled auto-merge
July 9, 2026 09:58
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Add server-side disabled-dates support: setDisabledDates for a fixed list, setDisabledWeekdays, and a DisabledDatesProvider callback that is called per date. The connector composes these into the web component's disabledDatesProvider (static rules client-side, the provider via a @ClientCallable range request evaluated per date on the server) and the validator rejects disabled values. DateTimePicker delegates the same API to its date part.
Part of vaadin/platform#2867