KoKuToru / gTox

[abandoned] GTK3 Tox-Client
GNU General Public License v3.0
45 stars 7 forks source link

TOX_ERR_FRIEND_SEND_MESSAGE_TOO_LONG #222

Open Lesik opened 8 years ago

Lesik commented 8 years ago

I did not try to reproduce this problem or get additional logs because it seems pretty clear what the issue is.

My idea would be that when gTox notices a message that is too long, it splits the message into multiple parts and delivers them in sequence.

I am using gtox-git from AUR, so probably the newest version.

bildschirmfoto vom 2016-01-04 22-18-35

KoKuToru commented 8 years ago

https://tox.gitbooks.io/tox-client-standard/content/communication/message_formatting.html

3.5.3 A sending client must not insert or remove any bytes from the original input.

how would I split without removing bytes ?


Ignoring that, I am against message splitting.

Here are some options:

message splitting

without message splitting

Lesik commented 8 years ago

Wow jayzuz calm down, it was just an idea.

Then tell the GtkEntry to not accept messages longer than a certain length.

KoKuToru commented 8 years ago

Wow jayzuz calm down, it was just an idea.

And I was just listing some options ?

here is a new idea

message splitting (sometimes)

Algram commented 8 years ago

I don't see the problem in just setting a fixed message length. It would be easy to implement and would be consistent on both ends of the chat aswell.

KoKuToru commented 8 years ago

consistent yeah that's why i thought message splitting is stupid. The receiver should get 1:1 what I sent. Not something else.

But splitting line-by-line... why not ? it will be rendered the same way as one non-splitted message. (EDIT: not true when markdown text formatting support gets added https://github.com/Tox/Tox-Client-Standard/pull/25, this will need extra handling for splitting)

bubble for 11:00:
this is a splited message line 1
message line 2

It might get a own bubble for time, which might suck.

bubble for 11:00:
this is a splited message line 1

bubble for 11:01:
message line 2

and in other tox-clients (without bubbles) it might get a own timestamp...