chore: code review fixes

jrainville · jrainville · commit f76f22ee574d · 2025-10-30T14:01:14.000-04:00
diff --git a/ui/imports/shared/controls/chat/MessageReactionsRow.qml b/ui/imports/shared/controls/chat/MessageReactionsRow.qml
@@ -36,6 +36,10 @@ Row {
             sourceModel: root.emojiModel
 
             filters: [
+                IndexFilter {
+                    // Only show the first 5 emojis
+                    maximumIndex: 4
+                },
                 AnyOf {
                     ValueFilter {
                         roleName: "skinColor"
diff --git a/ui/imports/shared/views/chat/MessageContextMenuView.qml b/ui/imports/shared/views/chat/MessageContextMenuView.qml
@@ -66,8 +66,8 @@ StatusMenu {
     }
 
     StatusMenuSeparator {
-        // FIXME there is a padding on each side
         visible: emojiRow.visible && !root.disabledForChat
+        horizontalPadding: 0
     }
 
     StatusAction {
@@ -150,6 +150,7 @@ StatusMenu {
                   editMessageAction.enabled ||
                   pinAction.enabled ||
                   markMessageAsUnreadAction.enabled)
+        horizontalPadding: 0
     }
 
     StatusAction {

Original file line number	Diff line number	Diff line change
`@@ -66,8 +66,8 @@ StatusMenu {`
`66`	`66`	`}`
`67`	`67`
`68`	`68`	`StatusMenuSeparator {`
`69`		`- // FIXME there is a padding on each side`
`70`	`69`	`visible: emojiRow.visible && !root.disabledForChat`
	`70`	`+ horizontalPadding: 0`
`71`	`71`	`}`
`72`	`72`
`73`	`73`	`StatusAction {`
`@@ -150,6 +150,7 @@ StatusMenu {`
`150`	`150`	`editMessageAction.enabled \|\|`
`151`	`151`	`pinAction.enabled \|\|`
`152`	`152`	`markMessageAsUnreadAction.enabled)`
	`153`	`+ horizontalPadding: 0`
`153`	`154`	`}`
`154`	`155`
`155`	`156`	`StatusAction {`