@patternfly/react-core 5.0.0-alpha.50 → 5.0.0-alpha.52

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@patternfly/react-core",
-  "version": "5.0.0-alpha.50",
+  "version": "5.0.0-alpha.52",
   "description": "This library provides a set of common React components for use with the PatternFly reference implementation.",
   "main": "dist/js/index.js",
   "module": "dist/esm/index.js",
@@ -45,15 +45,15 @@
     "subpaths": "node scripts/createSubpaths.js"
   },
   "dependencies": {
-    "@patternfly/react-icons": "^5.0.0-alpha.7",
-    "@patternfly/react-styles": "^5.0.0-alpha.5",
-    "@patternfly/react-tokens": "^5.0.0-alpha.5",
+    "@patternfly/react-icons": "^5.0.0-alpha.8",
+    "@patternfly/react-styles": "^5.0.0-alpha.6",
+    "@patternfly/react-tokens": "^5.0.0-alpha.6",
     "focus-trap": "7.4.0",
     "react-dropzone": "^14.2.3",
     "tslib": "^2.5.0"
   },
   "devDependencies": {
-    "@patternfly/patternfly": "5.0.0-alpha.33",
+    "@patternfly/patternfly": "5.0.0-alpha.37",
     "@rollup/plugin-commonjs": "^24.0.0",
     "@rollup/plugin-node-resolve": "^15.0.1",
     "@rollup/plugin-replace": "^5.0.2",
@@ -70,5 +70,5 @@
     "react": "^17 || ^18",
     "react-dom": "^17 || ^18"
   },
-  "gitHead": "c61ae62dda93e9b27afc23874d7219d10eac353d"
+  "gitHead": "5eea8522248a6f8caa1d7cb27f5f5b95ca823b7b"
 }

package/scripts/copyStyles.js

@@ -24,7 +24,7 @@ copySync(join(pfDir, 'assets/fonts'), join(stylesDir, 'assets/fonts'), {
 // Copy css
 const baseCssFiles = {
   'base.css': 'patternfly-base.css',
-  'base-no-reset.css': 'patternfly-base-no-reset.css'
+  'base-no-reset.css': 'patternfly-base-no-globals.css'
 };
 
 for (const [targetCss, baseCss] of Object.entries(baseCssFiles)) {

package/src/components/BackgroundImage/BackgroundImage.tsx

@@ -1,10 +1,12 @@
 import * as React from 'react';
-import cssVar from '@patternfly/react-tokens/dist/esm/c_background_image_BackgroundImage';
-import cssVarName2x from '@patternfly/react-tokens/dist/esm/c_background_image_BackgroundImage_2x';
-import cssVarNameSm from '@patternfly/react-tokens/dist/esm/c_background_image_BackgroundImage_sm';
-import cssVarNameSm2x from '@patternfly/react-tokens/dist/esm/c_background_image_BackgroundImage_sm_2x';
-import cssVarNameLg from '@patternfly/react-tokens/dist/esm/c_background_image_BackgroundImage_lg';
-import cssVarNameFilter from '@patternfly/react-tokens/dist/esm/c_background_image_Filter';
+// Commented imports that no longer exist. This component will be updated with issue #8452 resolution
+
+// import cssVar from '@patternfly/react-tokens/dist/esm/c_background_image_BackgroundImage';
+// import cssVarName2x from '@patternfly/react-tokens/dist/esm/c_background_image_BackgroundImage_2x';
+// import cssVarNameSm from '@patternfly/react-tokens/dist/esm/c_background_image_BackgroundImage_sm';
+// import cssVarNameSm2x from '@patternfly/react-tokens/dist/esm/c_background_image_BackgroundImage_sm_2x';
+// import cssVarNameLg from '@patternfly/react-tokens/dist/esm/c_background_image_BackgroundImage_lg';
+// import cssVarNameFilter from '@patternfly/react-tokens/dist/esm/c_background_image_Filter';
 import { css } from '@patternfly/react-styles';
 import styles from '@patternfly/react-styles/css/components/BackgroundImage/background-image';
 
@@ -16,17 +18,17 @@ export interface BackgroundImageSrcMap {
   lg: string;
 }
 
-const defaultFilter = (
-  <filter>
-    <feColorMatrix type="matrix" values="1 0 0 0 0 1 0 0 0 0 1 0 0 0 0 0 0 0 1 0"></feColorMatrix>
-    <feComponentTransfer colorInterpolationFilters="sRGB" result="duotone">
-      <feFuncR type="table" tableValues="0.086274509803922 0.43921568627451"></feFuncR>
-      <feFuncG type="table" tableValues="0.086274509803922 0.43921568627451"></feFuncG>
-      <feFuncB type="table" tableValues="0.086274509803922 0.43921568627451"></feFuncB>
-      <feFuncA type="table" tableValues="0 1"></feFuncA>
-    </feComponentTransfer>
-  </filter>
-);
+// const defaultFilter = (
+//   <filter>
+//     <feColorMatrix type="matrix" values="1 0 0 0 0 1 0 0 0 0 1 0 0 0 0 0 0 0 1 0"></feColorMatrix>
+//     <feComponentTransfer colorInterpolationFilters="sRGB" result="duotone">
+//       <feFuncR type="table" tableValues="0.086274509803922 0.43921568627451"></feFuncR>
+//       <feFuncG type="table" tableValues="0.086274509803922 0.43921568627451"></feFuncG>
+//       <feFuncB type="table" tableValues="0.086274509803922 0.43921568627451"></feFuncB>
+//       <feFuncA type="table" tableValues="0 1"></feFuncA>
+//     </feComponentTransfer>
+//   </filter>
+// );
 
 export interface BackgroundImageProps extends Omit<React.HTMLProps<HTMLDivElement>, 'src'> {
   /** Additional classes added to the background. */
@@ -37,41 +39,42 @@ export interface BackgroundImageProps extends Omit<React.HTMLProps<HTMLDivElemen
   src: string | BackgroundImageSrcMap;
 }
 
-let filterCounter = 0;
+// let filterCounter = 0;
 
 export const BackgroundImage: React.FunctionComponent<BackgroundImageProps> = ({
   className,
-  src,
-  filter = defaultFilter,
+  // src,
+  // filter = defaultFilter,
   ...props
-}: BackgroundImageProps) => {
-  const getUrlValue = (size: keyof BackgroundImageSrcMap) => {
-    if (typeof src === 'string') {
-      return `url(${src})`;
-    } else if (typeof src === 'object') {
-      return `url(${src[size]})`;
-    }
+}: BackgroundImageProps) => 
+  // const getUrlValue = (size: keyof BackgroundImageSrcMap) => {
+  //   if (typeof src === 'string') {
+  //     return `url(${src})`;
+  //   } else if (typeof src === 'object') {
+  //     return `url(${src[size]})`;
+  //   }
 
-    return '';
-  };
+  //   return '';
+  // };
 
-  const filterNum = React.useMemo(() => filterCounter++, []);
-  const filterId = `patternfly-background-image-filter-overlay${filterNum}`;
-  const style = {
-    [cssVar.name]: getUrlValue('xs'),
-    [cssVarName2x.name]: getUrlValue('xs2x'),
-    [cssVarNameSm.name]: getUrlValue('sm'),
-    [cssVarNameSm2x.name]: getUrlValue('sm2x'),
-    [cssVarNameLg.name]: getUrlValue('lg'),
-    [cssVarNameFilter.name]: `url(#${filterId})`
-  } as React.CSSProperties;
+  // const filterNum = React.useMemo(() => filterCounter++, []);
+  // const filterId = `patternfly-background-image-filter-overlay${filterNum}`;
+  // const style = {
+  //   [cssVar.name]: getUrlValue('xs'),
+  //   [cssVarName2x.name]: getUrlValue('xs2x'),
+  //   [cssVarNameSm.name]: getUrlValue('sm'),
+  //   [cssVarNameSm2x.name]: getUrlValue('sm2x'),
+  //   [cssVarNameLg.name]: getUrlValue('lg'),
+  // [cssVarNameFilter.name]: `url(#${filterId})`
+  // } as React.CSSProperties;
 
-  return (
-    <div className={css(styles.backgroundImage, className)} style={style} {...props}>
+   (
+    // TODO: removed inline style add it  back with issue #8452 resolution. Also commented out the "React.cloneElement(filter, { id: filterId })} "
+    <div className={css(styles.backgroundImage, className)} {...props}>
       <svg xmlns="http://www.w3.org/2000/svg" className="pf-c-background-image__filter" width="0" height="0">
-        {React.cloneElement(filter, { id: filterId })}
+        {/* {React.cloneElement(filter, { id: filterId })} */}
       </svg>
     </div>
-  );
-};
+  )
+;
 BackgroundImage.displayName = 'BackgroundImage';

package/src/components/BackgroundImage/__tests__/__snapshots__/BackgroundImage.test.tsx.snap

@@ -4,44 +4,14 @@ exports[`BackgroundImage 1`] = `
 <DocumentFragment>
   <div
     class="pf-c-background-image"
-    style="--pf-c-background-image--BackgroundImage: url(/assets/images/pfbg_576.jpg); --pf-c-background-image--BackgroundImage-2x: url(/assets/images/pfbg_576@2x.jpg); --pf-c-background-image--BackgroundImage--sm: url(/assets/images/pfbg_768.jpg); --pf-c-background-image--BackgroundImage--sm-2x: url(/assets/images/pfbg_768@2x.jpg); --pf-c-background-image--BackgroundImage--lg: url(/assets/images/pfbg_1200.jpg); --pf-c-background-image--Filter: url(#patternfly-background-image-filter-overlay0);"
+    src="[object Object]"
   >
     <svg
       class="pf-c-background-image__filter"
       height="0"
       width="0"
       xmlns="http://www.w3.org/2000/svg"
-    >
-      <filter
-        id="patternfly-background-image-filter-overlay0"
-      >
-        <fecolormatrix
-          type="matrix"
-          values="1 0 0 0 0 1 0 0 0 0 1 0 0 0 0 0 0 0 1 0"
-        />
-        <fecomponenttransfer
-          color-interpolation-filters="sRGB"
-          result="duotone"
-        >
-          <fefuncr
-            tableValues="0.086274509803922 0.43921568627451"
-            type="table"
-          />
-          <fefuncg
-            tableValues="0.086274509803922 0.43921568627451"
-            type="table"
-          />
-          <fefuncb
-            tableValues="0.086274509803922 0.43921568627451"
-            type="table"
-          />
-          <fefunca
-            tableValues="0 1"
-            type="table"
-          />
-        </fecomponenttransfer>
-      </filter>
-    </svg>
+    />
   </div>
 </DocumentFragment>
 `;
@@ -50,44 +20,14 @@ exports[`allows passing in a single string as the image src 1`] = `
 <DocumentFragment>
   <div
     class="pf-c-background-image"
-    style="--pf-c-background-image--BackgroundImage: url(/assets/images/pfbg_1200.jpg); --pf-c-background-image--BackgroundImage-2x: url(/assets/images/pfbg_1200.jpg); --pf-c-background-image--BackgroundImage--sm: url(/assets/images/pfbg_1200.jpg); --pf-c-background-image--BackgroundImage--sm-2x: url(/assets/images/pfbg_1200.jpg); --pf-c-background-image--BackgroundImage--lg: url(/assets/images/pfbg_1200.jpg); --pf-c-background-image--Filter: url(#patternfly-background-image-filter-overlay1);"
+    src="/assets/images/pfbg_1200.jpg"
   >
     <svg
       class="pf-c-background-image__filter"
       height="0"
       width="0"
       xmlns="http://www.w3.org/2000/svg"
-    >
-      <filter
-        id="patternfly-background-image-filter-overlay1"
-      >
-        <fecolormatrix
-          type="matrix"
-          values="1 0 0 0 0 1 0 0 0 0 1 0 0 0 0 0 0 0 1 0"
-        />
-        <fecomponenttransfer
-          color-interpolation-filters="sRGB"
-          result="duotone"
-        >
-          <fefuncr
-            tableValues="0.086274509803922 0.43921568627451"
-            type="table"
-          />
-          <fefuncg
-            tableValues="0.086274509803922 0.43921568627451"
-            type="table"
-          />
-          <fefuncb
-            tableValues="0.086274509803922 0.43921568627451"
-            type="table"
-          />
-          <fefunca
-            tableValues="0 1"
-            type="table"
-          />
-        </fecomponenttransfer>
-      </filter>
-    </svg>
+    />
   </div>
 </DocumentFragment>
 `;

package/src/components/Icon/Icon.tsx

@@ -17,7 +17,7 @@ export interface IconComponentProps extends Omit<React.HTMLProps<HTMLSpanElement
   /** Size of progress icon. Overrides the icon size set by the size property. */
   progressIconSize?: 'sm' | 'md' | 'lg' | 'xl';
   /** Status color of the icon */
-  status?: 'default' | 'info' | 'success' | 'warning' | 'danger';
+  status?: 'custom' | 'info' | 'success' | 'warning' | 'danger';
   /** Indicates the icon is inline and should inherit the text font size and color. Overriden by size and iconSize properties. */
   isInline?: boolean;
   /** Indicates the icon is in progress. Setting this property to true will swap the icon with the progressIcon. */

package/src/components/Icon/__tests__/Icon.test.tsx

@@ -109,10 +109,10 @@ test('check icon without size', () => {
   expect(Array.from(iconContainer?.classList || []).some(c => /pf-m-*/.test(c))); // Check no modifier classes have been added
 });
 
-Object.values(['default', 'info', 'success', 'warning', 'danger']).forEach(status => {
+Object.values(['custom', 'info', 'success', 'warning', 'danger']).forEach(status => {
   test(`sets status modifier successfully - ${status}`, () => {
     render(
-      <Icon status={status as 'default' | 'info' | 'success' | 'warning' | 'danger'} title={`${status}-icon`}>
+      <Icon status={status as 'custom' | 'info' | 'success' | 'warning' | 'danger'} title={`${status}-icon`}>
         <CheckIcon />
       </Icon>
     );

package/src/components/Icon/examples/IconStatus.tsx

@@ -20,7 +20,7 @@ export const IconStatus: React.FunctionComponent = () => (
     <Icon status="info">
       <InfoCircleIcon />
     </Icon>
-    <Icon status="default">
+    <Icon status="custom">
       <BellIcon />
     </Icon>
   </React.Fragment>

package/src/components/LoginPage/__tests__/__snapshots__/LoginPage.test.tsx.snap

@@ -5,44 +5,14 @@ exports[`check loginpage example against snapshot 1`] = `
   <div
     alt="Pf-background"
     class="pf-c-background-image"
-    style="--pf-c-background-image--BackgroundImage: url(Background src); --pf-c-background-image--BackgroundImage-2x: url(Background src); --pf-c-background-image--BackgroundImage--sm: url(Background src); --pf-c-background-image--BackgroundImage--sm-2x: url(Background src); --pf-c-background-image--BackgroundImage--lg: url(Background src); --pf-c-background-image--Filter: url(#patternfly-background-image-filter-overlay0);"
+    src="Background src"
   >
     <svg
       class="pf-c-background-image__filter"
       height="0"
       width="0"
       xmlns="http://www.w3.org/2000/svg"
-    >
-      <filter
-        id="patternfly-background-image-filter-overlay0"
-      >
-        <fecolormatrix
-          type="matrix"
-          values="1 0 0 0 0 1 0 0 0 0 1 0 0 0 0 0 0 0 1 0"
-        />
-        <fecomponenttransfer
-          color-interpolation-filters="sRGB"
-          result="duotone"
-        >
-          <fefuncr
-            tableValues="0.086274509803922 0.43921568627451"
-            type="table"
-          />
-          <fefuncg
-            tableValues="0.086274509803922 0.43921568627451"
-            type="table"
-          />
-          <fefuncb
-            tableValues="0.086274509803922 0.43921568627451"
-            type="table"
-          />
-          <fefunca
-            tableValues="0 1"
-            type="table"
-          />
-        </fecomponenttransfer>
-      </filter>
-    </svg>
+    />
   </div>
   <div
     class="pf-c-login"

package/src/components/Modal/ModalContent.tsx

@@ -177,7 +177,7 @@ export const ModalContent: React.FunctionComponent<ModalContentProps> = ({
       className={css(
         className,
         isVariantIcon(titleIconVariant) &&
-          modalStyles.modifiers[titleIconVariant as 'success' | 'warning' | 'info' | 'danger' | 'default']
+          modalStyles.modifiers[titleIconVariant as 'success' | 'warning' | 'info' | 'danger' | 'custom']
       )}
       variant={variant}
       position={position}

package/src/components/NotificationDrawer/NotificationDrawerListItem.tsx

@@ -16,7 +16,7 @@ export interface NotificationDrawerListItemProps extends React.HTMLProps<HTMLLIE
   /**  Tab index for the list item */
   tabIndex?: number;
   /**  Variant indicates the severity level */
-  variant?: 'default' | 'success' | 'danger' | 'warning' | 'info';
+  variant?: 'custom' | 'success' | 'danger' | 'warning' | 'info';
 }
 
 export const NotificationDrawerListItem: React.FunctionComponent<NotificationDrawerListItemProps> = ({
@@ -27,7 +27,7 @@ export const NotificationDrawerListItem: React.FunctionComponent<NotificationDra
   // eslint-disable-next-line @typescript-eslint/no-unused-vars
   onClick = (event: React.MouseEvent) => undefined as any,
   tabIndex = 0,
-  variant = 'default',
+  variant = 'custom',
   ...props
 }: NotificationDrawerListItemProps) => {
   const onKeyDown = (event: React.KeyboardEvent) => {

package/src/components/NotificationDrawer/__tests__/__snapshots__/NotificationDrawerListItem.test.tsx.snap

@@ -3,7 +3,7 @@
 exports[`NotificationDrawerListItem renders with PatternFly Core styles 1`] = `
 <DocumentFragment>
   <li
-    class="pf-c-notification-drawer__list-item pf-m-hoverable pf-m-default"
+    class="pf-c-notification-drawer__list-item pf-m-hoverable pf-m-custom"
     tabindex="0"
   />
 </DocumentFragment>

package/src/components/Popover/Popover.tsx

@@ -210,7 +210,7 @@ export interface PopoverProps {
 }
 
 const alertStyle = {
-  default: styles.modifiers.default,
+  default: styles.modifiers.custom,
   info: styles.modifiers.info,
   success: styles.modifiers.success,
   warning: styles.modifiers.warning,

package/src/components/TimePicker/TimePicker.tsx

@@ -373,9 +373,22 @@ export class TimePicker extends React.Component<TimePickerProps, TimePickerState
   onToggle = (isOpen: boolean) => {
     // on close, parse and validate input
     this.setState(prevState => {
-      const { timeRegex, isInvalid } = prevState;
-      const { delimiter, is24Hour, includeSeconds } = this.props;
-      const time = parseTime(prevState.timeState, timeRegex, delimiter, !is24Hour, includeSeconds);
+      const { timeRegex, isInvalid, timeState } = prevState;
+      const { delimiter, is24Hour, includeSeconds, onChange } = this.props;
+      const time = parseTime(timeState, timeRegex, delimiter, !is24Hour, includeSeconds);
+
+      // Call onChange when Enter is pressed in input and timeoption does not exist in menu
+      if (onChange && !isOpen && time !== timeState) {
+        onChange(
+          null,
+          time,
+          getHours(time, timeRegex),
+          getMinutes(time, timeRegex),
+          getSeconds(time, timeRegex),
+          this.isValid(time)
+        );
+      }
+
       return {
         isTimeOptionsOpen: isOpen,
         timeState: time,
@@ -414,7 +427,6 @@ export class TimePicker extends React.Component<TimePickerProps, TimePickerState
   onInputChange = (newTime: string, event: React.FormEvent<HTMLInputElement>) => {
     const { onChange } = this.props;
     const { timeRegex } = this.state;
-
     if (onChange) {
       onChange(
         event,

package/src/components/TimePicker/__tests__/TimePicker.test.tsx

@@ -86,6 +86,28 @@ describe('TimePicker', () => {
         expects: { hour: 23, minutes: 59, seconds: null }
       });
     });
+
+    // Disabling because this test does not work on CI
+    xtest('should call onChange when pressing Enter', async () => {
+      const onChange = jest.fn();
+      const user = userEvent.setup();
+
+      render(<TimePicker onChange={onChange} aria-label="time picker" />);
+
+      // Take into account timezones when tests are ran
+      const isPM = new Date().getHours() > 12;
+
+      await user.type(screen.getByLabelText('time picker'), `11:11`);
+      await user.keyboard('[Enter]');
+      expect(onChange).toHaveBeenLastCalledWith(
+        expect.any(Object),
+        `11:11 ${isPM ? 'PM' : 'AM'}`,
+        isPM ? 23 : 11,
+        11,
+        null,
+        true
+      );
+    });
   });
 
   describe('test isInvalid', () => {

package/src/demos/DatePicker/DatePicker.md

@@ -18,34 +18,35 @@ DateRangePicker = () => {
   const [from, setFrom] = React.useState();
   const [to, setTo] = React.useState();
 
-  const toValidator = date => isValidDate(from) && date >= from ? '' : 'To date must be less than from date';
-  const onFromChange = (_str, date) => {
+  const toValidator = (date) =>
+    isValidDate(from) && date >= from ? '' : 'The "to" date must be after the "from" date';
+
+  const onFromChange = (_event, _value, date) => {
     setFrom(new Date(date));
     if (isValidDate(date)) {
       date.setDate(date.getDate() + 1);
       setTo(yyyyMMddFormat(date));
-    }
-    else {
+    } else {
       setTo('');
     }
   };
 
+  const onToChange = (_event, _value, date) => {
+    if (isValidDate(date)) {
+      setTo(yyyyMMddFormat(date));
+    }
+  };
+
   return (
     <Split>
       <SplitItem>
-        <DatePicker
-          onChange={onFromChange}
-          aria-label="Start date"
-          placeholder="YYYY-MM-DD"
-        />
-      </SplitItem>
-      <SplitItem style={{ padding: '6px 12px 0 12px' }}>
-        to
+        <DatePicker onChange={onFromChange} aria-label="Start date" placeholder="YYYY-MM-DD" />
       </SplitItem>
+      <SplitItem style={{ padding: '6px 12px 0 12px' }}>to</SplitItem>
       <SplitItem>
         <DatePicker
           value={to}
-          onChange={date => setTo(date)}
+          onChange={onToChange}
           isDisabled={!isValidDate(from)}
           rangeStart={from}
           validators={[toValidator]}
@@ -55,23 +56,34 @@ DateRangePicker = () => {
       </SplitItem>
     </Split>
   );
-}
+};
 ```
 
 ### Date and time range picker
 
 ```js
-import { Flex, FlexItem, InputGroup, DatePicker, isValidDate, TimePicker, yyyyMMddFormat, updateDateTime } from '@patternfly/react-core';
+import {
+  Flex,
+  FlexItem,
+  InputGroup,
+  DatePicker,
+  isValidDate,
+  TimePicker,
+  yyyyMMddFormat,
+  updateDateTime
+} from '@patternfly/react-core';
 
 DateTimeRangePicker = () => {
   const [from, setFrom] = React.useState();
   const [to, setTo] = React.useState();
 
-  const toValidator = date => {
-    return isValidDate(from) && yyyyMMddFormat(date) >= yyyyMMddFormat(from) ? '' : 'To date must after from date';
+  const toValidator = (date) => {
+    return isValidDate(from) && yyyyMMddFormat(date) >= yyyyMMddFormat(from)
+      ? ''
+      : 'The "to" date must be after the "from" date';
   };
-  
-  const onFromDateChange = (inputDate, newFromDate) => {
+
+  const onFromDateChange = (_event, inputDate, newFromDate) => {
     if (isValidDate(from) && isValidDate(newFromDate) && inputDate === yyyyMMddFormat(newFromDate)) {
       newFromDate.setHours(from.getHours());
       newFromDate.setMinutes(from.getMinutes());
@@ -80,7 +92,7 @@ DateTimeRangePicker = () => {
       setFrom(new Date(newFromDate));
     }
   };
-  
+
   const onFromTimeChange = (_event, time, hour, minute) => {
     if (isValidDate(from)) {
       const updatedFromDate = new Date(from);
@@ -90,16 +102,16 @@ DateTimeRangePicker = () => {
     }
   };
 
-  const onToDateChange = (inputDate, newToDate) => {
+  const onToDateChange = (_event, inputDate, newToDate) => {
     if (isValidDate(to) && isValidDate(newToDate) && inputDate === yyyyMMddFormat(newToDate)) {
       newToDate.setHours(to.getHours());
       newToDate.setMinutes(to.getMinutes());
     }
-    if (isValidDate(newToDate) && inputDate === yyyyMMddFormat(newToDate)){
+    if (isValidDate(newToDate) && inputDate === yyyyMMddFormat(newToDate)) {
       setTo(newToDate);
     }
   };
-  
+
   const onToTimeChange = (_event, time, hour, minute) => {
     if (isValidDate(to)) {
       const updatedToDate = new Date(to);
@@ -110,24 +122,14 @@ DateTimeRangePicker = () => {
   };
 
   return (
-    <Flex direction={{default: 'column', lg: 'row'}}>
+    <Flex direction={{ default: 'column', lg: 'row' }}>
       <FlexItem>
         <InputGroup>
-          <DatePicker
-            onChange={onFromDateChange}
-            aria-label="Start date"
-            placeholder="YYYY-MM-DD"
-          />
-          <TimePicker 
-            aria-label="Start time"
-            style={{width: '150px'}} 
-            onChange={onFromTimeChange} 
-          />
+          <DatePicker onChange={onFromDateChange} aria-label="Start date" placeholder="YYYY-MM-DD" />
+          <TimePicker aria-label="Start time" style={{ width: '150px' }} onChange={onFromTimeChange} />
         </InputGroup>
       </FlexItem>
-      <FlexItem>
-        to
-      </FlexItem>
+      <FlexItem>to</FlexItem>
       <FlexItem>
         <InputGroup>
           <DatePicker
@@ -139,19 +141,22 @@ DateTimeRangePicker = () => {
             aria-label="End date"
             placeholder="YYYY-MM-DD"
           />
-          <TimePicker style={{width: '150px'}} onChange={onToTimeChange} isDisabled={!isValidDate(from)}/>
+          <TimePicker style={{ width: '150px' }} onChange={onToTimeChange} isDisabled={!isValidDate(from)} />
         </InputGroup>
       </FlexItem>
     </Flex>
   );
-}
+};
 ```
 
-
 ### Date and time pickers in modal
+
 Modals trap focus and watch a few document level events. In order to place a date picker in a modal:
+
 - To avoid the modal's escape press event handler from overruling the date picker's escape press handlers, use the `DatePickerRef` to close the calendar when it is open and the escape key is pressed.
 - Append the calendar to the modal to keep it as close to the date picker in the DOM while maintaining correct layouts visually
   In order to place a time picker in the modal, its menu must be appended to the time picker's parent.
+
 ```ts file="./examples/DateTimePickerInModal.tsx"
+
 ```