-
Notifications
You must be signed in to change notification settings - Fork 87
Ability to send text messages #225
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
base: master
Are you sure you want to change the base?
Conversation
Display text message as a transfer in UI, show notification, allow copy text
Send message button and dialog
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.
Beenden
"Delete all" |
|
Hi, I tested this out, some feedback:
I'd like to wait for @clefebvre's opinion on all of this stuff though - opinions tend to vary :) Thanks! |
I've added the ability to send and receive text messages. This could be useful for quickly sending URLs and tokens between devices and would fix #105

The messages can be selected and copied (selection, or whole message using the button). Up to 4 lines or first 120 characters are shown, although sometimes 4 lines can be exceeded due to wrapping. Unfortunately I did not figure out how to convince Gtk to properly ellipsize it, suggestions are welcome. Wrapping the text manually by inserting newlines looks the best, but then selecting and copying a part of it also includes the newlines... Another option would be to put the label in a ScrolledWindow, but I don't think that looks good. I think limiting the visible length in some way is needed to avoid having one message take up the whole window.
It also shows a notification:

I will add an identical feature to the Android and Windows versions later.