-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
New ideas #1
Comments
hanks for considering my ideas. If you had free time (and wanted to do this), could you improve the script to have a better Cumulative Layout Shift result? It's very bad performance on mobile? Check on https://pagespeed.web.dev if you want. Google doesn't like bad performance ;/ Thank you once again |
... and I found 1 issue too... when i choose some city the hourly hours show my local time, not local time the city I choose |
Thank you for taking the time to test it out. Your insights have been invaluable in enhancing the project, the current version is not the final product, as I am still working on refining it. I welcome any additional suggestions or ideas you may have, as I am always open to improving and iterating upon the project. (didn't notice the hourly hour thing 🤭 thx for mentioning it ) |
If you have time you can add Precipitation with mm (metric), inch (imperial) and Precipitation with % below the Humidity and Wind in Daily windows. It can be helpful for users. I think the WeatherAPI shares these data. Can you try add async or defer to the javascripts? In Pagespeed there is a info: "Reduce unused JavaScript and defer loading scripts until they are required to decrease bytes consumed by network activity" for:
I don't know if it help but you can try add async or defer to the javascripts. Thanks |
Thank you for changes. Sorry to bother you again, but I noticed some issues.
I didn't check the rest of the icons.
And here are probably the solutions:
Here's the app I sent you earlier and it doesn't have this problem (https://my-weather-app-wm.netlify.app/). It searches for a location automatically without asking for the user's consent, and that's exactly what it's all about. In this app Google doesn't see a problem. This app uses the https://ipwhois.io/ API, but I don't recommend it because it doesn't point to a proper location. I'll just show you that you can do it. Thank you once again and sorry for my English |
Hello again, I have one more request for you. Could you add Next Previous buttons to the script, because on desktops it is difficult to scroll using keyboard arrows. It's better to do it by clicking the buttons. Please add these buttons to Hourly and Daily forecast (if it is a 14-day forecast) Thank you |
Nice app, but I have a few ideas:
Is it possible that after entering the website it will automatically load the location (without permission because Google does not like applications forcing permissions?) Is it posible that after refreshing, the page remembers its previous location? (like here: https://my-weather-app-wm.netlify.app/)
Is it possible to add (for hours and per day) data such as: Wind, Humidity, Cloud Cover, Precipitation with mm and Precipitation with %, Percipation Hours, (and AQI, UV Index not necessarily)? More data more better for users.
Is it possible to add swipe buttons so that the swipe bar is not visible? (like here https://misterf1x.github.io/WeatherAPP/)
Sorry for my english. That's all for now.
Is it possible when you change the units from Celsius to Fahrenheit also change the wind speed units from kmp/h to mp/h?
Thank you
The text was updated successfully, but these errors were encountered: