Responsive art gallery website with customised calendar

Community feedback

• 

  P

  Christopher Adolphe•620

  @christopher-adolphe

  Posted about 3 years ago

  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 and Back 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 a 404 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.

  00

  Marked as helpful