D4962: [Folder View] Open context menu when pressing Menu key

2017-03-07 Thread Kai Uwe Broulik
This revision was automatically updated to reflect the committed changes. Closed by commit R119:895f617f0887: [Folder View] Open context menu when pressing Menu key (authored by broulik). REPOSITORY R119 Plasma Desktop CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D4962?vs=12256&id=1

D4962: [Folder View] Open context menu when pressing Menu key

2017-03-07 Thread Eike Hein
hein accepted this revision. This revision is now accepted and ready to land. REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D4962 To: broulik, #plasma, hein Cc: plasma-devel, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol

D4962: [Folder View] Open context menu when pressing Menu key

2017-03-07 Thread Kai Uwe Broulik
broulik updated this revision to Diff 12256. broulik edited the test plan for this revision. broulik added a comment. - Use GridView currentItem - Cannot reproduce placement problem, I might have forgotten to actually compile after changing it to use `currentItem.frame` :) The context menu n

D4962: [Folder View] Open context menu when pressing Menu key

2017-03-06 Thread Eike Hein
hein added a comment. Could you add a screenie for the pos problem you mentioned? REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D4962 To: broulik, #plasma, hein Cc: plasma-devel, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol

D4962: [Folder View] Open context menu when pressing Menu key

2017-03-06 Thread Kai Uwe Broulik
broulik edited the test plan for this revision. REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D4962 To: broulik, #plasma, hein Cc: plasma-devel, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol

D4962: [Folder View] Open context menu when pressing Menu key

2017-03-06 Thread Kai Uwe Broulik
broulik created this revision. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. REVISION SUMMARY If items are selected, show their context menu next to the first selected item, otherwise ignore the signal and let the containment handle it