@patternfly/react-core 5.1.1-prerelease.16 → 5.1.1-prerelease.17

package/helpers/package.json

@@ -1 +1 @@
-{"name":"@patternfly/react-core-helpers","main":"../dist/js/helpers/index.js","module":"../dist/esm/helpers/index.js","typings":"../dist/esm/helpers/index.d.ts","version":"5.1.1-prerelease.15","private":true}
+{"name":"@patternfly/react-core-helpers","main":"../dist/js/helpers/index.js","module":"../dist/esm/helpers/index.js","typings":"../dist/esm/helpers/index.d.ts","version":"5.1.1-prerelease.16","private":true}

package/layouts/package.json

@@ -1 +1 @@
-{"name":"@patternfly/react-core-layouts","main":"../dist/js/layouts/index.js","module":"../dist/esm/layouts/index.js","typings":"../dist/esm/layouts/index.d.ts","version":"5.1.1-prerelease.15","private":true}
+{"name":"@patternfly/react-core-layouts","main":"../dist/js/layouts/index.js","module":"../dist/esm/layouts/index.js","typings":"../dist/esm/layouts/index.d.ts","version":"5.1.1-prerelease.16","private":true}

package/next/package.json

@@ -1 +1 @@
-{"name":"@patternfly/react-core-next","main":"../dist/js/next/index.js","module":"../dist/esm/next/index.js","typings":"../dist/esm/next/index.d.ts","version":"5.1.1-prerelease.15","private":true}
+{"name":"@patternfly/react-core-next","main":"../dist/js/next/index.js","module":"../dist/esm/next/index.js","typings":"../dist/esm/next/index.d.ts","version":"5.1.1-prerelease.16","private":true}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@patternfly/react-core",
-  "version": "5.1.1-prerelease.16",
+  "version": "5.1.1-prerelease.17",
   "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",
@@ -70,5 +70,5 @@
     "react": "^17 || ^18",
     "react-dom": "^17 || ^18"
   },
-  "gitHead": "bf516c9a45a29344c825a4d0425726e27c9e565b"
+  "gitHead": "026096f9f505bb31b24bb25b551ec81a743ef1fa"
 }

package/src/components/Slider/Slider.tsx

@@ -46,16 +46,18 @@ export interface SliderProps extends Omit<React.HTMLProps<HTMLDivElement>, 'onCh
   inputAriaLabel?: string;
   /** Text label that is place after the input field. */
   inputLabel?: string | number;
-  /** Position of the input. */
-  inputPosition?: 'aboveThumb' | 'right';
+  /** Position of the input. Note "right" is deprecated. Use "end" instead*/
+  inputPosition?: 'aboveThumb' | 'right' | 'end';
   /** Value displayed in the input field. */
   inputValue?: number;
   /** Adds disabled styling, and disables the slider and the input component if present. */
   isDisabled?: boolean;
   /** Flag to show value input field. */
   isInputVisible?: boolean;
-  /** Actions placed to the left of the slider. */
+  /** @deprecated Use startActions instead. Actions placed at the start of the slider. */
   leftActions?: React.ReactNode;
+  /** Actions placed at the start of the slider. */
+  startActions?: React.ReactNode;
   /** The maximum permitted value. */
   max?: number;
   /** The minimum permitted value. */
@@ -67,8 +69,10 @@ export interface SliderProps extends Omit<React.HTMLProps<HTMLDivElement>, 'onCh
     inputValue?: number,
     setLocalInputValue?: React.Dispatch<React.SetStateAction<number>>
   ) => void;
-  /** Actions placed to the right of the slider. */
+  /** @deprecated Use endActions instead. Actions placed to the right of the slider. */
   rightActions?: React.ReactNode;
+  /** Actions placed at the end of the slider. */
+  endActions?: React.ReactNode;
   /** Flag to indicate if boundaries should be shown for slider that does not have custom steps. */
   showBoundaries?: boolean;
   /** Flag to indicate if ticks should be shown for slider that does not have custom steps. */
@@ -95,10 +99,12 @@ export const Slider: React.FunctionComponent<SliderProps> = ({
   inputAriaLabel = 'Slider value input',
   thumbAriaLabel = 'Value',
   hasTooltipOverThumb = false,
-  inputPosition = 'right',
+  inputPosition = 'end',
   onChange,
   leftActions,
+  startActions,
   rightActions,
+  endActions,
   step = 1,
   min = 0,
   max = 100,
@@ -114,6 +120,12 @@ export const Slider: React.FunctionComponent<SliderProps> = ({
   const [localValue, setValue] = useState(value);
   const [localInputValue, setLocalInputValue] = useState(inputValue);
 
+  let isRTL: boolean;
+
+  React.useEffect(() => {
+    isRTL = window.getComputedStyle(sliderRailRef.current).getPropertyValue('direction') === 'rtl';
+  });
+
   React.useEffect(() => {
     setValue(value);
   }, [value]);
@@ -182,7 +194,11 @@ export const Slider: React.FunctionComponent<SliderProps> = ({
     e.stopPropagation();
     e.preventDefault();
 
-    diff = e.clientX - thumbRef.current.getBoundingClientRect().left;
+    if (isRTL) {
+      diff = thumbRef.current.getBoundingClientRect().right - e.clientX;
+    } else {
+      diff = e.clientX - thumbRef.current.getBoundingClientRect().left;
+    }
 
     document.addEventListener('mousemove', callbackThumbMove);
     document.addEventListener('mouseup', callbackThumbUp);
@@ -191,7 +207,11 @@ export const Slider: React.FunctionComponent<SliderProps> = ({
   const handleTouchStart = (e: React.TouchEvent) => {
     e.stopPropagation();
 
-    diff = e.touches[0].clientX - thumbRef.current.getBoundingClientRect().left;
+    if (isRTL) {
+      diff = thumbRef.current.getBoundingClientRect().right - e.touches[0].clientX;
+    } else {
+      diff = e.touches[0].clientX - thumbRef.current.getBoundingClientRect().left;
+    }
 
     document.addEventListener('touchmove', callbackThumbMove, { passive: false });
     document.addEventListener('touchend', callbackThumbUp);
@@ -216,8 +236,13 @@ export const Slider: React.FunctionComponent<SliderProps> = ({
     }
 
     const clientPosition = e.touches && e.touches.length ? e.touches[0].clientX : e.clientX;
+    let newPosition;
 
-    let newPosition = clientPosition - diff - sliderRailRef.current.getBoundingClientRect().left;
+    if (isRTL) {
+      newPosition = sliderRailRef.current.getBoundingClientRect().right - clientPosition - diff;
+    } else {
+      newPosition = clientPosition - diff - sliderRailRef.current.getBoundingClientRect().left;
+    }
 
     const end = sliderRailRef.current.offsetWidth - thumbRef.current.offsetWidth;
 
@@ -288,21 +313,43 @@ export const Slider: React.FunctionComponent<SliderProps> = ({
     if (!areCustomStepsContinuous && customSteps) {
       const stepIndex = customSteps.findIndex((stepObj) => stepObj.value === localValue);
       if (key === 'ArrowRight') {
-        if (stepIndex + 1 < customSteps.length) {
-          {
-            newValue = customSteps[stepIndex + 1].value;
+        if (isRTL) {
+          if (stepIndex - 1 >= 0) {
+            newValue = customSteps[stepIndex - 1].value;
+          }
+        } else {
+          if (stepIndex + 1 < customSteps.length) {
+            {
+              newValue = customSteps[stepIndex + 1].value;
+            }
           }
         }
       } else if (key === 'ArrowLeft') {
-        if (stepIndex - 1 >= 0) {
-          newValue = customSteps[stepIndex - 1].value;
+        if (isRTL) {
+          if (stepIndex + 1 < customSteps.length) {
+            {
+              newValue = customSteps[stepIndex + 1].value;
+            }
+          }
+        } else {
+          if (stepIndex - 1 >= 0) {
+            newValue = customSteps[stepIndex - 1].value;
+          }
         }
       }
     } else {
       if (key === 'ArrowRight') {
-        newValue = localValue + step <= max ? localValue + step : max;
+        if (isRTL) {
+          newValue = localValue - step >= min ? localValue - step : min;
+        } else {
+          newValue = localValue + step <= max ? localValue + step : max;
+        }
       } else if (key === 'ArrowLeft') {
-        newValue = localValue - step >= min ? localValue - step : min;
+        if (isRTL) {
+          newValue = localValue + step <= max ? localValue + step : max;
+        } else {
+          newValue = localValue - step >= min ? localValue - step : min;
+        }
       }
     }
 
@@ -347,8 +394,8 @@ export const Slider: React.FunctionComponent<SliderProps> = ({
     for (let i = min; i <= max; i = i + step) {
       const stepValue = getStepValue(i, min, max);
 
-      // If we boundaries but not ticks just generate the needed steps
-      // so that we don't pullute them DOM with empty divs
+      // If boundaries but not ticks just generate the needed steps
+      // so that we don't pollute them DOM with empty divs
       if (!showTicks && showBoundaries && i !== min && i !== max) {
         continue;
       }
@@ -394,7 +441,7 @@ export const Slider: React.FunctionComponent<SliderProps> = ({
       style={{ ...style, ...inputStyle }}
       {...props}
     >
-      {leftActions && <div className={css(styles.sliderActions)}>{leftActions}</div>}
+      {(leftActions || startActions) && <div className={css(styles.sliderActions)}>{leftActions || startActions}</div>}
       <div className={css(styles.sliderMain)}>
         <div className={css(styles.sliderRail)} ref={sliderRailRef} onClick={!isDisabled ? onSliderRailClick : null}>
           <div className={css(styles.sliderRailTrack)} />
@@ -434,8 +481,10 @@ export const Slider: React.FunctionComponent<SliderProps> = ({
           <div className={css(styles.sliderValue, styles.modifiers.floating)}>{displayInput()}</div>
         )}
       </div>
-      {isInputVisible && inputPosition === 'right' && <div className={css(styles.sliderValue)}>{displayInput()}</div>}
-      {rightActions && <div className={css(styles.sliderActions)}>{rightActions}</div>}
+      {isInputVisible && (inputPosition === 'right' || inputPosition === 'end') && (
+        <div className={css(styles.sliderValue)}>{displayInput()}</div>
+      )}
+      {(rightActions || endActions) && <div className={css(styles.sliderActions)}>{rightActions || endActions}</div>}
     </div>
   );
 };

package/src/components/Slider/_tests_/Slider.test.tsx

@@ -46,7 +46,7 @@ describe('slider', () => {
 
   test('renders slider with input', () => {
     const { asFragment } = render(
-      <Slider value={50} isInputVisible inputValue={50} inputLabel="%" inputPosition="right" />
+      <Slider value={50} isInputVisible inputValue={50} inputLabel="%" inputPosition="end" />
     );
     expect(asFragment()).toMatchSnapshot();
   });
@@ -62,8 +62,8 @@ describe('slider', () => {
     const { asFragment } = render(
       <Slider
         value={50}
-        leftActions={<Button variant="plain" aria-label="Minus" />}
-        rightActions={<Button variant="plain" aria-label="Plus" />}
+        startActions={<Button variant="plain" aria-label="Minus" />}
+        endActions={<Button variant="plain" aria-label="Plus" />}
       />
     );
     expect(asFragment()).toMatchSnapshot();

package/src/components/Slider/examples/SliderActions.tsx

@@ -65,12 +65,12 @@ export const SliderActions: React.FunctionComponent = () => {
       <Slider
         value={value1}
         onChange={onChange1}
-        leftActions={
+        startActions={
           <Button variant="plain" aria-label="Minus" onClick={onMinusClick}>
             <MinusIcon />
           </Button>
         }
-        rightActions={
+        endActions={
           <Button variant="plain" aria-label="Plus" onClick={onPlusClick}>
             <PlusIcon />
           </Button>
@@ -85,7 +85,7 @@ export const SliderActions: React.FunctionComponent = () => {
         inputLabel="%"
         isInputVisible
         isDisabled={isDisabled}
-        rightActions={buildAction(isDisabled)}
+        endActions={buildAction(isDisabled)}
       />
     </>
   );

package/src/components/Tabs/examples/Tabs.md

@@ -28,7 +28,7 @@ You can adjust a tab in the following ways:
 - To disable a tab, but keep it perceivable to assistive technology users, use the `isAriaDisabled` property. If a disabled tab has a tooltip, use this property instead of `isDisabled`.
 - To add a tooltip to an aria-disabled tab, use the `tooltip` property. 
 
-Most tab variations can either be 'default' or 'boxed': 
+Tabs can be styled as 'default' or 'boxed': 
 
 - Default tabs do not have any borders and use a bottom line to distinguish between a selected tab, a hovered tab, and an inactive tab. 
 - Boxed tabs are outlined to emphasize the area that a tab spans. To preview boxed tabs in the following examples, select the 'isBox' checkbox, which sets the `isBox` property to true.
@@ -207,8 +207,6 @@ The tab its content should only be mounted when the tab is visible.
 
 To enable closeable tabs, pass the `onClose` property to the `<Tabs>` component. To enable a button that adds new tabs, pass the `onAdd` property to `<Tabs>`. 
 
-Aria labels may be controlled manually by passing the `closeButtonAriaLabel` property to a `<Tab>` and the `addButtonAriaLabel` property to `<Tabs>`.
-
 ```ts file="./TabsDynamic.tsx"
 ```
 

package/src/components/Toolbar/examples/Toolbar.md

@@ -14,12 +14,6 @@ import EllipsisVIcon from '@patternfly/react-icons/dist/esm/icons/ellipsis-v-ico
 
 ## Examples
 
-### TEST - remove before merging
-
-```ts file="./Test.tsx"
-
-```
-
 ### Toolbar items
 
 A toolbar can contain multiple toolbar items, like filters and buttons.

package/src/components/Toolbar/examples/Test.tsx

@@ -1,258 +0,0 @@
-import React from 'react';
-import {
-  MenuToggle,
-  MenuToggleElement,
-  Toolbar,
-  ToolbarItem,
-  ToolbarContent,
-  ToolbarToggleGroup,
-  ToolbarGroup,
-  SearchInput,
-  Select,
-  SelectList,
-  SelectOption,
-  ToolbarFilter,
-  Badge
-} from '@patternfly/react-core';
-import FilterIcon from '@patternfly/react-icons/dist/esm/icons/filter-icon';
-
-export const Test: React.FunctionComponent = () => {
-  const statusChipContainerRef = React.useRef<HTMLDivElement>();
-  const riskChipContainerRef = React.useRef<HTMLDivElement>();
-
-  const [isStatusGroupExpanded, setIsStatusGroupExpanded] = React.useState(false);
-  const [isRiskGroupExpanded, setIsRiskGroupExpanded] = React.useState(false);
-
-  const [isStatusMenuExpanded, setIsStatusExpanded] = React.useState(false);
-  const [isRiskMenuExpanded, setIsRiskExpanded] = React.useState(false);
-  const [inputValue, setInputValue] = React.useState('');
-  const [filters, setFilters] = React.useState({
-    risk: ['Low'],
-    status: ['New', 'Pending']
-  });
-
-  const closeToggleGroups = () => {
-    setIsStatusGroupExpanded(false);
-    setIsRiskGroupExpanded(false);
-  };
-
-  React.useEffect(() => {
-    window.addEventListener('resize', closeToggleGroups); // Resize observer to toggle off expand groups is required to properly reformat toolbar when growing
-    return () => {
-      window.removeEventListener('resize', closeToggleGroups);
-    };
-  }, []);
-
-  const onInputChange = (newValue: string) => {
-    setInputValue(newValue);
-  };
-
-  const onStatusToggle = () => {
-    setIsStatusExpanded(!isStatusMenuExpanded);
-  };
-
-  const onRiskToggle = () => {
-    setIsRiskExpanded(!isRiskMenuExpanded);
-  };
-
-  const onSelect = (type: string, event: React.MouseEvent | React.ChangeEvent | undefined, selection: string) => {
-    const checked = (event?.target as HTMLInputElement).checked;
-    setFilters((prev) => {
-      const prevSelections = prev[type];
-      return {
-        ...prev,
-        [type]: checked ? [...prevSelections, selection] : prevSelections.filter((value) => value !== selection)
-      };
-    });
-  };
-
-  const onStatusSelect = (event: React.MouseEvent | React.ChangeEvent | undefined, selection: string) => {
-    onSelect('status', event, selection);
-  };
-
-  const onRiskSelect = (event: React.MouseEvent | React.ChangeEvent | undefined, selection: string) => {
-    onSelect('risk', event, selection);
-  };
-
-  const onDelete = (type: string, id: string) => {
-    if (type === 'Risk') {
-      setFilters({ risk: filters.risk.filter((fil: string) => fil !== id), status: filters.status });
-    } else if (type === 'Status') {
-      setFilters({ risk: filters.risk, status: filters.status.filter((fil: string) => fil !== id) });
-    } else {
-      setFilters({ risk: [], status: [] });
-    }
-  };
-
-  const onDeleteGroup = (type: string) => {
-    if (type === 'Risk') {
-      setFilters({ risk: [], status: filters.status });
-    } else if (type === 'Status') {
-      setFilters({ risk: filters.risk, status: [] });
-    }
-  };
-
-  const statusToggleGroupItems = (
-    <React.Fragment>
-      <ToolbarItem variant="search-filter">
-        <SearchInput
-          aria-label="Consumer toggle groups example search input"
-          onChange={(_event, value) => onInputChange(value)}
-          value={inputValue}
-          onClear={() => {
-            onInputChange('');
-          }}
-        />
-      </ToolbarItem>
-      <ToolbarGroup variant="filter-group">
-        <ToolbarFilter
-          chips={filters.status}
-          deleteChip={(category, chip) => onDelete(category as string, chip as string)}
-          deleteChipGroup={(category) => onDeleteGroup(category as string)}
-          categoryName="Status"
-          isExpanded={isStatusGroupExpanded}
-          expandableChipContainerRef={statusChipContainerRef} // Required to link the toolbar filter chip group to the custom expandable group
-        >
-          <Select
-            aria-label="Status"
-            role="menu"
-            toggle={(toggleRef: React.Ref<MenuToggleElement>) => (
-              <MenuToggle
-                ref={toggleRef}
-                onClick={onStatusToggle}
-                isExpanded={isStatusMenuExpanded}
-                style={
-                  {
-                    width: '140px'
-                  } as React.CSSProperties
-                }
-              >
-                Status
-                {filters.status.length > 0 && <Badge isRead>{filters.status.length}</Badge>}
-              </MenuToggle>
-            )}
-            onSelect={(event, selection) => onStatusSelect(event, selection as string)}
-            selected={filters.status}
-            isOpen={isStatusMenuExpanded}
-            onOpenChange={(isOpen) => setIsStatusExpanded(isOpen)}
-          >
-            <SelectList>
-              <SelectOption hasCheckbox key="statusNew" value="New" isSelected={filters.status.includes('New')}>
-                New
-              </SelectOption>
-              <SelectOption
-                hasCheckbox
-                key="statusPending"
-                value="Pending"
-                isSelected={filters.status.includes('Pending')}
-              >
-                Pending
-              </SelectOption>
-              <SelectOption
-                hasCheckbox
-                key="statusRunning"
-                value="Running"
-                isSelected={filters.status.includes('Running')}
-              >
-                Running
-              </SelectOption>
-              <SelectOption
-                hasCheckbox
-                key="statusCancelled"
-                value="Cancelled"
-                isSelected={filters.status.includes('Cancelled')}
-              >
-                Cancelled
-              </SelectOption>
-            </SelectList>
-          </Select>
-        </ToolbarFilter>
-      </ToolbarGroup>
-    </React.Fragment>
-  );
-
-  const riskToggleGroupItems = (
-    <ToolbarGroup variant="filter-group">
-      <ToolbarFilter
-        chips={filters.risk}
-        deleteChip={(category, chip) => onDelete(category as string, chip as string)}
-        categoryName="Risk"
-        isExpanded={isRiskGroupExpanded}
-        expandableChipContainerRef={riskChipContainerRef} // Required to link the toolbar filter chip group to the custom expandable group
-      >
-        <Select
-          aria-label="Risk"
-          role="menu"
-          toggle={(toggleRef: React.Ref<MenuToggleElement>) => (
-            <MenuToggle
-              ref={toggleRef}
-              onClick={onRiskToggle}
-              isExpanded={isRiskMenuExpanded}
-              style={
-                {
-                  width: '140px'
-                } as React.CSSProperties
-              }
-            >
-              Risk
-              {filters.risk.length > 0 && <Badge isRead>{filters.risk.length}</Badge>}
-            </MenuToggle>
-          )}
-          onSelect={(event, selection) => onRiskSelect(event, selection as string)}
-          selected={filters.risk}
-          isOpen={isRiskMenuExpanded}
-          onOpenChange={(isOpen) => setIsRiskExpanded(isOpen)}
-        >
-          <SelectList>
-            <SelectOption hasCheckbox key="riskLow" value="Low" isSelected={filters.risk.includes('Low')}>
-              Low
-            </SelectOption>
-            <SelectOption hasCheckbox key="riskMedium" value="Medium" isSelected={filters.risk.includes('Medium')}>
-              Medium
-            </SelectOption>
-            <SelectOption hasCheckbox key="riskHigh" value="High" isSelected={filters.risk.includes('High')}>
-              High
-            </SelectOption>
-          </SelectList>
-        </Select>
-      </ToolbarFilter>
-    </ToolbarGroup>
-  );
-
-  return (
-    <Toolbar id="toolbar-multiple-toggle-groups" className="pf-m-toggle-group-container">
-      <ToolbarContent>
-        <ToolbarToggleGroup
-          isExpanded={isStatusGroupExpanded} // Required to control expanded state
-          onToggle={() => {
-            setIsStatusGroupExpanded(!isStatusGroupExpanded);
-            setIsRiskGroupExpanded(false);
-          }} // Required to control expanded state
-          toggleIcon={<FilterIcon />}
-          breakpoint="md"
-          chipContainerRef={statusChipContainerRef}
-          showClearFiltersButton
-          clearAllFilters={() => onDeleteGroup('Status')}
-          clearFiltersButtonText="Clear status filter"
-        >
-          {statusToggleGroupItems}
-        </ToolbarToggleGroup>
-        <ToolbarToggleGroup
-          isExpanded={isRiskGroupExpanded} // Required to control expanded state
-          onToggle={() => {
-            setIsRiskGroupExpanded(!isRiskGroupExpanded);
-            setIsStatusGroupExpanded(false);
-          }} // Required to control expanded state
-          toggleIcon={<FilterIcon />}
-          breakpoint="xl"
-          chipContainerRef={riskChipContainerRef}
-          showClearFiltersButton
-          clearAllFilters={() => onDeleteGroup('Risk')}
-          clearFiltersButtonText={'Clear risk filter'}
-        >
-          {riskToggleGroupItems}
-        </ToolbarToggleGroup>
-      </ToolbarContent>
-    </Toolbar>
-  );
-};