Skip to content

Conversation

@lmarini
Copy link
Member

@lmarini lmarini commented Oct 10, 2022

Depends on #108 . Please merge that first.

New Layout component to wrap pages using the MUI Drawer. All new pages should include this Layout.

Fixed bug in link to registration.

TODO: store open/closed variable for drawer in redux store. Right now the drawer always shows closed after switching pages.

Screen Shot 2022-10-10 at 11 58 20 AM

Copy link
Member

@longshuicy longshuicy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

At any appropriate time, maybe we can run reformat code to get rid of the unused import?

image

import {RootState} from "./types/data";
import {resetLogout} from "./actions/common";
import Layout from "./components/Layout";
import {Paragraph} from "./components/Paragraph";
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think Paragraph is being used

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same as Layout

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done. Thanks for the reminder. I try to do it when I remember, but we don't have an automatic way to do this. Also many of my files show in red because of the import React, {useEffect, useState} from "react"; import at the top. React is never used and Typescript doesn't like that.

<div className="inner-container">
<Grid container spacing={4}>
<Grid item xs>
<Grid item>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I still saw the same issue
image

I think last time you tried xs to fix it; but this time it won't work for me

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I made a new file called Explore.tsx to make it clear that that is the explore page. I fixed it there and remove Dasboard.tsx file to not create confusion.

@lmarini lmarini requested a review from longshuicy October 10, 2022 20:22
@longshuicy longshuicy merged commit aae305c into main Oct 10, 2022
@lmarini lmarini deleted the drawer_menu branch November 4, 2022 15:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants