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

Mentioning is broken when whitespace is removed #11813

Closed
SystemKeeper opened this issue Mar 15, 2024 · 4 comments
Closed

Mentioning is broken when whitespace is removed #11813

SystemKeeper opened this issue Mar 15, 2024 · 4 comments

Comments

@SystemKeeper
Copy link
Contributor

How to use GitHub

  • Please use the 👍 reaction to show that you are affected by the same issue.
  • Please don't comment if you have no relevant information to add. It's just extra noise for everyone subscribed to this issue.
  • Subscribe to receive notifications on status change and new comments.

Steps to reproduce

  1. Mention Person A
  2. Mention Person B
  3. Make sure there's no space between the mention chips

Expected behaviour

Mentioning should still work

Actual behaviour

Bildschirmaufnahme.2024-03-15.um.15.18.30.mov

Could argue that it's expected to fail, but since the chip is still rendered, it gives the feeling of everything working fine.

Talk app

Talk app version: main

@nickvergessen
Copy link
Member

Make sure there's no space between the mention chips

That's a human error and nothing we can fix 🤷

Last week Ivan and you complained about the double space, so we removed one, now you remove the other and complain again?

@nickvergessen nickvergessen closed this as not planned Won't fix, can't repro, duplicate, stale Mar 15, 2024
@SystemKeeper
Copy link
Contributor Author

That's a human error and nothing we can fix 🤷

Well, if the inputbar shows it correctly, but is not handled by the server, I don't think this is purely a human error. If this is not possible, why is the autocomplete opened for a "@" without a space in front?

Last week Ivan and you complained about the double space, so we removed one, now you remove the other and complain again?

Not intentionally when it happened the first time, just here for reproduction. But still, this is something different, when you look at the issue (nextcloud-libraries/nextcloud-vue#4127) this is about showing something different in the input, then what is send to the server.

@nickvergessen
Copy link
Member

Sounds liek nextcloud-libraries/nextcloud-vue#5351 then

@SystemKeeper
Copy link
Contributor Author

SystemKeeper commented Mar 15, 2024

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

2 participants