Skip to content

Commit eefcd71

Browse files
committed
layout tweaks to match original
1 parent b217d40 commit eefcd71

3 files changed

Lines changed: 25 additions & 23 deletions

File tree

src/features/filters/FilterModal.jsx

Lines changed: 7 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -57,9 +57,9 @@ const FilterModal = ({ show, onHide, filters, onChange }) => {
5757
<Modal.Header closeButton>
5858
<Modal.Title>Other than fish, these are the things I&apos;m looking for in a Fish Fry:</Modal.Title>
5959
</Modal.Header>
60-
<Modal.Body>
60+
<Modal.Body className="p-4">
6161
<Row>
62-
<Col sm={{ span: 8, offset: 2 }}>
62+
<Col >
6363
{FILTER_FIELDS.map((filter) => (
6464
<Form.Group className="mb-3" key={filter.key}>
6565
<h4>
@@ -78,9 +78,9 @@ const FilterModal = ({ show, onHide, filters, onChange }) => {
7878
))}
7979
</Col>
8080
</Row>
81-
<hr />
81+
8282
<Row>
83-
<Col sm={{ span: 8, offset: 2 }} className="bg-info-subtle rounded py-2">
83+
<Col className="bg-primary rounded py-2 my-2">
8484
<Form.Check
8585
className="filter"
8686
id="publish"
@@ -92,14 +92,12 @@ const FilterModal = ({ show, onHide, filters, onChange }) => {
9292
</Col>
9393
</Row>
9494
<Row>
95-
<Col sm={{ span: 8, offset: 2 }}>
96-
<h4>
97-
<small>
95+
<Col >
96+
<p>
9897
Note that while we try our best to verify all details about Fish Fries, some details may be
9998
incomplete. Please check with the Fish Fry venue ahead of time and rely on the above filters at your
10099
own discretion.
101-
</small>
102-
</h4>
100+
</p>
103101
</Col>
104102
</Row>
105103
</Modal.Body>

src/features/layout/Sidebar.jsx

Lines changed: 10 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -18,11 +18,11 @@ const Sidebar = ({
1818
<div className="sidebar-wrapper">
1919
<Card id="features" className="h-100 border-0 rounded-0">
2020
<Card.Header>
21-
<h3 className="h5 m-0 d-flex align-items-center justify-content-between">
21+
<h3 className="h5 mt-3 d-flex align-items-center justify-content-between">
2222
<span>Fish Fry Filter</span>
2323
<Button
2424
type="button"
25-
variant="outline-secondary"
25+
variant="primary"
2626
size="sm"
2727
id="sidebar-hide-btn"
2828
onClick={onHide}
@@ -31,19 +31,20 @@ const Sidebar = ({
3131
</Button>
3232
</h3>
3333
</Card.Header>
34-
<Card.Body>
34+
<Card.Body className="px-0">
35+
<div className="mx-2">
3536
<Button
3637
type="button"
3738
size="sm"
38-
variant={hasActiveFilters ? "primary" : "outline-secondary"}
39+
variant={hasActiveFilters ? "primary" : "outline-primary"}
3940
className="w-100"
4041
id="filterSidebar-btn"
4142
onClick={onOpenFilter}
4243
>
4344
<FontAwesomeIcon icon={faFilter} /> {hasActiveFilters ? "Filtered" : "Filter"}
4445
</Button>
45-
</Card.Body>
46-
<div className="sidebar-table">
46+
</div>
47+
<div className="sidebar-table mt-3">
4748
<Table hover size="sm" id="feature-list">
4849
<thead className="visually-hidden">
4950
<tr>
@@ -73,7 +74,9 @@ const Sidebar = ({
7374
))}
7475
</tbody>
7576
</Table>
76-
</div>
77+
</div>
78+
</Card.Body>
79+
7780
</Card>
7881
</div>
7982
</div>

src/features/layout/TopNav.jsx

Lines changed: 8 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -43,17 +43,15 @@ const TopNav = ({
4343
<Navbar.Brand className="text-white fw-bold fs-6 d-md-none">Fish Fry Map</Navbar.Brand>
4444

4545
<Navbar.Collapse>
46-
<Form className="ms-auto fishfry-search-form" role="search" onSubmit={(event) => event.preventDefault()}>
47-
<SearchBox {...searchProps} />
48-
</Form>
49-
<Nav className="ms-md-3">
46+
47+
<Nav className="ms-auto">
5048
<Nav.Item className="d-none d-md-block">
51-
<Nav.Link as="button" type="button" id="list-btn" onClick={onToggleSidebar} className="text-white">
49+
<Nav.Link as="button" type="button" id="list-btn" onClick={onToggleSidebar} >
5250
<FontAwesomeIcon icon={faFilter} className="white" />&nbsp;&nbsp;Filter
5351
</Nav.Link>
5452
</Nav.Item>
5553
<Nav.Item>
56-
<Nav.Link as="button" type="button" id="about-btn" onClick={onOpenAbout} className="text-white">
54+
<Nav.Link as="button" type="button" id="about-btn" onClick={onOpenAbout}>
5755
<FontAwesomeIcon icon={faCircleQuestion} className="white" />&nbsp;&nbsp;About
5856
</Nav.Link>
5957
</Nav.Item>
@@ -63,7 +61,7 @@ const TopNav = ({
6361
target="_blank"
6462
rel="noreferrer"
6563
id="data-btn"
66-
className="text-white"
64+
6765
>
6866
<FontAwesomeIcon icon={faDatabase} />&nbsp;&nbsp;Data
6967
</Nav.Link>
@@ -80,6 +78,9 @@ const TopNav = ({
8078
</Nav.Link>
8179
</Nav.Item>
8280
</Nav>
81+
<Form className="fishfry-search-form" role="search" onSubmit={(event) => event.preventDefault()}>
82+
<SearchBox {...searchProps} />
83+
</Form>
8384
</Navbar.Collapse>
8485
</Container>
8586
</Navbar>

0 commit comments

Comments
 (0)