[Differential] [Request, 10 lines] D2585: [Applet Configuration] Only scroll down if item is barely visible

[broulik (Kai Uwe Broulik)]

broulik created this revision.
broulik added a reviewer: Plasma.
broulik set the repository for this revision to rPLASMADESKTOP Plasma Desktop.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.

REVISION SUMMARY
  The logic for scrolling a focus item into view was mostly meant for keyboard-navigating through a list of CheckBoxes.
  However, an inner ScrollView - like plasma-pa has in its config dialog - can also gain activeFocus. This had us try to scroll the ScrollView into view, usually scrolling away other view contents.
  With the added logic we only try to scroll if the control is barely visible at the bottom but not if significant portions of it are already visible.

TEST PLAN
  I can still tab through task manager settings and have the view scroll properly.
  Switching tabs in plasma-pa applet config dialog no longer scrolls the tabs out of view.

REPOSITORY
  rPLASMADESKTOP Plasma Desktop

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

AFFECTED FILES
  desktoppackage/contents/configuration/AppletConfiguration.qml

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: broulik, #plasma
Cc: plasma-devel, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20160826/2f4e1023/attachment.html>