| Issue 190: | @ autocomplete not functioning correctly | |
| 3 people starred this issue and may be notified of changes. | Back to list |
What steps will reproduce the problem? 1. Launch latest experimental 2. Type @ in a new tweet What is the expected output? What do you see instead? Typing @ results in $1 followed by the twitter name of the first user alphabetically. The keyboard shortcuts (tab, up, down) to select the user don't work. Typing another letter does not continue the autocomplete. What version of the product are you using? On what operating system? Experimental, Vista x64
Comment
1
by
johnshee...@gmail.com,
Jan 19, 2009
,
Jan 27, 2009
If you follow someone who's twitter name starts with a number (like @37Signals), it breaks the autocomplete. It also crashes witty if you follow a user like this and type @ somewhere other than the first character in the box see http://www.screencast.com/users/JohnSheehan/folders/Jing/media/3dc81c57-f0da- 4cc6-a4e7-19bbd718ff36
,
Jan 30, 2009
This is also occurring for me, except I'm not using the experimental version (2.1.0.540). I often get $137signals when trying to @ someone. Will try unfollowing 37signals (and any other numerical ones) to check.
,
Jan 30, 2009
I'm not a regex wizard, so I'm sure there is a more elegant way to fix this. Here is a patch to fix this issue for now.
,
Jan 30, 2009
correct me if I'm wrong but won't that patch replace all spaces in the text box so if I had typed "Hey check out this tweet by @blehbleh" it would change that to "Heychecloutthistweetby@blehbleh" ???
,
Jan 30, 2009
I believe by escaping the $1 with ${1} (which inside a string.Format must be ${{1}})
it will solve the problem. Patch attached.
,
Jan 30, 2009
@kris.scott Yeah, if there is text before the @<name> it would remove the spaces, didn't see that, late night. I'll fix it, thanks!
,
Jan 30, 2009
@kris.scott Just saw your post, cool patch.
,
Feb 2, 2009
Applied patch. Thanks for catching and fixing this, guys!
Status: Fixed
Owner: jongalloway |
|
| ► Sign in to add a comment |