@patternfly/react-core 4.219.0 → 4.219.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@patternfly/react-core",
-  "version": "4.219.0",
+  "version": "4.219.1",
   "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",
@@ -34,9 +34,9 @@
     "generate": "node scripts/copyStyles.js"
   },
   "dependencies": {
-    "@patternfly/react-icons": "^4.70.0",
-    "@patternfly/react-styles": "^4.69.0",
-    "@patternfly/react-tokens": "^4.71.0",
+    "@patternfly/react-icons": "^4.70.1",
+    "@patternfly/react-styles": "^4.69.1",
+    "@patternfly/react-tokens": "^4.71.1",
     "focus-trap": "6.2.2",
     "react-dropzone": "9.0.0",
     "tippy.js": "5.1.2",
@@ -60,5 +60,5 @@
     "react": "^16.8.0 || ^17.0.0",
     "react-dom": "^16.8.0 || ^17.0.0"
   },
-  "gitHead": "35dea6aaa7ed4c6f49ae0d3ee6644dda3efb2889"
+  "gitHead": "7a00f2e2733a593e840464c786f7096e004d02f1"
 }

package/src/components/ExpandableSection/examples/ExpandableSection.md

@@ -5,6 +5,8 @@ cssPrefix: pf-c-expandable-section
 propComponents: ['ExpandableSection', 'ExpandableSectionToggle']
 ---
 
+import CheckCircleIcon from '@patternfly/react-icons/dist/esm/icons/check-circle-icon';
+
 ## Examples
 
 ### Basic
@@ -178,3 +180,48 @@ class SimpleExpandableSection extends React.Component {
   }
 }
 ```
+
+### With custom toggle content
+
+By using the `toggleContent` prop, you can pass in content other than a simple string such as an icon or a badge. When passing in custom content in this way, you should not pass in any interactive element such as a button.
+
+```js
+import React from "react";
+import { ExpandableSection, Badge } from "@patternfly/react-core";
+import CheckCircleIcon from "@patternfly/react-icons/dist/esm/icons/check-circle-icon";
+
+class CustomExpandableSection extends React.Component {
+  constructor(props) {
+    super(props);
+    this.state = {
+      isExpanded: false
+    };
+    this.onToggle = (isExpanded) => {
+      this.setState({
+        isExpanded
+      });
+    };
+  }
+
+  render() {
+    const { isExpanded } = this.state;
+    return (
+      <ExpandableSection
+        toggleContent={
+          <div>
+            <span>You can also use icons </span>
+            <CheckCircleIcon />
+            <span> or badges </span>
+            <Badge isRead={true}>4</Badge>
+            <span> !</span>
+          </div>
+        }
+        onToggle={this.onToggle}
+        isExpanded={isExpanded}
+      >
+        This content is visible only when the component is expanded.
+      </ExpandableSection>
+    );
+  }
+}
+```

package/src/components/Modal/ModalBoxTitle.tsx

@@ -37,7 +37,7 @@ export const ModalBoxTitle: React.FunctionComponent<ModalBoxTitleProps> = ({
   titleLabel = '',
   ...props
 }: ModalBoxTitleProps) => {
-  const [isTooltipVisible, setIsTooltipVisible] = React.useState(false);
+  const [hasTooltip, setHasTooltip] = React.useState(false);
   const h1 = React.useRef<HTMLHeadingElement>();
   const label = titleLabel || (isVariantIcon(titleIconVariant) ? `${capitalize(titleIconVariant)} alert:` : titleLabel);
   const variantIcons = {
@@ -50,7 +50,7 @@ export const ModalBoxTitle: React.FunctionComponent<ModalBoxTitleProps> = ({
   const CustomIcon = !isVariantIcon(titleIconVariant) && titleIconVariant;
 
   useIsomorphicLayoutEffect(() => {
-    setIsTooltipVisible(h1.current && h1.current.offsetWidth < h1.current.scrollWidth);
+    setHasTooltip(h1.current && h1.current.offsetWidth < h1.current.scrollWidth);
   }, []);
 
   const content = (
@@ -70,12 +70,6 @@ export const ModalBoxTitle: React.FunctionComponent<ModalBoxTitleProps> = ({
     </h1>
   );
 
-  return isTooltipVisible ? (
-    <Tooltip content={title} isVisible>
-      {content}
-    </Tooltip>
-  ) : (
-    content
-  );
+  return hasTooltip ? <Tooltip content={title}>{content}</Tooltip> : content;
 };
 ModalBoxTitle.displayName = 'ModalBoxTitle';

package/src/components/Slider/Slider.tsx

@@ -235,9 +235,7 @@ export const Slider: React.FunctionComponent<SliderProps> = ({
     if (!areCustomStepsContinuous && customSteps) {
       let percentage = newPercentage;
       if (customSteps[customSteps.length - 1].value !== 100) {
-        const numSteps = customSteps.length - 1;
-        const percentagePerStep = 100 / numSteps;
-        percentage = newPercentage / percentagePerStep;
+        percentage = (newPercentage * (max - min)) / 100 + min;
       }
       const stepIndex = customSteps.findIndex(stepObj => stepObj.value >= percentage);
       if (customSteps[stepIndex].value === percentage) {

package/src/components/Slider/examples/Slider.md

@@ -29,7 +29,8 @@ class DiscreteInput extends React.Component {
       value3: 25,
       value4: 50,
       value5: 50,
-      value6: 3
+      value6: 3,
+      value7: 25
     };
 
     this.steps = [
@@ -53,6 +54,15 @@ class DiscreteInput extends React.Component {
       { value: 5, label: "F" }
     ];
 
+    this.stepsDiscreteNoLinearWithMaxMin = [
+      { value: 12, label: '12' },
+      { value: 15, label: '15' },
+      { value: 25, label: '25' },
+      { value: 54, label: '54' },
+      { value: 67, label: '67' },
+      { value: 86, label: '86' }
+  ];
+
     this.onChange = value => {
         this.setState({
           value1: value
@@ -88,6 +98,12 @@ class DiscreteInput extends React.Component {
           value6: value
         });
     };
+
+      this.onChange7 = value => {
+        this.setState({
+          value7: value
+        });
+    };
   }
 
   render() {
@@ -123,7 +139,18 @@ class DiscreteInput extends React.Component {
             customSteps={this.stepsDiscreteWithMax}
             onChange={this.onChange6}
           />
-          <br />
+        <br />
+        <Text component={TextVariants.h3}>Slider value is: {Math.floor(this.state.value7)}</Text>
+        <Text component={TextVariants.small}>(min = 12, max = 86, custom steps with non linear data) </Text>
+        <Slider
+            value={this.state.value7}
+            showTicks
+            customSteps={this.stepsDiscreteNoLinearWithMaxMin}
+            onChange={this.onChange7}
+            min={12}
+            max={86}
+          />
+        <br />
       </>
     );
   }