Differential D2337

[Kickoff] Fix being unable to reorder entries in favorites menu after scrolling down
ClosedPublic
Actions

Authored by net147 on Aug 2 2016, 2:26 PM.

Details

Reviewers
broulik
bshah
Group Reviewers
Plasma
Commits
R119:784b07f26ae8: [Kickoff] Fix being unable to reorder entries in favorites menu after…
Summary

BUG: 345741

Test Plan

I have many entries in the favorites menu. I can reorder the entries if the vertical scrollbar is at the top but if I scroll to the bottom of the favorites menu and try to reorder the items, it doesn't show a drop indicator for anywhere other than the end of the favorites list.

Steps to Reproduce:

  1. Add lots of applications to favorites such that less than half of the favorites are shown when you scroll to the top of the favorites menu list
  2. Scroll to the bottom of the favorites list
  3. Click and drag the 3rd-last item up one position

Diff Detail

Repository
R119 Plasma Desktop
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
net147 updated this revision to Diff 5641.Aug 2 2016, 2:26 PM
net147 retitled this revision from to [Kickoff] Fix being unable to reorder entries in favorites menu after scrolling down.
net147 updated this object.
net147 edited the test plan for this revision. (Show Details)
net147 added reviewers: broulik, Plasma.
net147 set the repository for this revision to R119 Plasma Desktop.
Restricted Application added a project: Plasma. · View Herald TranscriptAug 2 2016, 2:26 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
net147 added a comment.Aug 2 2016, 2:30 PM

This is a backport of https://phabricator.kde.org/D2310 fix for Plasma 5.7.

net147 added a reviewer: bshah.Aug 2 2016, 2:33 PM
broulik accepted this revision.Aug 8 2016, 7:58 AM
broulik edited edge metadata.
This revision is now accepted and ready to land.Aug 8 2016, 7:58 AM
Closed by commit R119:784b07f26ae8: [Kickoff] Fix being unable to reorder entries in favorites menu after… (authored by net147, committed by bshah). · Explain WhyAug 8 2016, 9:41 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 5736

View Options

applets/kickoff/package/contents/ui/FavoritesView.qml

Loading...