[Differential] [Request, 3 lines] D4648: try harder to make the selected wallpaper visible

[Marco Martin]

mart created this revision.
mart added a reviewer: Plasma.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.

REVISION SUMMARY
  qml gridview don't have an official way to ensure the
  current index is visible when the view is first instantiated,
  the only way we have is to do positionViewAtIndex when we know the
  proper space has already been allocated (Component.onCompleted
  is way too early) we used to do that on onCountChanged, but that
  too seems to early sometimes, as the selected wallpaper
  is visible only sometimes, attaching it to onContentHeightChanged
  seems to be more reliable (for all branches)

TEST PLAN
  proper wallpaper is visible every time the window opens

REPOSITORY
  R120 Plasma Workspace

BRANCH
  phab/scrollHack

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

AFFECTED FILES
  wallpapers/image/imagepackage/contents/ui/config.qml

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

To: mart, #plasma
Cc: plasma-devel, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20170217/4b1fcbe5/attachment-0001.html>