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

Andrius 130

@AndriusM5

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


Let me know what you think about my solution to this challenge!

Community feedback

Account Deleted

Hello Andrius,

Congratulations on your result, for the most part it is very good! Here is some quick feedback:

  1. Your solution only accepts an IP address and not a domain name as requested from the brief

  2. You shouldn't be using the deprecated var keyword in your js files, except under very, very specific circumstances. Use let or const instead. This is because var is not block scoped and as such can be changed anywhere from your application, for this application it's probably not a big deal, but in others, it could cause huge potential headaches, so it would best to break this bad habit now.

  3. You don't need to declare row on a Flex container unless you are changing it from a column in a media query for instance, by default Flex containers are set to row, in a large application this could save many lines of code and make your code less verbose.

Apart from that good job!

If you found this helpful please mark as helpful,

Thank you

1

Andrius 130

@AndriusM5

Posted

@ashmaddenweb Thank you for your feedback, added search by domain functionality! More updates coming soon.

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