D25920: [KCMs/Night Color] Make activation criteria and times more obvious
Nathaniel Graham
noreply at phabricator.kde.org
Fri Dec 13 16:49:38 GMT 2019
ngraham added a comment.
In D25920#576966 <https://phabricator.kde.org/D25920#576966>, @abetts wrote:
> Suggestions:
>
> "To reduce eye strain, Night Color makes the colors on the screen warmer at the time of your choosing"
>
> Change to:
>
> "Night Color makes the screen colors warmer to reduce eye strain"
That eliminates the "at the time of your choosing", bit, without which I worry that users might mistakenly believe that this feature permanently tints the screen while on, missing the entire point that it does this on;y at certain times of day.
> "Night color temperature"
>
> Change to:
>
> "Temperature"
Or maybe even... "Color"?
> - Sunset/sunrise at manual location
>
> Change to:
> - Sunset to sunrise at different location
Other proposed changes seem fine, but this would make that feature misleading, I feat. My first iteration of this patch used "different location", but upon reflection and consultation with the developers, using a location other than your own makes no sense whatsoever. The point of the manual location entry is to enter your own location without needing to use the geolocator in the case that it's broken or you're very privacy-conscious. There are no valid user use cases to living in Australia and setting the location to Canada.
> Instead of having latitude/longitude + Detect Location, swap to Detect Location + Lat/long after.
Good idea!
> "Color change begins at XX:XX"
>
> Change to:
>
> Night color begins at XX;XX
>
> Color fully changed, might not be necessary? Too much detail?
>
> "Color begins changing back at XX:XX"
>
> Change to
>
> Night color ends XX:XX
>
> Normal coloration restored by XX:XX, might be too much info.
Because of the lengthy transition time, if we just list the time when it starts, people will get confused because they'll say, "hey, this thing says it's on, but I don't see any color change!" because the color has just started changing and has not finished the transition yet.
> Since the color changes gradually, maybe we could add an option to enable/disable that function? Something like
>
> [X] Gradual activation
> [X] Change colors gradually
There is, when using manual times mode. In that mode, you can choose the transition time. When using a location-based mode, this isn't user-selectable because the transition time is perfectly coordinated to come on when the sun starts to set and complete when it's dark. It doesn't make sense to let the user override that when in a location-based mode.
REPOSITORY
R119 Plasma Desktop
BRANCH
polish-nightcolor-ui (branched from master)
REVISION DETAIL
https://phabricator.kde.org/D25920
To: ngraham, #vdg, #plasma, davidedmundson, romangg
Cc: abetts, romangg, plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, sebas, apol, ahiemstra, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20191213/bbb36d3e/attachment-0001.html>
More information about the Plasma-devel
mailing list