Hello,
I just noticed that when I'm writing an email address in DiscussionTools, the system starts to show me a list of users, just like when I'm going to mention someone.
I think the regex/whatever is responsible for catching the at sign should require a word boundary, so it doesn't do its job when the at sign is in a middle of the word.
Thanks for your awesome work,
Martin Urbanec
We have discussed this in T255153 – originally the autocompletion was only done after word boundaries, but we realized that this would make it useless in languages there there aren't word boundaries, like Chinese.
@Urbanecm for context: were you able to successfully include the email address you were intending to in the comment you were writing?
Yes, but the username selector surprised me quite a bit. Maybe make that configurable by a config flag?
You can also cancel the suggestions dropdown by pressing Escape (it was a bit buggy and would reappear, but T259277 fixes that).
Making it configurable would be technically doable, as it's just a lightly modified reversion of the original removal.
There'd be an argument about whether to make it a user config or a wiki-level config. It'd also add to the general matrix of behaviors to test, which can be inconvenient.
I'd say this would be higher priority if we ever gave in to one of the "automatically link usernames" tickets, since as-is you can just not-interact with the popup and you'll get your desired output.
Change 673521 had a related patch set uploaded (by Bartosz Dziewoński; owner: Bartosz Dziewoński):
[mediawiki/extensions/DiscussionTools@master] Avoid triggering username autocompletion when typing email addresses and article titles
Test wiki created on Patch Demo by ESanders (WMF) using patch(es) linked to this task:
Change 673521 abandoned by Bartosz Dziewoński:
[mediawiki/extensions/DiscussionTools@master] Avoid triggering username autocompletion when typing email addresses and article titles
Reason:
Doesn't work as well as I hoped in languages using other scripts
Test wiki on Patch Demo by ESanders (WMF) using patch(es) linked to this task was deleted:
I saw this in https://ru.wikipedia.org/?diff=132016244 and found it very confusing. I think the automatic mention work should be changed to check whether it has preceding characters (@ → gets autosuggestion, a@ does not).