P
Christopher Adolphe• 580
@christopher-adolphe
Posted
Hello @Hannah-Ogunyinka,
You did a good job on the overall for this challenge. 👍 You have handle the website's responsiveness well.👌 For the styling of the links, I wouldn't have thought of achieving this with a gradient. Well done! 👍 Below are a few things I have noticed. You might want to check them in order to improve your solution.
- I have viewed the solution on Chrome browser and the
<h1>
is not visible. At the moment, I see a white and black rectangle while on Safari and Firefox it is working fine. Maybe you should find a fallback that works on Chrome. - For the links
Our location
andBack to Home
, please remove the<button>
element from inside the<a>
element and move the styles you have applied to the<a>
element itself. It would be even better if you add a class like.btn
which will have all these styles that you could then apply to these links. - Please do check the
href
attributes on the links as well because at the moment, I get a404
error. Try to change<a href="/location.html">
to<a href="location.html">
. - Lastly, if you use
<h2>
elements instead of<h3>
in the<main>
section, you could get rid of this accessibility warning.
I'm currently tackling this challenge. I felt like I was cheating while doing this review 😂
I hope this helps.
Keep it up.
Marked as helpful
0