blue/squawk
blue
/
squawk
2
3
Fork 6
Code Issues 21 Pull Requests 2 Actions Releases 10 Wiki Activity

feature: paste image in chat #51

Merged
blue merged 3 commits from shunf4/squawk:feat/paste_img into messageFeed 2021-10-16 15:34:01 +00:00
Conversation 0 Commits 3 Files Changed 7 +86 -7
shunf4 commented 2021-10-06 16:35:15 +00:00
Contributor
Copy Link

Add feature: paste image in chat (context menu/Ctrl-V).

This PR is based on #50. The changes for this PR is at the lastest commit.

Add feature: paste image in chat (context menu/Ctrl-V). This PR is based on #50. The changes for this PR is at the lastest commit.
shunf4 added 17 commits 2021-10-06 16:35:15 +00:00
blue reviewed 2021-10-11 12:29:16 +00:00
ui/widgets/conversation.cpp
@ -84,2 +90,3 @@
m_ui->messageEditor->installEventFilter(&ker);
QAction* pasteImageAction = new QAction(tr("Paste Image"), this);
blue commented 2021-10-11 12:29:16 +00:00
Owner
Copy Link

Do you mind if we better move it to initialization block? Cuz, since you declare here that variable again as QAction* it might be even considered like another variable rather than that one declared as a property...

Do you mind if we better move it to initialization block? Cuz, since you declare here that variable again as QAction* it might be even considered like another variable rather than that one declared as a property...
shunf4 commented 2021-10-13 12:08:15 +00:00
Author
Contributor
Copy Link

How about this change? 52551c1ce0

How about this change? https://git.macaw.me/blue/squawk/commit/52551c1ce0be0a223e388d339389b7a55b159b6c
blue commented 2021-10-13 12:32:55 +00:00
Owner
Copy Link

That's good, thank you!

That's good, thank you!
blue marked this conversation as resolved
blue reviewed 2021-10-11 12:34:58 +00:00
ui/widgets/conversation.cpp Outdated
@ -221,0 +253,4 @@
if (image.isNull()) {
return;
}
QTemporaryFile *tempFile = new QTemporaryFile(QDir::tempPath() + QStringLiteral("/squawk_img_attach_XXXXXX.png"), QApplication::instance());
blue commented 2021-10-11 12:34:58 +00:00
Owner
Copy Link

Interestring solution...
I actually was thinking of it, but was more into may be saving it into downloads folder, for the next time. Like this way at somepoint the image is not going to be accesible, so, we're going to be forced to download it to see it.
Dunno, i really have mixed feelings about my way, what do you think?

Interestring solution... I actually was thinking of it, but was more into may be saving it into downloads folder, for the next time. Like this way at somepoint the image is not going to be accesible, so, we're going to be forced to download it to see it. Dunno, i really have mixed feelings about my way, what do you think?
shunf4 commented 2021-10-13 12:11:00 +00:00
Author
Contributor
Copy Link

I agree with you. Placing the pasted image into downloads folder is better.

Or putting it to temp folder, then moving it to downloads folder if it is sent successfully?

I agree with you. Placing the pasted image into downloads folder is better. Or putting it to temp folder, then moving it to downloads folder if it is sent successfully?
blue commented 2021-10-13 12:35:21 +00:00
Owner
Copy Link

Yeah, that would do! But if you will move the image on successfull send you're going to need to update the path to the image in databases after the image is moved, I hope I've done that handler...

Yeah, that would do! But if you will move the image on successfull send you're going to need to update the path to the image in databases after the image is moved, I hope I've done that handler...
shunf4 commented 2021-10-15 16:23:13 +00:00
Author
Contributor
Copy Link

Done in 39f2f3d975 .

The file is copied (rather than moved) because it might still be used by QImageReader in message preview widget.

Done in https://git.macaw.me/blue/squawk/commit/39f2f3d975a1ce38144ef1992506416a934e5005 . The file is copied (rather than moved) because it might still be used by `QImageReader` in message preview widget.
shunf4 added 1 commit 2021-10-13 12:07:07 +00:00
shunf4 added 1 commit 2021-10-15 16:21:15 +00:00
blue merged commit 893ff53aa8 into messageFeed 2021-10-16 15:34:01 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
Bug

Something is not working

  
Documentation

Documentation related issues

  
Easy

   
Feature

New feature

  
Localization

Localization related issues

  
Packaging

Issues related to packaging and building

  
Refactoring

Need to reimplement something in a better way

  
Test

Need to test something

No Label
Bug
Documentation
Easy
Feature
Localization
Packaging
Refactoring
Test
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: blue/squawk#51
No description provided.
Delete Branch "shunf4/squawk:feat/paste_img"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?