medvedych/harbour-fernschreiber
2
0
Fork 1
Code Issues Pull requests 1 Projects Releases Packages Wiki Activity

Merge pull request #493 from monich/sessions

Browse source 
Made sure that context menu stays inside the screen area
This commit is contained in:
Sebastian Wolf 2022-05-01 23:50:34 +02:00 committed by GitHub
commit 2c53848d04
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 11 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

18
qml/components/settingsPage/SettingsSession.qml
View file

@ -25,6 +25,7 @@ import "../../js/functions.js" as Functions
AccordionItem {
text: qsTr("Sessions")
property SilicaFlickable flickable: parent.flickable
Component {
Column {
id: activeSessionsItem
@ -63,11 +64,7 @@ AccordionItem {
SilicaListView {
id: activeSessionsListView
width: parent.width
// one activeSessionListItem is about 1.52 times itemSizeLarge
// show max 5 items at a time
height: Theme.itemSizeLarge * 1.5 * Math.min(5 , activeSessionsItem.activeSessions.length )
clip: true
height: contentHeight
model: activeSessionsItem.activeSessions
headerPositioning: ListView.OverlayHeader
header: Separator {
@ -82,6 +79,15 @@ AccordionItem {
menu: ContextMenu {
hasContent: !modelData.is_current
onHeightChanged: {
if (parent && flickable) {
// Make sure we are inside the screen area
var bottom = parent.mapToItem(flickable, x, y).y + height
if (bottom > flickable.height) {
flickable.contentY += bottom - flickable.height
}
}
}
MenuItem {
onClicked: {
var sessionId = modelData.id;
@ -172,8 +178,6 @@ AccordionItem {
}
}
VerticalScrollDecorator {}
}
}
}