Improve scrolling behaviour of emoji picker on desktop, show scrollbars for categories

Signed-off-by: Claudio Cambra <claudio.cambra@gmail.com>
This commit is contained in:
Claudio Cambra
2022-10-12 14:20:47 +02:00
committed by Tobias Fella
parent 47a4e5c447
commit dc8b68931e

View File

@@ -20,14 +20,13 @@ ColumnLayout {
spacing: 0 spacing: 0
ListView { ScrollView {
Layout.fillWidth: true Layout.fillWidth: true
Layout.preferredHeight: Kirigami.Units.gridUnit * 2 + 2 // for the focus line Layout.preferredHeight: Kirigami.Units.gridUnit * 2 + ScrollBar.horizontal.height + 2 // for the focus line
ScrollBar.horizontal.height: ScrollBar.horizontal.visible ? ScrollBar.horizontal.implicitHeight : 0
boundsBehavior: Flickable.DragOverBounds
ListView {
clip: true clip: true
orientation: ListView.Horizontal orientation: ListView.Horizontal
model: ListModel { model: ListModel {
@@ -77,22 +76,22 @@ ColumnLayout {
onClicked: emojiCategory = category onClicked: emojiCategory = category
} }
} }
}
Kirigami.Separator { Kirigami.Separator {
Layout.fillWidth: true Layout.fillWidth: true
Layout.preferredHeight: 1 Layout.preferredHeight: 1
} }
GridView { ScrollView {
Layout.fillWidth: true Layout.fillWidth: true
Layout.preferredHeight: Kirigami.Units.gridUnit * 8 Layout.preferredHeight: Kirigami.Units.gridUnit * 8
Layout.fillHeight: true Layout.fillHeight: true
GridView {
cellWidth: Kirigami.Units.gridUnit * 2 cellWidth: Kirigami.Units.gridUnit * 2
cellHeight: Kirigami.Units.gridUnit * 2 cellHeight: Kirigami.Units.gridUnit * 2
boundsBehavior: Flickable.DragOverBounds
clip: true clip: true
model: { model: {
@@ -159,7 +158,6 @@ ColumnLayout {
emojiModel.emojiUsed(modelData) emojiModel.emojiUsed(modelData)
} }
} }
}
ScrollBar.vertical: ScrollBar {}
} }
} }