-
Notifications
You must be signed in to change notification settings - Fork 78
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix(gif popup): close the popup after sending a GIF #6364
fix(gif popup): close the popup after sending a GIF #6364
Conversation
@@ -35,7 +35,7 @@ Rectangle { | |||
property var emojiPopup: null | |||
// Use this to only enable the Connections only when this Input opens the Emoji popup | |||
property bool emojiPopupOpened: false | |||
property bool closeGifPopupAfterSelection: false | |||
property bool closeGifPopupAfterSelection: true |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wonder why this was false in the first place..
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No idea... well it was introduced earlier to prevent a crash (due to adding a GIF to favs while the popup was being destroyed), but even more curious is the fact that this and the emoji popup have totally different logic for opening/closing/handling the image selected
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh and those callbacks 😮
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The boolean was introduced here: a70d800
Close #6083
What does the PR do
By default, enable automatically closing the GIF popup after having sent a GIF to the chat
Affected areas
Chat, chat input
Screenshot of functionality
Zaznam.obrazovky.z.5.7.2022.10.53.33.webm