flpvn / wittytwitter

Automatically exported from code.google.com/p/wittytwitter
0 stars 0 forks source link

Don't show "D <name>" after sending direct messages #110

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Right click on a tweet and select "send direct message"
2. Noticed that "D <name>" is shown
3. type something after that and click update

What is the expected output? What do you see instead?
status bar should say "direct message to <name> sent". Instead, the direct
message is shown in the tweets list, which it shouldn't.

Original issue reported on code.google.com by alan...@gmail.com on 5 Apr 2008 at 1:17

GoogleCodeExporter commented 9 years ago
The patch attached implements the requested behavior as described above in the
expected output section.

I initially coded it so that the behavior matched twhirl (showed the message in 
the
regular timeline with the "d <username>" part stripped out) but scrapped it.  To
truly mirror the behavior regarding the way the tweet was displayed would 
require too
much refactoring/changing just to match.

Original comment by klmcma...@gmail.com on 22 May 2009 at 5:09

Attachments:

GoogleCodeExporter commented 9 years ago
Pardon me if this isn't the best place to be suggesting this, but what about 
having
the "Send Direct Message" option in the context menu trigger the appropriate 
drawer
from the "Messages" tab instead for the sake of consistency?  I haven't had the 
time
to dig into anything yet, but I assume this would also fix the aforementioned 
issue.

Original comment by NickTabick on 22 May 2009 at 5:43

GoogleCodeExporter commented 9 years ago
Applied patch

Nick - we're thinking of getting rid of the Messages tab since it's kind of a 
hidden
feature, doesn't take advantage of autocomplete, etc.

Original comment by jongallo...@gmail.com on 25 May 2009 at 7:01