Gundi's IP Address Tracker

Community feedback

• 

  P

  Teodor Jenkler•3,760

  @TedJenklerPosted 3 months ago

  Hi @Martinsgundi,

  Nice project! I like the extra features you've added. To remove the Leaflet ZoomControl on mobile screens using ReactJS, you can use the useEffect hook to detect the screen size and conditionally remove the control (another option is to create a custom hook). Alternatively, you might consider removing the buttons entirely, as they're not needed when users can zoom in with the mouse (if configured correctly).

  I also noticed that your site isn't fully responsive. Depending on the address, the layout can break. Consider shortening the IP or using "..." if it overflows or causes issues on the page.

  Overall, great work with just a few areas to refine, Hope this feedback was helpful

  Best regards, Teodor

  Marked as helpful

  0

  ---

  

  Martinsgundi•380

  @MartinsgundiPosted 2 months ago

  Hey Teodor,

  Thank you so much for your feedback—I really appreciate it.

  I recently discovered from the LeafletJs doc that users can zoomOut by holding the Shift key and left-clicking twice. It was the lack of this info that made me add the zoomControl button in the first place. So, definitely, the best and most efficient choice was to remove the zoomControl button.

  To improve the user experience, I added an info (ℹ️) notification to inform desktop users about how to zoomOut.

  Regarding responsiveness, I didn’t make any significant changes—just adjusted some padding and widths here and there. However, if you notice any differences on your end, please let me know.

  Thanks again!

  1