joelwatson / mention-field

A textarea field for Ext JS that supports 'mentions'
3 stars 0 forks source link

List doesn't populate when mentionTrigger, mentionStartPattern and mentionEndPattern are set to ":" #1

Open sandeepkashyap opened 8 years ago

sandeepkashyap commented 8 years ago

If mentionTrigger, mentionStartPattern and mentionEndPattern are set to ":" and you type ":" after a few words, the list doesn't get populated.

joelwatson commented 8 years ago

Thanks for the report. I think the current regex expects that the patterns are different. I can take a look into supporting the same pattern for each.

sandeepkashyap commented 8 years ago

Did you get a chance to look into this?

joelwatson commented 8 years ago

When the trigger, start, and end pattern are all the same, the logic gets a bit fuzzy for detecting the boundaries for selections vs. new triggers vs. normal text characters. I'll have to think about this and see if there's a better way to support it. Thanks!

sandeepkashyap commented 8 years ago

I understand. Let me try to come up with a solution :)

joelwatson commented 8 years ago

@sandeepkashyap Sounds good--pull requests are more than welcome!