After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 657278 - remove chats after all read and inactive for a while
remove chats after all read and inactive for a while
Status: RESOLVED OBSOLETE
Product: gnome-shell
Classification: Core
Component: message-tray
unspecified
Other Linux
: Normal normal
: ---
Assigned To: gnome-shell-maint
gnome-shell-maint
Depends on:
Blocks:
 
 
Reported: 2011-08-24 19:55 UTC by William Jon McCann
Modified: 2021-07-05 14:10 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
telepathyClient: auto-close chat notification when inactive (3.29 KB, patch)
2011-09-15 11:48 UTC, Xavier Claessens
needs-work Details | Review
how it looks like (14.85 KB, image/png)
2011-09-15 11:57 UTC, Xavier Claessens
  Details

Description William Jon McCann 2011-08-24 19:55:42 UTC
Chats are bursty. You will asynchronously talk to someone for a while but then you may finish without "ending". Basically trail off. It is important for a "conversation" to remain as long as their are unread/unresponded messages. But once you respond and a time has passed there is little need to keep it around taking up space and adding clutter.

For example, Cosimo and I chatted a while to figure out where to go grab lunch but when I came back from lunch our conversation was still in the tray. It doesn't need to be.

On the other hand, having some people pinned to the tray may be useful. I'm sure my mom (to my dismay) would want to pin me to the tray to contact me as quickly as possible. We can do this from the options menu that we currently use to remove the conversation. Having a "Keep in Tray" or similar would be nice.

As usual with timings we may have to iterate but I'm thinking that something around 15-20 minutes might be ok.
Comment 1 Guillaume Desmottes 2011-09-15 09:01:18 UTC
This should be pretty easy to implement:

- Each time all the incoming messages are acked: start/reset the timer
- Each time an outgoing message is sent: start/reset the timer
- Each time we receive an incoming message: stop the timer
- Once the timer times out remove the chat source (if it's not pinned)
Comment 2 Xavier Claessens 2011-09-15 11:13:00 UTC
if notification timedout and got closed, then I send a message from empathy chat window, do we expect the notification to appear again?
Comment 3 Xavier Claessens 2011-09-15 11:48:45 UTC
Created attachment 196609 [details] [review]
telepathyClient: auto-close chat notification when inactive

This also add a "Spin" item in the context-menu to disable
auto-closing behaviour
Comment 4 Xavier Claessens 2011-09-15 11:57:11 UTC
Created attachment 196610 [details]
how it looks like

Not sure that's really nice looking, but here it is :)
Comment 5 Guillaume Desmottes 2011-09-15 11:59:18 UTC
(In reply to comment #2)
> if notification timedout and got closed, then I send a message from empathy
> chat window, do we expect the notification to appear again?

I'd say yes, as the conversation is re-actived.
Comment 6 Florian Müllner 2011-09-15 12:32:15 UTC
(In reply to comment #3)
> This also add a "Spin" item in the context-menu to disable
> auto-closing behaviour

Oh, on IRC I thought this was a typo - do you really mean "Spin" rather than "Pin"? (and no, I don't think we should use a switch there - how about toggling the label text ("Pin"/"Unpin")?)
Comment 7 Xavier Claessens 2011-09-15 12:37:49 UTC
argh, I English-noobed. Pin of course :)

Yep, changing label would be nicer I guess.
Comment 8 William Jon McCann 2011-09-15 15:41:25 UTC
I don't really like using pin as a verb here. I'd rather use "keep". "Keep in Tray" or something similar.
Comment 9 Florian Müllner 2011-09-15 15:43:07 UTC
What would be the opposite in that case - "Don't keep in tray"?
Comment 10 Guillaume Desmottes 2011-09-16 08:00:19 UTC
Maybe use the same UI as the one to add an app as favorite in the overview?
Comment 11 Marina Zhurakhinskaya 2011-09-27 00:08:26 UTC
Perhaps "Keep in Tray"/"Remove when Idle"?
Comment 12 Jasper St. Pierre (not reading bugmail) 2012-06-01 11:29:33 UTC
Review of attachment 196609 [details] [review]:

I think there's interest in auto-expiry at the Telepathy level as well. I don't think we really need a "Pin" menu item now that we have contact search, from where it's fairly easy to open a contact.

::: js/ui/telepathyClient.js
@@ +557,3 @@
+        this._stopCloseTimeout();
+        if (!this._spinned && this._pendingMessages.length == 0) {
+            this._closeTimeoutId = Mainloop.timeout_add_seconds(15*60,

This needs to be in a constant at the top.
Comment 13 GNOME Infrastructure Team 2021-07-05 14:10:00 UTC
GNOME is going to shut down bugzilla.gnome.org in favor of  gitlab.gnome.org.
As part of that, we are mass-closing older open tickets in bugzilla.gnome.org
which have not seen updates for a longer time (resources are unfortunately
quite limited so not every ticket can get handled).

If you can still reproduce the situation described in this ticket in a recent
and supported software version, then please follow
  https://wiki.gnome.org/GettingInTouch/BugReportingGuidelines
and create a new ticket at
  https://gitlab.gnome.org/GNOME/gnome-shell/-/issues/

Thank you for your understanding and your help.