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
