search

kotelnik / plasma-applet-redshift-control

Plasma 5 applet for controlling redshift (screen temperature modifying). It allowes basic redshift settings and fast manual temperature control by mouse wheel.
GNU General Public License v2.0
66 stars 12 forks source link

Allow to disable geoclue #33

Closed akien-mga closed 7 years ago

akien-mga commented 7 years ago

My system usually starts without networking (due to a misconfiguration somewhere, I need to give kwallet my password before it starts the Wi-Fi, but that's another topic). Anyway, it makes Redshift Control fail at each Plasma session start since geoclue can't find the network.

It would be nice if the geolocation could be made optional in the settings, and when disabled, only the manually specified coordinates would be used.

kotelnik commented 7 years ago

I believe that automatic geolocation (geoclue) is already optional. In fact it is disabled by default. Check out Redshift Control Settings, section Advanced -> Location. There is also a button "Locate" to manually determine and set your location (click it and wait a moment), then you can click Apply and redshift will use that location.

akien-mga commented 7 years ago

That works, yes, but it still runs geoclue2 on startup apparently. I can take a screenshot of the error dialog tomorrow when I boot my laptop :)

akien-mga commented 7 years ago

Here's the error dialog I get when starting plasma without WiFi connection: spectacle w10093

Even then, Redshift Control still starts and works fine, but this error is annoying :)

ArchangeGabriel commented 7 years ago

That’s very strange. I never have any connection on startup (by design/choice), but I don’t have this issue.

However, I have the issue that redshift (sometimes?) doesn’t seems to pick my location once network is back. But maybe that’s because I need to configure it.

akien-mga commented 7 years ago

Hm, I think this error actually comes from redshift-gtk... /me hides in shame.

It just appeared when I started using Redshift Control instead, I guess having the GTK+ systray util and the Plasma applet at the same time causes some conflicts. Anyway, not a bug :)