palantir/blueprint

DateInput closeOnUserDifferentMonthSelection

Open

#5,053 opened on Dec 1, 2021

View on GitHub
 (3 comments) (0 reactions) (0 assignees)TypeScript (2,167 forks)batch import
P2Package: datetimeType: feature requesthelp wanted

Repository metrics

Stars
 (20,263 stars)
PR merge metrics
 (Avg merge 43d 10h) (27 merged PRs in 30d)

Description

As per the following code:

https://github.com/palantir/blueprint/blob/e148da20dc778e3b177062f29a1f5930d7a0d8ac/packages/datetime/src/dateInput.tsx#L293-L299

I think this is unintuitive and we should have an option to close the popup when user select a date, even if its in a different month.

I am willing to raise a PR for v4 to add a closeOnUserDifferentMonthSelection, defaulting it to false. And if set to true, and a user selects for example last day of previous month, for it to close (as if they selected a day in the same month). Happy for any other name suggestion.

More often than not, we want to select a single business day. Usually Today, sometimes yesterday. When selecting yesterday it is a 2 click process, where any day in the same month (so for rest of month), it is a single click to select date.

Contributor guide

Research direction
Examine the DateInput component's close logic at the linked code (lines 293-299) to understand the current behavior and identify where a new prop 'closeOnUserDifferentMonthSelection' can be added to close the popup when a date in a different month is selected.
Tech stack
typescriptreact
Domain
frontend
Issue type
Feature
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
3
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-2 days
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Active
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Clear
Prerequisites
ReactTypeScriptBlueprint
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
70