-
Notifications
You must be signed in to change notification settings - Fork 4
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Mm 005 header v2 #7
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Really great work - super clear to read and nice testing
I have a couple of formatting comments but otherwise it's looking great - please let me know when it's ready for re-review
src/App.tsx
Outdated
function App() { | ||
return ( | ||
<div className="App"> | ||
<Router> | ||
<h1>MarsioKart</h1> | ||
<div><Header/></div> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If you remove the <div>
s from this line and just have <Header/>
it will be easier for a screen reader to see what is happening and translate it - please can you do that?
src/Hamburgerbutton.scss
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is looking great and is nice and clear - in order to prevent the src folder from being too full of separate files (and therefore harder to read through) please can you create a Hamburgerbutton folder and put this file and Hamburgerbusston.tsx in it
src/header.tsx
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As above - please can you also create a Header folder and put this file in it
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Really good work
Description of changes -
added a button in the style of hamburger in header
when button is clicked, outputs msg to console log
added unit test for onclick
Screenshots - for changes to the UI
Things to Check