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
