Skip to content
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

Add Weather info #248

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from
Draft

Add Weather info #248

wants to merge 2 commits into from

Conversation

danirabbit
Copy link
Member

@danirabbit danirabbit commented Feb 7, 2021

Fixes #255

Todo:

  • add an icon to the panel?
  • Change background color with conditions
  • Figure out what to do in failure cases
  • Decide which information to show
  • Figure out what to do when it's not today
  • Use the correct location app ID

Screenshot from 2021-02-07 12 52 17@2x

@tintou
Copy link
Member

tintou commented Feb 8, 2021

Evolution Data Server already has a "Weather" calendar source that you can use natively without manually linking on libgweather

@4jNsY6fCVqZv
Copy link

How does the privacy of users stay protected in the context of this feature? Will it be enabled by default?

@hanaral
Copy link

hanaral commented Mar 14, 2021

How does the privacy of users stay protected in the context of this feature? Will it be enabled by default?

I presume this needs UX review anyway, like requesting access location before doing anything

@amka
Copy link

amka commented Jun 30, 2021

Even windows 11 shows the weather in the tray. That said the point "add an icon to the panel" required 😉

@andirsun
Copy link

Example of icon in panel.
Screenshot from 2021-08-17 11-15-00

@1280px
Copy link

1280px commented May 5, 2023

Is this PR still active?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Include weather info
8 participants