From 1e098a90c1d7bb76cdf25feb5c5acce71def8286 Mon Sep 17 00:00:00 2001 From: Audric Ackermann Date: Mon, 22 Aug 2022 16:42:20 +1000 Subject: [PATCH 1/2] bump to Session 1.9.2 --- package.json | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/package.json b/package.json index e7d52cbe6..90136bcb9 100644 --- a/package.json +++ b/package.json @@ -2,7 +2,7 @@ "name": "session-desktop", "productName": "Session", "description": "Private messaging from your desktop", - "version": "1.9.1", + "version": "1.9.2", "license": "GPL-3.0", "author": { "name": "Oxen Labs", From 866527de174168ef1ffc38b04965d970ec66a9be Mon Sep 17 00:00:00 2001 From: Audric Ackermann Date: Tue, 23 Aug 2022 10:37:29 +1000 Subject: [PATCH 2/2] fix: make sure a double click on the picker does not trigger reply msg --- .../MessageContentWithStatus.tsx | 25 +++++++++++-------- 1 file changed, 14 insertions(+), 11 deletions(-) diff --git a/ts/components/conversation/message/message-content/MessageContentWithStatus.tsx b/ts/components/conversation/message/message-content/MessageContentWithStatus.tsx index b0439bf5e..a069e5983 100644 --- a/ts/components/conversation/message/message-content/MessageContentWithStatus.tsx +++ b/ts/components/conversation/message/message-content/MessageContentWithStatus.tsx @@ -30,6 +30,7 @@ type Props = { dataTestId?: string; enableReactions: boolean; }; +// tslint:disable: use-simple-attributes const StyledMessageContentContainer = styled.div<{ direction: 'left' | 'right' }>` display: flex; @@ -66,17 +67,19 @@ export const MessageContentWithStatuses = (props: Props) => { const currentSelection = window.getSelection(); const currentSelectionString = currentSelection?.toString() || undefined; - // if multiple word are selected, consider that this double click was actually NOT used to reply to - // but to select - if ( - !currentSelectionString || - currentSelectionString.length === 0 || - !currentSelectionString.includes(' ') - ) { - void replyToMessage(messageId); - currentSelection?.empty(); - e.preventDefault(); - return; + if ((e.target as any).localName !== 'em-emoji-picker') { + if ( + !currentSelectionString || + currentSelectionString.length === 0 || + !/\s/.test(currentSelectionString) + ) { + // if multiple word are selected, consider that this double click was actually NOT used to reply to + // but to select + void replyToMessage(messageId); + currentSelection?.empty(); + e.preventDefault(); + return; + } } };