palantir/blueprint

Modifying the time doesn't show up on the timepicker in DateRangePicker

Open

#5502 aperta il 22 ago 2022

Vedi su GitHub
 (1 commento) (0 reazioni) (0 assegnatari)TypeScript (2167 fork)batch import
P2Package: datetimeType: bughelp wanted

Metriche repository

Star
 (20.263 star)
Metriche merge PR
 (Merge medio 43g 10h) (27 PR mergiate in 30 g)

Descrizione

Environment

  • Package version(s): 4.3.4
  • Operating System: MacOS Monterey
  • Browser name and version: Firefox 103.0.2

Code Sandbox

Link to a minimal repro: https://codesandbox.io/s/blueprint-sandbox-forked-gb14b7

Steps to reproduce

  1. Select two dates in the daterangepicker

Actual behavior

Modifying the endTime (from 00:00 to 23:59) doesn't show the modified end time in the time picker. (I modify the time in the onChange)

Expected behavior

When I change the time on the endDate value, it should update on the DateRangePicker end time

Possible solution

It looks like it doesn't update the startTime or endTime in the code when the props change, only on the construction. Modify it so it does do that.

Guida contributor

Direzione di ricerca
Ispeziona la logica di inizializzazione e aggiornamento dello stato del componente DateRangePicker per startTime e endTime. Il problema suggerisce che la modifica di endTime tramite onChange non viene riflessa nel selettore dell'ora. Controlla il costruttore del componente o getDerivedStateFromProps per vedere se startTime/endTime sono impostati solo al montaggio. Una possibile soluzione è utilizzare componentDidUpdate o useEffect per sincronizzare il valore controllato con lo stato interno quando le prop cambiano.
Tech stack
typescriptreact
Dominio
frontend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
ReactGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70