Differential D14354

Improve weather widget usability
AbandonedPublic
Actions

Authored by underwit on Jul 25 2018, 7:14 AM.

Details

Reviewers
kossebau
Group Reviewers
Plasma
Summary

I changed the weather widget setting for better usability.
I removed the search button. I chose the default station and added a live search when typing.

Diff Detail

Repository
R114 Plasma Addons
Lint
Lint Skipped
Unit
Unit Tests Skipped
underwit created this revision.Jul 25 2018, 7:14 AM
Restricted Application added a subscriber: plasma-devel. · View Herald TranscriptJul 25 2018, 7:14 AM
underwit requested review of this revision.Jul 25 2018, 7:14 AM
underwit updated this revision to Diff 38392.Jul 25 2018, 9:32 AM

Perform a search when changing the station list

kossebau added a comment.Jul 25 2018, 9:46 AM

Thanks for the patch, I agree with you that live searching is better usability.

So far though I had stayed away from implementing that, as I have no idea how the weather service providers like their servers getting more hits once in a while .

I would prefer if we could first check the terms of usage of the currently used weather services before implementing this (where needed, IIRC for some there is a static list downloaded).

kdeplasma-addons/applets/weather/package/contents/ui/config/ConfigWeatherStation.qml
72

Defaulting to the private service wetter.com goes against the idea of keeping the user in control of their privacy by default.

I know the current UX sucks, that popup menu is broken by design. But this is not a fix I can support.

underwit abandoned this revision.Aug 7 2018, 2:02 PM

Revision Contents
Changeset List

Diff 38392

View Options

kdeplasma-addons/applets/weather/package/contents/ui/config/ConfigWeatherStation.qml

Loading...