Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

IP address tracker - html/css/js

David Parsons• 205

@Davros2014

Desktop design screenshot for the IP Address Tracker coding challenge

This is a solution for...

  • HTML
  • CSS
  • JS
  • API
3intermediate
View challenge

Design comparison


SolutionDesign

Solution retrospective


Feedback appreciated

Community feedback

@SebaFarias

Posted

I noticed that in the mobile layout, your trackerOutput panel overlaps the search bar, maybe this could help:

You should nest your trackerOutput panel inside your header section, that way, if you add "position: relative;" propety to the header and "top: 170px;"(or something around it) to the trackerOutput it should work fine on the mobile view.

I am asuming you made this for semantic reasons, not wanting to have a section nested inside another section but i think, in this case, that is fine.

Although by semantics maybe the titles "IP Address, Location, Timezone and ISP" should be h2 and the values an h3 or p, even if they are smaller.

0

David Parsons• 205

@Davros2014

Posted

@SebaFarias Hi, thanks for your input, makes total sense, not sure why I didn't do that, I didn't actually check the responsive aspects too much if I'm honest, probably would have picked it up sooner in that case. Headers make sense also. Thanks again!

0

Please log in to post a comment

Log in with GitHub
Discord logo

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord