NW6-ZelihaPala-Bike-Project-html-week-1 #468
Conversation
✅ Deploy Preview for cyf-bfr ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
|
|
||
| <div> | ||
| <ul class="navigation__list"> | ||
| <li class="navigation__item"> |
There was a problem hiding this comment.
these navigation links all look lumped together, do you think you can add space between them using css?
Check out padding through the MDN docs to have an idea on what to do
There was a problem hiding this comment.
Also, the colour of the links are meant to become orange on hover, please make that happen
styles/style.css
Outdated
| button:hover { | ||
| background-color: rgb(55, 178, 235); | ||
| } | ||
| .orange { |
There was a problem hiding this comment.
instead of .orange can you use a css class name like .btn-orange do the same for .white in like 123>
Its a better naming convention
Adjust your PR title to fit the format also |
|
|
||
| <button>Donate Now</button> | ||
|
|
||
| <button>Donate Now</button> |
There was a problem hiding this comment.
on hover of the button, it becomes blue, can you make it orange as required, thanks
There was a problem hiding this comment.
Thank you for your help. I have reviewed and made corrections to the areas you pointed out. If you find something Idid not do correctly just let me know. Thank you for your time
There was a problem hiding this comment.
When I hover on this button, the background should be white and the text should be orange, see a picture of what it should look like
zelihapala
changed the title
zelihapala
changed the title
zelihapala
changed the title
| </h3> | ||
| </div> | ||
| <div class="sidebar"> | ||
| <h2 class="heading-underline">Upcoming events</h2> |
There was a problem hiding this comment.
There is a space before the text, can you remove this? you might need to check on the padding you are using at the moment
There was a problem hiding this comment.
would you mind checking the project again please? Thank you!
NW6-Zeliha Pala- Bike project-html-week-1