-
-
Notifications
You must be signed in to change notification settings - Fork 279
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
feat: show hourly forecast #455
feat: show hourly forecast #455
Conversation
This took me quite some trial and error to figure out. Signed-off-by: André Jaenisch <[email protected]>
I want to add an example_response.json to the repo and also respect day/night information as suggested in my last PR before I open this for inclusion. That being said, by opening a PR now I can invite others to check it out and run locally. Feedback welcome. |
NeoVim tried to be helpful. Signed-off-by: André Jaenisch <[email protected]>
Hi @Ryuno-Ki, Would you consider to [optionally] add popup on mouse hover? weather_widget:connect_signal("mouse::enter", function()
weather_widget:set_bg(beautiful.bg_focus)
weather_popup:move_next_to(mouse.current_widget_geometry)
end)
weather_widget:connect_signal("mouse::leave", function()
if weather_popup.visible then
weather_widget:set_bg('#00000000')
weather_popup.visible = not weather_popup.visible
end
end) |
This was a nice contribution! Co-authored-by: trap000d Signed-off-by: André Jaenisch <[email protected]>
Done. |
This should ease the job of updating this widget. Signed-off-by: André Jaenisch <[email protected]>
This was highly requested. I hope I catched all cases. Signed-off-by: André Jaenisch <[email protected]>
I think I have everythink now. I can see unrelated failures in luacheck to this extension. @streetturtle Mind taking a review on this? |
I removed the executable flag from all of them. Signed-off-by: André Jaenisch <[email protected]>
Good catch! |
I only need to add a prefix when it is not day. Signed-off-by: André Jaenisch <[email protected]>
This is great! Thank you! 👍 |
This took me quite some trial and error to figure out.