Просмотр исходного кода

Moved `nav-title` to `ModalHeader` and adjusted the appearance.

oshikishintaro 5 лет назад
Родитель
Сommit
9dcfc55a15

+ 6 - 9
src/client/js/components/PageAccessoriesModal.jsx

@@ -2,7 +2,7 @@ import React, { useEffect, useState } from 'react';
 import PropTypes from 'prop-types';
 
 import {
-  Button, Modal, ModalBody, Nav, NavItem, NavLink, TabContent, TabPane,
+  Button, Modal, ModalBody, ModalHeader, Nav, NavItem, NavLink, TabContent, TabPane,
 } from 'reactstrap';
 
 import { withTranslation } from 'react-i18next';
@@ -103,7 +103,7 @@ const PageAccessoriesModal = (props) => {
     <React.Fragment>
       <Modal size="xl" isOpen={props.isOpen} toggle={closeModalHandler} className="grw-page-accessories-modal">
         {/* [TODO: insert a modal header and move nav tabs there  by gw-3890] */}
-        <ModalBody className="overflow-auto grw-modal-body-style p-0">
+        <ModalHeader className="p-0" toggle={closeModalHandler}>
           <Nav className="nav-title" id="nav-title">
             {Object.entries(navTabMapping).map(([key, value]) => {
               return (
@@ -115,14 +115,11 @@ const PageAccessoriesModal = (props) => {
                 </NavItem>
               );
             })}
-            <Button
-              className="my-auto ml-auto mr-3"
-              close
-              onClick={closeModalHandler}
-            />
           </Nav>
-          <hr className="my-0 grw-nav-slide-hr" style={{ width: `${sliderWidth}%`, marginLeft: `${sliderMarginLeft}%` }} />
-          <hr className="modal-split-hr m-0" />
+          <hr className="my-0 grw-nav-slide-hr border-none" style={{ width: `${sliderWidth}%`, marginLeft: `${sliderMarginLeft}%` }} />
+        </ModalHeader>
+        <ModalBody className="overflow-auto grw-modal-body-style p-0">
+          {/* <hr className="modal-split-hr m-0" /> */}
           <TabContent activeTab={activeTab}>
             <TabPane tabId="pagelist">
               {pageAccessoriesContainer.state.activeComponents.has('pagelist') && <PageList />}

+ 6 - 0
src/client/styles/scss/_page_accessaries_modal.scss

@@ -8,6 +8,12 @@
       }
     }
   }
+  .modal-header {
+    button.close {
+      margin: auto 0rem auto auto;
+    }
+  }
+
   .grw-nav-slide-hr {
     border-top: 0rem;
     border-bottom: 3px solid;

+ 5 - 0
src/client/styles/scss/theme/_apply-colors.scss

@@ -284,6 +284,11 @@ pre:not(.hljs):not(.CodeMirror-line) {
   .nav-title {
     color: $color-link;
   }
+  .modal-header {
+    button.close {
+      color: $secondary;
+    }
+  }
   .nav-link svg {
     fill: $color-link;
   }