status-im
/
status-desktop
mirror of https://github.com/status-im/status-desktop.git
2
0
Fork 0
Code Issues Projects Releases Wiki Activity

fix(StatusEmojiPopup): remove unnecessary spacing 

Fixes #5433
This commit is contained in:
Pascal Precht 2022-04-12 15:47:23 +02:00 committed by r4bbit.eth
parent 59631cad8f
commit b72f53e62c
2 changed files with 3 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
ui/imports/shared/status/StatusEmojiPopup.qml
View File

@ -276,7 +276,6 @@ Popup {
property int activeCategory: 0
id: scrollView
topPadding: Style.current.smallPadding
leftPadding: Style.current.smallPadding
rightPadding: Style.current.smallPadding / 2
Layout.fillWidth: true
@ -307,7 +306,7 @@ Popup {
var totalHeight = 0
var categoryHeights = []
for (let i = 0; i < emojiSectionsRepeater.count; i++) {
totalHeight += emojiSectionsRepeater.itemAt(i).height + Style.current.padding
totalHeight += emojiSectionsRepeater.itemAt(i).height
categoryHeights.push(totalHeight)
}
var ratios = []
@ -316,7 +315,7 @@ Popup {
})
categrorySectionHeightRatios = ratios
return totalHeight + Style.current.padding
return totalHeight
}
Repeater {

2
ui/imports/shared/status/StatusEmojiSection.qml
View File

@ -19,7 +19,7 @@ Item {
visible: emojis.length > 0 || !!(modelData && modelData.length && modelData[0].empty && searchString === "")
anchors.top: index === 0 ? parent.top : parent.children[index - 1].bottom
anchors.topMargin: index === 0 ? 0 : Style.current.padding
anchors.topMargin: 0
width: parent.width
// childrenRect caused a binding loop here