Challenge completed with HTML|CSS|Leaflet JS|GRID and responsive.

Community feedback

• 

  Vanza Setia•27,775

  @vanzasetiaPosted over 2 years ago

  Hi, Vinicius Henrique! 👋

  Congratulations on finishing this challenge! 🎉 It's good to see you completing another challenge! 😄

  Nice work for the animation! 👍 But, I suggest removing the animation for the users who prefer not to see it. You can use prefers-reduced-motion media query to do that. Also, if you have read the modern CSS reset by Piccalilli (Andy Bell) then it has the code where it would disable all animations. I recommend using his CSS reset on every new project.

  I notice that when I visit the location page, it gives me an alert, "Geolocation error: User denied Geolocation.". Why this is happening? I use Leaflet JS as well but there's no such error though.

  You have done a good job on accessibility by having the :focus-visible styling for interactive elements and also having an aria-label for each social media link. 👏

  I hope this helps! Keep up the good work! 👍

  Marked as helpful

  1

  ---

  

  Vinicius Henrique•590

  @viniciusshenri96Posted over 2 years ago

  @vanzasetia Thanks, your advice always helps me, and the map I don't know why the error occurred, i followed the leafletjs tutorial, I'll try to solve it, I recently discovered that it's good to have a style for :focus-visible, now I'm putting it into practiceand I've also seen you do this in your projects, I like to focus on accessibility as well and reading your code ends up helping me!

  I will add prefers-reduced-motion to my project.

  0

  

  Vanza Setia•27,775

  @vanzasetiaPosted over 2 years ago

  @viniciusshenri96 Happy to help! 😄

  Amazing! It's great to know that you focus on creating accessible website. I am glad that my project helped you. 😊

  1