D25923: [KCMs/Night Color] Remove "manual location entry" mode

Nathaniel Graham noreply at phabricator.kde.org
Thu Dec 12 15:30:10 GMT 2019


ngraham created this revision.
ngraham added reviewers: VDG, Plasma, davidedmundson.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
ngraham requested review of this revision.

REVISION SUMMARY
  This mode allows you to set Night Color to trigger based on the sunset/sunrise times of an
  arbitrary latitude/longitude location. However this is completely pointless; there is no
  conceivable use case for setting the location to any place other than your current
  location. The only reason why you would want to enter your current location manually is
  if the automatic geolocation feature is broken. However if this is the case, it is much
  faster to just enter start/end times manually than it is to look up your latitude and
  longitude and enter them into the text fields here.
  
  Therefore, this patch removes that mode from the KCM and requires that people who don't
  use the automatic geolocation feature enter the start/end times manually.
  
  In the future, we could even remove the geolocator and base the location on the current
  time zone so we can bypass failures of the geolocator.

TEST PLAN
  Modes now: F7814125 <https://phabricator.kde.org/F7814125>

REPOSITORY
  R119 Plasma Desktop

BRANCH
  remove-manual-location-entry-mode (branched from master)

REVISION DETAIL
  https://phabricator.kde.org/D25923

AFFECTED FILES
  kcms/nightcolor/package/contents/ui/LocationsFixedView.qml
  kcms/nightcolor/package/contents/ui/main.qml

To: ngraham, #vdg, #plasma, davidedmundson
Cc: plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20191212/168fd88a/attachment-0001.html>


More information about the Plasma-devel mailing list