D14988: Implement option to toggle page navigation wraps around for pager plasmoid

[Andres Betts]

abetts added a comment.


  In D14988#316633 <https://phabricator.kde.org/D14988#316633>, @phuongn wrote:
  
  > Hi, it's just the default desktop changing effect. Activated by scrolling with the cursor hovering on the pager. I haven't changed the effect, just added an option to toggle page wrapping
  >  F6220920: simplescreenrecorder-2018-08-28_17.44.42.mp4 <https://phabricator.kde.org/F6220920>
  
  
  Thank you for the video. Just wanted to be sure I was understanding the patch. Looks good to me. I am not sure that the effect should be called "wrap around" since wrapping around evokes ideas about reaching a corner or a continuation of a sheet for example. What do you think of using a string that says something like:"
  
  "Enable desktop pager scrolling"
  "Enable desktop switch on pager"

REPOSITORY
  R119 Plasma Desktop

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

To: phuongn, #plasma, #vdg, davidedmundson
Cc: abetts, davidedmundson, broulik, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20180828/01647560/attachment.html>