D16425: Added new Suspend then Hibernate option

[Reverend Homer]

reverendhomer added a comment.


  Hi,
  
  In D16425#465325 <https://phabricator.kde.org/D16425#465325>, @avaldes wrote:
  
  > In D16425#465321 <https://phabricator.kde.org/D16425#465321>, @ngraham wrote:
  >
  > > UI looks good enough for now. But is this the full diff? It seems like something got lost. The whole patch should include the changes from all commits in your branch, not just the last one.
  >
  >
  > should be fixed now, I'm not sure what happened with arcanist.
  
  
  This patch doesn't seem to work for me. I have Arch Linux, powerdevil-5.15.5-1 and your patch applied. After clicking the check-box the "Apply" button doesn't get active. Therefore, this setting is not being saved.
  
  Also, why does the checkbox label say "While asleep, hibernate after 3 hours"? This value can be overriden in /etc/systemd/sleep.conf

REPOSITORY
  R122 Powerdevil

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

To: avaldes, broulik, ngraham
Cc: reverendhomer, meven, soriano, abalaji, graesslin, ngraham, plasma-devel, jraleigh, GB_2, ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20190516/f7fe4f30/attachment.html>