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

Add autocomplete in comment text areas for '@' mentions, similar to GitHub, Twitter, etc. #20

Closed
wants to merge 2 commits into
base: master
from

Conversation

Projects
None yet
3 participants
@rjsamson
Contributor

rjsamson commented Sep 17, 2012

This commit adds autocomplete when adding an '@' mention in a comment using jquery ui autocomplete and a custom triggerAutocomplete plugin. The autocomplete is triggered by the '@' and does not interfere with other normal use of the '@', for example if entering an email address. The following screenshot shows how this looks as implemented:

screenshot

@pjutard

This comment has been minimized.

Show comment
Hide comment

pjutard commented Dec 5, 2012

@rjsamson

This comment has been minimized.

Show comment
Hide comment
@rjsamson

rjsamson Dec 6, 2012

Contributor

@pjutard that looks pretty interesting - might be worth taking a look at. I'm not entirely sure if this is a feature that @jcs is interested in merging though - if it is, it might be worth trying out though.

Contributor

rjsamson commented Dec 6, 2012

@pjutard that looks pretty interesting - might be worth taking a look at. I'm not entirely sure if this is a feature that @jcs is interested in merging though - if it is, it might be worth trying out though.

@jcs

This comment has been minimized.

Show comment
Hide comment
@jcs

jcs Dec 8, 2012

Contributor

I haven't really decided one way or the other, which is why I've left this open. I can see the utility in it, but I am hesitant to add lots of cruft (i.e., JavaScript) to the interface.

Contributor

jcs commented Dec 8, 2012

I haven't really decided one way or the other, which is why I've left this open. I can see the utility in it, but I am hesitant to add lots of cruft (i.e., JavaScript) to the interface.

@jcs jcs closed this Jul 10, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment