Skip to content
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

Bridged IRC channels should provide some UI to suggest limiting length of messages #1849

Open
michaelkaye opened this issue Feb 18, 2019 · 4 comments

Comments

@michaelkaye
Copy link

michaelkaye commented Feb 18, 2019

Description

When sending messages bridged to IRC channels, i can go over the IRC message length limit. This then sends the message as a link to the full contents as a bit of media.

It would be better if I could be warned that i'm writing a 300-character missive which will have this occur, and suggest i reduce my message length / compose multiple messages such that it can be sent to irc natively.

Steps to reproduce

  • Send a long message
  • Receive a link to it on irc

Describe how what happens differs from what you expected.

I would prefer not to send those long messages - a popup warning me that this will happen and requesting confirmation seems useful.

The normal messaging flow is not to send very long messages, so this shouldn't cause fatigue due to repeatedly happening and needing dismissal.

Version information

riot.im/develop

@michaelkaye michaelkaye changed the title IRC channels should provide some UX to Bridged IRC channels should provide some UI to suggest limiting length of messages Feb 18, 2019
@t3chguy
Copy link
Member

t3chguy commented Feb 18, 2019

Part of the issue here is that the IRC limit includes things like your Nick, and riot doesn't even know your IRC nick

@t3chguy
Copy link
Member

t3chguy commented Feb 18, 2019

@lampholder
Copy link
Member

Can this all be handled bridgeside? Does the bridge know your IRC nick? Couldn't it split into acceptably-sized chunks?

@lampholder lampholder added the X-Needs-Info This issue is blocked awaiting information from the reporter label Feb 26, 2019
@t3chguy
Copy link
Member

t3chguy commented Feb 26, 2019

It could split, yes, but it could cause flooding without the intent of the sender

@aaronraimist aaronraimist removed the X-Needs-Info This issue is blocked awaiting information from the reporter label Oct 22, 2021
@t3chguy t3chguy transferred this issue from element-hq/element-web Jul 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants