Skip to content
This repository was archived by the owner on Nov 27, 2022. It is now read-only.

Commit b2ba100

Browse files
authored
Merge pull request arqex#733 from arqex/i730
Fixes time updating with controlled components
2 parents 3258157 + 4648377 commit b2ba100

File tree

5 files changed

+6
-7
lines changed

5 files changed

+6
-7
lines changed

dist/react-datetime.cjs.js

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

dist/react-datetime.cjs.js.map

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

dist/react-datetime.umd.js

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

dist/react-datetime.umd.js.map

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

src/datetime/DateTime.js

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -383,8 +383,7 @@ export default class Datetime extends React.Component {
383383
}
384384

385385
_setTime = ( type, value ) => {
386-
const state = this.state;
387-
let date = (state.selectedDate || state.viewDate).clone();
386+
let date = (this.getSelectedDate() || this.state.viewDate).clone();
388387

389388
date[ type ]( value );
390389

@@ -396,7 +395,7 @@ export default class Datetime extends React.Component {
396395
});
397396
}
398397

399-
this.props.onChange( date.clone() );
398+
this.props.onChange( date );
400399
}
401400

402401
_openCalendar = () => {

0 commit comments

Comments
 (0)