zzag added inline comments. INLINE COMMENTS
> main.qml:130 > + (pagerModel.currentPage + 1) % repeater.count : > + Math.max(pagerModel.currentPage + 1, repeater.count); > + pagerModel.changePage(nextPage); I think it should be `Math.min`. > main.qml:135 > + (repeater.count + pagerModel.currentPage - 1) % > repeater.count : > + Math.min(pagerModel.currentPage - 1, 0); > + pagerModel.changePage(previousPage); Math.max(pagerModel.currentPage - 1, 0) REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D14988 To: phuongn, #plasma, #vdg, davidedmundson Cc: zzag, ngraham, abetts, davidedmundson, broulik, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, sebas, apol, mart
