@atlaskit/datetime-picker 14.1.1 → 14.1.2

package/CHANGELOG.md

@@ -1,5 +1,12 @@
 # @atlaskit/datetime-picker
 
+## 14.1.2
+
+### Patch Changes
+
+- [`728a2eb75d298`](https://stash.atlassian.com/projects/CONFCLOUD/repos/confluence-frontend/commits/728a2eb75d298) -
+  Fix bug in handling controlled values.
+
 ## 14.1.1
 
 ### Patch Changes

package/dist/cjs/components/date-picker.js

@@ -36,7 +36,7 @@ function _isNativeReflectConstruct() { try { var t = !Boolean.prototype.valueOf.
  * @jsx jsx
  */ // eslint-disable-next-line @atlaskit/ui-styling-standard/use-compiled -- Ignored via go/DSP-18766
 var packageName = "@atlaskit/datetime-picker";
-var packageVersion = "14.1.1";
+var packageVersion = "14.1.2";
 var datePickerDefaultProps = {
   appearance: 'default',
   autoFocus: false,

package/dist/cjs/components/date-time-picker.js

@@ -32,7 +32,7 @@ function _objectSpread(e) { for (var r = 1; r < arguments.length; r++) { var t =
  * @jsx jsx
  */ // eslint-disable-next-line @atlaskit/ui-styling-standard/use-compiled -- Ignored via go/DSP-18766
 var packageName = "@atlaskit/datetime-picker";
-var packageVersion = "14.1.1";
+var packageVersion = "14.1.2";
 // Make DatePicker 50% the width of DateTimePicker
 // If rendering an icon container, shrink the TimePicker
 var datePickerContainerStyles = (0, _react2.css)({

package/dist/cjs/components/time-picker.js

@@ -25,9 +25,9 @@ var _excluded = ["styles"];
 function _getRequireWildcardCache(e) { if ("function" != typeof WeakMap) return null; var r = new WeakMap(), t = new WeakMap(); return (_getRequireWildcardCache = function _getRequireWildcardCache(e) { return e ? t : r; })(e); }
 function _interopRequireWildcard(e, r) { if (!r && e && e.__esModule) return e; if (null === e || "object" != _typeof(e) && "function" != typeof e) return { default: e }; var t = _getRequireWildcardCache(r); if (t && t.has(e)) return t.get(e); var n = { __proto__: null }, a = Object.defineProperty && Object.getOwnPropertyDescriptor; for (var u in e) if ("default" !== u && {}.hasOwnProperty.call(e, u)) { var i = a ? Object.getOwnPropertyDescriptor(e, u) : null; i && (i.get || i.set) ? Object.defineProperty(n, u, i) : n[u] = e[u]; } return n.default = e, t && t.set(e, n), n; }
 function ownKeys(e, r) { var t = Object.keys(e); if (Object.getOwnPropertySymbols) { var o = Object.getOwnPropertySymbols(e); r && (o = o.filter(function (r) { return Object.getOwnPropertyDescriptor(e, r).enumerable; })), t.push.apply(t, o); } return t; }
-function _objectSpread(e) { for (var r = 1; r < arguments.length; r++) { var t = null != arguments[r] ? arguments[r] : {}; r % 2 ? ownKeys(Object(t), !0).forEach(function (r) { (0, _defineProperty2.default)(e, r, t[r]); }) : Object.getOwnPropertyDescriptors ? Object.defineProperties(e, Object.getOwnPropertyDescriptors(t)) : ownKeys(Object(t)).forEach(function (r) { Object.defineProperty(e, r, Object.getOwnPropertyDescriptor(t, r)); }); } return e; } // eslint-disable-next-line no-restricted-imports
+function _objectSpread(e) { for (var r = 1; r < arguments.length; r++) { var t = null != arguments[r] ? arguments[r] : {}; r % 2 ? ownKeys(Object(t), !0).forEach(function (r) { (0, _defineProperty2.default)(e, r, t[r]); }) : Object.getOwnPropertyDescriptors ? Object.defineProperties(e, Object.getOwnPropertyDescriptors(t)) : ownKeys(Object(t)).forEach(function (r) { Object.defineProperty(e, r, Object.getOwnPropertyDescriptor(t, r)); }); } return e; }
 var packageName = "@atlaskit/datetime-picker";
-var packageVersion = "14.1.1";
+var packageVersion = "14.1.2";
 var menuStyles = {
   /* Need to remove default absolute positioning as that causes issues with position fixed */
   position: 'static',
@@ -120,11 +120,11 @@ var TimePicker = /*#__PURE__*/(0, _react.forwardRef)(function (_ref, ref) {
     _useState6 = (0, _slicedToArray2.default)(_useState5, 2),
     _ = _useState6[0],
     setIsFocused = _useState6[1];
-  var _useState7 = (0, _react.useState)(providedIsOpen || defaultIsOpen),
+  var _useState7 = (0, _react.useState)(defaultIsOpen),
     _useState8 = (0, _slicedToArray2.default)(_useState7, 2),
     isOpen = _useState8[0],
     setIsOpen = _useState8[1];
-  var _useState9 = (0, _react.useState)(providedValue || defaultValue),
+  var _useState9 = (0, _react.useState)(defaultValue),
     _useState10 = (0, _slicedToArray2.default)(_useState9, 2),
     value = _useState10[0],
     setValue = _useState10[1];
@@ -140,6 +140,16 @@ var TimePicker = /*#__PURE__*/(0, _react.forwardRef)(function (_ref, ref) {
     action: 'selectedTime',
     analyticsData: analyticsContext
   }, analyticsAttributes));
+  (0, _react.useEffect)(function () {
+    if (providedValue) {
+      setValue(providedValue);
+    }
+  }, [providedValue]);
+  (0, _react.useEffect)(function () {
+    if (providedIsOpen) {
+      setIsOpen(providedIsOpen);
+    }
+  }, [providedIsOpen]);
   var onChange = function onChange(newValue, action) {
     var rawValue = newValue ? newValue.value || newValue : '';
     var finalValue = rawValue.toString();
@@ -322,7 +332,10 @@ var TimePicker = /*#__PURE__*/(0, _react.forwardRef)(function (_ref, ref) {
     placeholder: placeholder || l10n.formatTime(_internal.placeholderDatetime),
     styles: mergedStyles,
     value: initialValue,
-    spacing: spacing,
+    spacing: spacing
+    // We need this to get things to work, even though it's not supported.
+    // @ts-ignore
+    ,
     fixedLayerRef: containerRef,
     isInvalid: isInvalid,
     testId: testId

package/dist/es2019/components/date-picker.js

@@ -19,7 +19,7 @@ import { Menu } from '../internal/menu';
 import { getSafeCalendarValue, getShortISOString } from '../internal/parse-date';
 import { makeSingleValue } from '../internal/single-value';
 const packageName = "@atlaskit/datetime-picker";
-const packageVersion = "14.1.1";
+const packageVersion = "14.1.2";
 const datePickerDefaultProps = {
   appearance: 'default',
   autoFocus: false,

package/dist/es2019/components/date-time-picker.js

@@ -18,7 +18,7 @@ import { convertTokens } from '../internal/parse-tokens';
 import DatePicker from './date-picker';
 import TimePicker from './time-picker';
 const packageName = "@atlaskit/datetime-picker";
-const packageVersion = "14.1.1";
+const packageVersion = "14.1.2";
 // Make DatePicker 50% the width of DateTimePicker
 // If rendering an icon container, shrink the TimePicker
 const datePickerContainerStyles = css({

package/dist/es2019/components/time-picker.js

@@ -1,7 +1,5 @@
 import _extends from "@babel/runtime/helpers/extends";
-import React, { forwardRef, useReducer, useState } from 'react';
-
-// eslint-disable-next-line no-restricted-imports
+import React, { forwardRef, useEffect, useReducer, useState } from 'react';
 import { format, isValid } from 'date-fns';
 import { usePlatformLeafEventHandler } from '@atlaskit/analytics-next';
 import __noop from '@atlaskit/ds-lib/noop';
@@ -13,7 +11,7 @@ import parseTime from '../internal/parse-time';
 import { convertTokens } from '../internal/parse-tokens';
 import { makeSingleValue } from '../internal/single-value';
 const packageName = "@atlaskit/datetime-picker";
-const packageVersion = "14.1.1";
+const packageVersion = "14.1.2";
 const menuStyles = {
   /* Need to remove default absolute positioning as that causes issues with position fixed */
   position: 'static',
@@ -76,8 +74,8 @@ const TimePicker = /*#__PURE__*/forwardRef(({
   const [clearingFromIcon, setClearingFromIcon] = useState(false);
   // TODO: Remove isFocused? Does it do anything?
   const [_, setIsFocused] = useState(false);
-  const [isOpen, setIsOpen] = useState(providedIsOpen || defaultIsOpen);
-  const [value, setValue] = useState(providedValue || defaultValue);
+  const [isOpen, setIsOpen] = useState(defaultIsOpen);
+  const [value, setValue] = useState(defaultValue);
 
   // Hack to force update: https://legacy.reactjs.org/docs/hooks-faq.html#is-there-something-like-forceupdate
   const [, forceUpdate] = useReducer(x => x + 1, 0);
@@ -87,6 +85,16 @@ const TimePicker = /*#__PURE__*/forwardRef(({
     analyticsData: analyticsContext,
     ...analyticsAttributes
   });
+  useEffect(() => {
+    if (providedValue) {
+      setValue(providedValue);
+    }
+  }, [providedValue]);
+  useEffect(() => {
+    if (providedIsOpen) {
+      setIsOpen(providedIsOpen);
+    }
+  }, [providedIsOpen]);
   const onChange = (newValue, action) => {
     const rawValue = newValue ? newValue.value || newValue : '';
     const finalValue = rawValue.toString();
@@ -272,7 +280,10 @@ const TimePicker = /*#__PURE__*/forwardRef(({
     placeholder: placeholder || l10n.formatTime(placeholderDatetime),
     styles: mergedStyles,
     value: initialValue,
-    spacing: spacing,
+    spacing: spacing
+    // We need this to get things to work, even though it's not supported.
+    // @ts-ignore
+    ,
     fixedLayerRef: containerRef,
     isInvalid: isInvalid,
     testId: testId

package/dist/esm/components/date-picker.js

@@ -29,7 +29,7 @@ import { Menu } from '../internal/menu';
 import { getSafeCalendarValue, getShortISOString } from '../internal/parse-date';
 import { makeSingleValue } from '../internal/single-value';
 var packageName = "@atlaskit/datetime-picker";
-var packageVersion = "14.1.1";
+var packageVersion = "14.1.2";
 var datePickerDefaultProps = {
   appearance: 'default',
   autoFocus: false,

package/dist/esm/components/date-time-picker.js

@@ -28,7 +28,7 @@ import { convertTokens } from '../internal/parse-tokens';
 import DatePicker from './date-picker';
 import TimePicker from './time-picker';
 var packageName = "@atlaskit/datetime-picker";
-var packageVersion = "14.1.1";
+var packageVersion = "14.1.2";
 // Make DatePicker 50% the width of DateTimePicker
 // If rendering an icon container, shrink the TimePicker
 var datePickerContainerStyles = css({

package/dist/esm/components/time-picker.js

@@ -5,9 +5,7 @@ import _slicedToArray from "@babel/runtime/helpers/slicedToArray";
 var _excluded = ["styles"];
 function ownKeys(e, r) { var t = Object.keys(e); if (Object.getOwnPropertySymbols) { var o = Object.getOwnPropertySymbols(e); r && (o = o.filter(function (r) { return Object.getOwnPropertyDescriptor(e, r).enumerable; })), t.push.apply(t, o); } return t; }
 function _objectSpread(e) { for (var r = 1; r < arguments.length; r++) { var t = null != arguments[r] ? arguments[r] : {}; r % 2 ? ownKeys(Object(t), !0).forEach(function (r) { _defineProperty(e, r, t[r]); }) : Object.getOwnPropertyDescriptors ? Object.defineProperties(e, Object.getOwnPropertyDescriptors(t)) : ownKeys(Object(t)).forEach(function (r) { Object.defineProperty(e, r, Object.getOwnPropertyDescriptor(t, r)); }); } return e; }
-import React, { forwardRef, useReducer, useState } from 'react';
-
-// eslint-disable-next-line no-restricted-imports
+import React, { forwardRef, useEffect, useReducer, useState } from 'react';
 import { format, isValid } from 'date-fns';
 import { usePlatformLeafEventHandler } from '@atlaskit/analytics-next';
 import __noop from '@atlaskit/ds-lib/noop';
@@ -19,7 +17,7 @@ import parseTime from '../internal/parse-time';
 import { convertTokens } from '../internal/parse-tokens';
 import { makeSingleValue } from '../internal/single-value';
 var packageName = "@atlaskit/datetime-picker";
-var packageVersion = "14.1.1";
+var packageVersion = "14.1.2";
 var menuStyles = {
   /* Need to remove default absolute positioning as that causes issues with position fixed */
   position: 'static',
@@ -112,11 +110,11 @@ var TimePicker = /*#__PURE__*/forwardRef(function (_ref, ref) {
     _useState6 = _slicedToArray(_useState5, 2),
     _ = _useState6[0],
     setIsFocused = _useState6[1];
-  var _useState7 = useState(providedIsOpen || defaultIsOpen),
+  var _useState7 = useState(defaultIsOpen),
     _useState8 = _slicedToArray(_useState7, 2),
     isOpen = _useState8[0],
     setIsOpen = _useState8[1];
-  var _useState9 = useState(providedValue || defaultValue),
+  var _useState9 = useState(defaultValue),
     _useState10 = _slicedToArray(_useState9, 2),
     value = _useState10[0],
     setValue = _useState10[1];
@@ -132,6 +130,16 @@ var TimePicker = /*#__PURE__*/forwardRef(function (_ref, ref) {
     action: 'selectedTime',
     analyticsData: analyticsContext
   }, analyticsAttributes));
+  useEffect(function () {
+    if (providedValue) {
+      setValue(providedValue);
+    }
+  }, [providedValue]);
+  useEffect(function () {
+    if (providedIsOpen) {
+      setIsOpen(providedIsOpen);
+    }
+  }, [providedIsOpen]);
   var onChange = function onChange(newValue, action) {
     var rawValue = newValue ? newValue.value || newValue : '';
     var finalValue = rawValue.toString();
@@ -314,7 +322,10 @@ var TimePicker = /*#__PURE__*/forwardRef(function (_ref, ref) {
     placeholder: placeholder || l10n.formatTime(placeholderDatetime),
     styles: mergedStyles,
     value: initialValue,
-    spacing: spacing,
+    spacing: spacing
+    // We need this to get things to work, even though it's not supported.
+    // @ts-ignore
+    ,
     fixedLayerRef: containerRef,
     isInvalid: isInvalid,
     testId: testId

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@atlaskit/datetime-picker",
-	"version": "14.1.1",
+	"version": "14.1.2",
 	"description": "A date time picker allows the user to select an associated date and time.",
 	"publishConfig": {
 		"registry": "https://registry.npmjs.org/"
@@ -44,7 +44,7 @@
 		"@atlaskit/popper": "^6.3.0",
 		"@atlaskit/select": "^17.19.0",
 		"@atlaskit/theme": "^13.0.0",
-		"@atlaskit/tokens": "^1.60.0",
+		"@atlaskit/tokens": "^1.61.0",
 		"@babel/runtime": "^7.0.0",
 		"@emotion/react": "^11.7.1",
 		"date-fns": "^2.17.0"