SakonBel• 145
@SakonBel
Submitted
Enjoy my ip-tracker site!! And please give any feedback on how I can improve or some error that you found so that I can fix it. Thanks for your feedback!!
Skanthan
@skanthanvijay
All comments
- Skanthan• 295
@skanthanvijay
Posted
Very solid solution! The one thing I'd fix is to remove the outline when you click on the text input, it doesn't fit in with the theming. Other than that, great job
Marked as helpful
1 - Rohail Taha• 355
@rohailtaha
Submitted
How can we hide the API key using environment variables on client side. Any idea? Do we need to code a server to do this?
Skanthan• 295@skanthanvijay
Posted
The most common way to hide API keys would be to do it as a fullstack app with environment variables off-client. For the Mapbox API key however, you can just restrict its URL to where you host the page so that others cant use it even if they see it.
Marked as helpful
1 - DEEP PATEL• 395
@DeepPatel6128
Submitted
Skanthan• 295@skanthanvijay
Posted
The theme toggles don't work on your calculator. To make them work, I used a fieldset with radio buttons to make the toggle itself and used some javascript with an event listener for the clicking of the radio buttons to change theme.
0 - Skanthan• 295
@skanthanvijay
Submitted
Skanthan• 295@skanthanvijay
Posted
Notes to those seeing the solution (OUTDATED):
-The radio buttons do not have labels as they are supposed to be invisible. If there is a more efficient way to create the toggle however (potentially using libraries like jQuery), that would be awesome to hear
-There are spaces between the numbers above the toggle, but they do not show up on the generated screenshot
EDIT: The code for the toggle was revamped, the above notes can be ignored
0
