Update: Removing some redundant code (#9)

Author: priyajeet

README.md

@@ -6,7 +6,7 @@
 
 [Box UI Elements](https://developer.box.com/docs/box-ui-elements)
 ==========================================================================
-The Box UI Elements are currently in beta and may contain bugs. Their interfaces may also update as needed. Please email [box-ui-kit-feedback@box.com](mailto:box-ui-kit-feedback@box.com) to report issues or provide feedback.
+The Box UI Elements are currently in beta and may contain bugs. Their interfaces may also update as needed. Please email [box-ui-elements@box.com](mailto:box-ui-elements@box.com) to report issues or provide feedback.
 
 ## Usage
 If you are not building a [React](https://facebook.github.io/react) based app, please follow the [documentation on our developer docs](https://developer.box.com/docs/box-ui-elements), which shows how to use the Box UI Elements via script includes. Continue reading below to import the components into your React based app.
@@ -47,8 +47,8 @@ render(
 );
 ```
 
-### Attributes
-| Attribute | Type | Default | Description |
+### Props
+| Prop | Type | Default | Description |
 | --- | --- | --- | --- |
 | token* | string |  | *See the [developer docs](https://developer.box.com/docs/box-content-explorer#section-options).* |
 | getLocalizedMessage* | function(string, { [string]: string }) |  | Function to get localized strings. |
@@ -100,8 +100,8 @@ render(
 );
 ```
 
-### Attributes
-| Attribute | Type | Default | Description |
+### Props
+| Prop | Type | Default | Description |
 | --- | --- | --- | --- |
 | token* | string |  | *See the [developer docs](https://developer.box.com/docs/box-content-picker#section-options).* |
 | getLocalizedMessage* | function(string, { [string]: string }) |  | Function to get localized strings. |
@@ -143,8 +143,8 @@ render(
 );
 ```
 
-### Attributes
-| Attribute | Type | Default | Description |
+### Props
+| Prop | Type | Default | Description |
 | --- | --- | --- | --- |
 | token* | string |  | *See the [developer docs](https://developer.box.com/docs/box-content-uploader#section-options).* |
 | getLocalizedMessage* | function(string, { [string]: string }) |  | Function to get localized strings. |
@@ -181,8 +181,8 @@ render(
 );
 ```
 
-### Attributes
-| Attribute | Type | Default | Description |
+### Props
+| Prop | Type | Default | Description |
 | --- | --- | --- | --- |
 | token* | string |  | *See the [developer docs](https://developer.box.com/docs/box-content-tree#section-options).* |
 | getLocalizedMessage* | function(string, { [string]: string }) |  | Function to get localized strings. |
@@ -218,8 +218,8 @@ render(
 );
 ```
 
-### Attributes
-| Attribute | Type | Default | Description |
+### Props
+| Prop | Type | Default | Description |
 | --- | --- | --- | --- |
 | token* | string |  | *See the [developer docs](https://developer.box.com/docs/box-content-preview#section-initialization-and-options).* |
 | fileId* | string | | The id of the file to preview. |

src/components/Breadcrumbs/BreadcrumbDropdown.js

@@ -18,7 +18,7 @@ type Props = {
 };
 
 const BreadcrumbDropdown = ({ crumbs, onCrumbClick, className = '' }: Props) =>
-    <DropdownMenu constrainToScrollParent renderElementTo='.buik-app-element'>
+    <DropdownMenu constrainToScrollParent>
         <PlainButton className={`buik-breadcrumbs-drop-down ${className}`}>
             ···
         </PlainButton>

src/components/ContentExplorer/ItemList.js

@@ -162,7 +162,7 @@ const ItemList = ({
                         disableSort
                         dataKey={FIELD_ID}
                         cellRenderer={moreOptionsCell}
-                        width={isSmall ? 58 : 140}
+                        width={isSmall || !canShare ? 58 : 140}
                         flexShrink={0}
                     />
                 </Table>}

src/components/ContentExplorer/moreOptionsCellRenderer.js

@@ -63,7 +63,7 @@ export default (
 
     return (
         <div className='bce-more-options'>
-            <DropdownMenu isRightAligned constrainToScrollParent renderElementTo='.bce-content'>
+            <DropdownMenu isRightAligned constrainToScrollParent>
                 <Button onFocus={onFocus} className='bce-btn-more-options'>
                     ···
                 </Button>

src/components/DropdownMenu/DropdownMenu.js

@@ -14,7 +14,7 @@ type Props = {
     constrainToScrollParent: boolean,
     constrainToWindow: boolean,
     isRightAligned: boolean,
-    renderElementTo: string
+    bodyElement: string | HTMLElement
 };
 
 type DefaultProps = {|
@@ -217,13 +217,7 @@ class DropdownMenu extends PureComponent<DefaultProps, Props, State> {
      * @return {Element}
      */
     render() {
-        const {
-            children,
-            isRightAligned,
-            constrainToScrollParent,
-            constrainToWindow,
-            renderElementTo
-        }: Props = this.props;
+        const { children, isRightAligned, constrainToScrollParent, constrainToWindow, bodyElement }: Props = this.props;
         const { isOpen, initialFocusIndex }: State = this.state;
         const elements = Children.toArray(children);
 
@@ -287,7 +281,7 @@ class DropdownMenu extends PureComponent<DefaultProps, Props, State> {
                 targetAttachment={targetAttachment}
                 constraints={constraints}
                 enabled={isOpen}
-                renderElementTo={renderElementTo}
+                bodyElement={bodyElement}
             >
                 {cloneElement(menuButton, menuButtonProps)}
                 {isOpen ? cloneElement(menu, menuProps) : null}

test/explorer-no-react.html

@@ -85,6 +85,8 @@ <h1>Content Explorer</h1>
                 document.querySelector('.explorer1').innerHTML = '';
                 explorer1.show(folderId, token, {
                     container: '.explorer1',
+                    currentFolderId: '4579229774',
+                    canShare: false,
                     logoUrl: 'https://d30y9cdsu7xlg0.cloudfront.net/png/12458-200.png'
                 });
             }