-
-
Notifications
You must be signed in to change notification settings - Fork 786
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
Nw6/nohe/html css/bike for refugee/week1 #481
base: main
Are you sure you want to change the base?
Nw6/nohe/html css/bike for refugee/week1 #481
Conversation
…guidance Update git guidances for coursework submission
✅ Deploy Preview for cyf-bfr ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
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.
Well done.
Please rename your PR to the format required and address the other comments I left.
@@ -30,7 +29,7 @@ | |||
/> | |||
</a> | |||
|
|||
<div> | |||
<nav class="nav"> | |||
<ul class="navigation__list"> | |||
<li class="navigation__item"> | |||
<a class="navigation__link" href="#">Home</a> |
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.
<button>Donate a bike today</button> | ||
<button>Volunteer</button> | ||
<button class="donate-button">Donate a bike today</button> | ||
<button class="volunteer-button">Volunteer</button> |
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.
i added space for learn-more section and added hover for nav