Merge pull request metabase#2096 from metabase/fix-large-download

fix the overuse of `Modal` component which is implied with `ModalWithTrigger`.
sturgill · Mar 7, 2016 · 9592216 · 9592216
2 parents 9d1ed5f + 584622c
commit 9592216
Showing 1 changed file with 11 additions and 12 deletions.
diff --git a/frontend/src/query_builder/QueryVisualization.jsx b/frontend/src/query_builder/QueryVisualization.jsx
@@ -165,21 +165,20 @@ export default class QueryVisualization extends Component {
                     <ModalWithTrigger
                         key="download"
                         ref="downloadModal"
+                        className="Modal Modal--small"
                         triggerElement={<Icon className="mx1" title="Download this data" name='download' width="16px" height="16px" />}
                     >
-                        <Modal className="Modal Modal--small">
-                            <div className="p4 text-centered relative">
-                                <span className="absolute top right p4 text-normal text-grey-3 cursor-pointer" onClick={() => this.refs.downloadModal.toggle()}>
-                                    <Icon name={'close'} width={16} height={16} />
-                                </span>
-                                <div className="p3 text-strong">
-                                    <h2 className="text-bold">Download large data set</h2>
-                                    <div className="pt2">Your answer has a large amount of data so we wanted to let you know it could take a while to download.</div>
-                                    <div className="py4">The maximum download amount is 1 million rows.</div>
-                                    {downloadButton}
-                                </div>
+                        <div style={{width: "480px"}} className="Modal--small p4 text-centered relative">
+                            <span className="absolute top right p4 text-normal text-grey-3 cursor-pointer" onClick={() => this.refs.downloadModal.toggle()}>
+                                <Icon name={'close'} width={16} height={16} />
+                            </span>
+                            <div className="p3 text-strong">
+                                <h2 className="text-bold">Download large data set</h2>
+                                <div className="pt2">Your answer has a large amount of data so we wanted to let you know it could take a while to download.</div>
+                                <div className="py4">The maximum download amount is 1 million rows.</div>
+                                {downloadButton}
                             </div>
-                        </Modal>
+                        </div>
                     </ModalWithTrigger>
                 );
             } else {