chore(OrderSummary): avoid redundant rendering upon auto-generating e…

…lement IDs (carbon-design-system#529)
joshblack · Jan 24, 2018 · 549363a · 549363a
1 parent 6d2ddd9
commit 549363a
Showing 1 changed file with 10 additions and 6 deletions.
diff --git a/src/components/OrderSummary/OrderSummary.js b/src/components/OrderSummary/OrderSummary.js
@@ -37,9 +37,11 @@ export class OrderSummaryHeader extends Component {
     const {
       children,
       className,
-      id = `order__summary__header__id_${Math.random()
-        .toString(36)
-        .substr(2)}`,
+      id = (this._sectionId =
+        this._sectionId ||
+        `order__summary__header__id_${Math.random()
+          .toString(36)
+          .substr(2)}`),
       title,
       ...other
     } = this.props;
@@ -191,9 +193,11 @@ export class OrderSummaryFooter extends Component {
   render() {
     const {
       className,
-      id = `order__summary__footer__id_${Math.random()
-        .toString(36)
-        .substr(2)}`,
+      id = (this._sectionId =
+        this._sectionId ||
+        `order__summary__footer__id_${Math.random()
+          .toString(36)
+          .substr(2)}`),
       footerText,
       linkText,
       href,