Converse: Mentions and line break
When writing a message, if I want to mention someone after a line break I have to press "@" then press backspace - sometimes remove half of my message - to get the list.
No child items are currently assigned. Use child items to break down this issue into smaller parts.
Link issues together to show that they're related. Learn more.
Activity
- Developer
Converse needs a delimiter in order to detect the @ character is being used to trigger a mentions list. Most of the time, a single space before the @ character works. It would be nice if it used the line break character as a delimiter, but it currently does not.
If I understand this issue correctly, you want Converse to treat line feed/carriage return \n as a delimiter to trigger the mentions list?
- Author Owner
Woah, weird behavior. :)
Well, to be honest, I want Converse to open the "@" list whenever I need it.
So if I take example on Gitlab:
- I type @ with a space before: It opens
- I type @ with a line break, regardless of my os: It opens
- I type @ with a special char before: It opens (Say: "@someone" or (@someone")
- I type @ with a letter or a number before: It does not opens
- Developer
This is one those issues that reminds me that I am not a front-end web developer and just a plain developer :-)
- Maintainer
@XaviFP @jbpasquier Can we have a time estimate on this one. I believe it makes that UX of our chat painful. If the time estimate is not absurd, I would tackle it.
- Maintainer
@XaviFP This issue is funded, no problem. However I'm not so sure what takes 12h. Isn't it about rewriting a regular expression and job done?
- Developer
My bad @alexbourlier , for some reason after watching the video I though this issue and #158 (moved) where the same one ^^' I can estimate them separately if it's more convenient :)
- Maintainer
Indeed, they aren't. It's helpful indeed if you estimate them separately.
- Please register or sign in to reply
- Jean-Baptiste Pasquier unassigned @deleolajide
unassigned @deleolajide
- Xavier Ferrer de la Penyita changed time estimate to 1d 4h
changed time estimate to 1d 4h
- Xavier Ferrer de la Penyita changed time estimate to 4h
changed time estimate to 4h
- Developer
This will be fixed when https://github.com/conversejs/converse.js/pull/2267 gets merged.
- Maintainer
Cool news, it is merged
- Xavier Ferrer de la Penyita mentioned in merge request !63 (merged)
mentioned in merge request !63 (merged)
- Jean-Baptiste Pasquier mentioned in commit a016efa5
mentioned in commit a016efa5
- Jean-Baptiste Pasquier closed via merge request !63 (merged)
closed via merge request !63 (merged)
- Jean-Baptiste Pasquier reopened
reopened
- Jean-Baptiste Pasquier added Feature request label
added Feature request label