Closed Bug 954792 Opened 11 years ago Closed 10 years ago

IRC should notify of away message

Categories

(Chat Core :: IRC, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED DUPLICATE of bug 954049

People

(Reporter: clokep, Unassigned)

References

(Depends on 1 open bug)

Details

*** Original post on bio 1358 at 2012-04-03 12:49:00 UTC ***

Currently JS-IRC doesn't show a user's away message when IMing them, previously the libpurple IRC implementation showed it every time you sent an IM to a user, there's probably a good middle ground between these two cases:

(In reply to bug 954602 (bio 1170) comment #5)
> We should show this auto-reply message (sent back by the server after each
> message we send) only if we have a reason to believe it will be of interest to
> the user.
> 
> Mook proposed on IRC (http://log.bezut.info/instantbird/111115#m687) a behavior
> that seemed to make sense: "suppress displaying it until either the window is
> closed/hidden, the away message changes, or it changes (so if the user comes
> back and goes away again with the same message, show it anyway, if possible)?"
> 
> So basically, display the message when it first appears, then remember it, and
> if the same message is sent again, don't display it. I don't think putting the
> conversation on hold (which isn't even visible for the prpl plugin) should
> change anything to the behavior.
> We may want to redisplay the message after a few minutes. For example, if I'm
> talking to foo who is away, I'll see the auto-reply message immediately, then
> I'm sending a few more messages without seeing the auto-reply again, then if I
> come back N (5? 20?) minutes later to say something more, I'll see the message
> again.
*** Original post on bio 1358 at 2012-04-06 00:23:13 UTC ***

The conversation top would be the obvious place to show the away message (where the status message goes for other protocols).
*** Original post on bio 1358 at 2012-05-16 16:11:02 UTC ***

Bug 954803 (bio 1369) has fixed this for buddies. Essentially the remainder of this bug should be fixed by a proper patch for bug 954049 (bio 613).
Depends on: 954049
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.