From 630704843a1af32d7b07b5d57db4d983fe6e345b Mon Sep 17 00:00:00 2001
From: Anthony Ng <newyork.anthonyng@gmail.com>
Date: Sun, 12 Mar 2017 20:31:28 -0700
Subject: [PATCH] Add matchAnchorWidth to Popover

---
 packages/boundless-popover/index.js      | 22 +++++++++++++++-------
 packages/boundless-popover/index.spec.js | 12 ++++++++++++
 2 files changed, 27 insertions(+), 7 deletions(-)

diff --git a/packages/boundless-popover/index.js b/packages/boundless-popover/index.js
index 09edd873..eb158303 100644
--- a/packages/boundless-popover/index.js
+++ b/packages/boundless-popover/index.js
@@ -99,6 +99,11 @@ export default class Popover extends React.PureComponent {
          */
         autoReposition: PropTypes.bool,
 
+        /**
+         *  if popover element should match the anchor
+         */
+        matchAnchorWidth: PropTypes.bool,
+
         /**
          * a DOM element or React reference (ref) to one for positioning purposes, the caret component will
          * be automatically positioned to center on this provided anchor; by default it will center
@@ -148,6 +153,7 @@ export default class Popover extends React.PureComponent {
         ...Dialog.defaultProps,
         anchor: undefined,
         autoReposition: true,
+        matchAnchorWidth: false,
         captureFocus: false,
         caretAnchor: undefined,
         caretComponent: DEFAULT_CARET_COMPONENT,
@@ -373,19 +379,16 @@ export default class Popover extends React.PureComponent {
     }
 
     align = () => {
-        const anchor = this.props.anchor instanceof HTMLElement
-                       ? this.props.anchor
-                       : findDOMNode(this.props.anchor);
+        const anchor = findDOMNode(this.props.anchor);
 
-        // eslint-disable-next-line no-nested-ternary
         const caretAnchor = this.props.caretAnchor
-                            ? this.props.caretAnchor instanceof HTMLElement
-                              ? this.props.caretAnchor
-                              : findDOMNode(this.props.caretAnchor)
+                            ? findDOMNode(this.props.caretAnchor)
                             : anchor;
 
         this.cacheViewportCartography(anchor, caretAnchor);
 
+        if (this.props.matchAnchorWidth) { this.matchAnchorWidth(); }
+
         const preset = this.getValidAlignmentPreset();
         const frag = Popover.getAlignmentClassFragment;
 
@@ -411,6 +414,11 @@ export default class Popover extends React.PureComponent {
         this.$caret.style[longitudinal ? 'top' : 'left'] = '0px';
     }
 
+    matchAnchorWidth = () => {
+      const dialog = this.dialog.$dialog;
+      dialog.style.width = `${this.anchorRect.width}px`;
+    }
+
     componentDidMount() {
         this.align();
         window.addEventListener('resize', this.align, true);
diff --git a/packages/boundless-popover/index.spec.js b/packages/boundless-popover/index.spec.js
index 626de558..20a29107 100644
--- a/packages/boundless-popover/index.spec.js
+++ b/packages/boundless-popover/index.spec.js
@@ -408,6 +408,16 @@ describe('Popover component', () => {
             expect(popoverNode.classList.contains('b-popover-anchor-y-start')).toBeTruthy();
             expect(popoverNode.classList.contains('b-popover-self-y-start')).toBeTruthy();
         });
+
+        it('updates the width of the Dialog if matchAnchorWidth is true', () => {
+          sandbox.stub(anchor, 'getBoundingClientRect').returns({
+              width: 50,
+          });
+
+          render(<Popover {...baseProps} anchor={anchor} matchAnchorWidth />);
+
+          expect($('.b-dialog').style.width).toBe('50px');
+        });
     });
 
     describe('passthrough to Dialog', () => {
@@ -422,6 +432,8 @@ describe('Popover component', () => {
         it('passes down nested children', () => {
             expect($('.b-dialog').textContent).toContain('bar');
         });
+
+
     });
 
     describe('passthrough to Portal', () => {