Skip to content

11 drawer component#12

Open
vanpertsch wants to merge 6 commits intohenrymoews:masterfrom
vanpertsch:11-Drawer-to-MainMenu-component
Open

11 drawer component#12
vanpertsch wants to merge 6 commits intohenrymoews:masterfrom
vanpertsch:11-Drawer-to-MainMenu-component

Conversation

@vanpertsch
Copy link

Hallo @henrymoews,
i changed the menu in the MainMenu to drawer.

Copy link
Owner

@henrymoews henrymoews left a comment

Choose a reason for hiding this comment

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

Hi Vanessa,
könntest du nochmal schauen, ob mal wirklich den State für left, right, top und bottom braucht? Ich habe das Gefühl, dass das die Klasse unnötig kompliziert macht.


export default function TemporaryDrawer() {
const classes = useStyles()
const [state, setState] = React.useState({
Copy link
Owner

Choose a reason for hiding this comment

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

Planst du den Drawer an verschiedenen Seiten anzubringen? Würde hier nicht als default links reichen bzw. einfach "isOpen" o.ä.?


return (
<div>
{['left'].map((anchor) => (
Copy link
Owner

Choose a reason for hiding this comment

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

Hier steht ja sowieso nur "left", dann braucht man das Array ja eigentlich nicht.

@vanpertsch
Copy link
Author

Hallo @henrymoews, ich habe "left" als default gesetzt und es funktioniert noch. :).
Leider war ich in einer kleinen git Hölle und bin mir nicht sicher ob mein (versehentliches) mergen des remote-tracking branches jetzt ein Problem darstellt. Ich hoffe nicht.

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