palantir / blueprint

A React-based UI toolkit for the web
https://blueprintjs.com/
Apache License 2.0
20.51k stars 2.15k forks source link

Respect showOutsideDays prop for single month date range #6862

Open evansjohnson opened 6 days ago

evansjohnson commented 6 days ago

Fixes https://github.com/palantir/blueprint/issues/4373

Checklist

Changes proposed in this pull request:

When a DateRangePicker3 is limited to a single month, respect the showOutsideDays passed on dayPickerProps. Per https://github.com/palantir/blueprint/pull/586/files#r98813760 it was explicitly decided that showing outside days when multiple months are shown is not intended, and I agree that it still leads to some weird questions, but for a single month these issues do not exist.

Reviewers should focus on:

Alternate option

Simply document that showOutsideDays is never respected for DateRangePicker3

Screenshot

Screenshot 2024-06-24 at 4 06 48 PM Screenshot 2024-06-24 at 4 06 43 PM

svc-palantir-github commented 3 days ago

assert warn

Build artifact links for this commit: documentation | landing | table | demo

This is an automated comment from the deploy-preview CircleCI job.