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 551096 - Chat window has two close buttons
Chat window has two close buttons
Status: RESOLVED FIXED
Product: empathy
Classification: Core
Component: Chat
2.23.x
Other Linux
: Normal normal
: ---
Assigned To: empathy-maint
Depends on:
Blocks:
 
 
Reported: 2008-09-06 09:29 UTC by Frederic Peters
Modified: 2008-09-23 09:21 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
hide the tab close button where there is a single tab (1.71 KB, patch)
2008-09-06 09:31 UTC, Frederic Peters
committed Details | Review

Description Frederic Peters 2008-09-06 09:29:07 UTC
As noted in mpt review, about the chat window, with a single chat: "The window had two close buttons, one in the title bar and one inside the window’s single tab".
Comment 1 Frederic Peters 2008-09-06 09:31:51 UTC
Created attachment 118153 [details] [review]
hide the tab close button where there is a single tab

This patch hides the tab close button when there is a single tab.

About the code itself: there was already a bunch of g_object_set_data(), I am adding one but it may be the sign of a needed refactoring.
Comment 2 Xavier Claessens 2008-09-07 09:42:59 UTC
Thanks, looks good :D
Comment 3 Frederic Peters 2008-09-07 09:50:10 UTC
Thanks; commited in r1426.
Comment 4 Will Thompson 2008-09-22 20:42:25 UTC
Now the close button suddenly vanishes when I get to one tab, which means I can't point at the right-most × and repeatedly mash the mouse button to close all the tabs.

(Yes, I could click the other ×.)
Comment 5 Xavier Claessens 2008-09-23 08:42:56 UTC
Will, there is obviously no solution between your requirements and this bug. In fact I agree with you but I use ctr-W anyway...
Comment 6 Dafydd Harries 2008-09-23 08:59:03 UTC
I see three options:

 1) Keep the current behaviour (close buttons on tabs when there's more than one tab). This might be less confusing (no redundant close buttons).
 2) Revert to the old behaviour (close button on tabs always). This is arguably more consistent because close buttons don't suddenly disappear.
 3) Don't use tabs when there is only one conversation in the window. This is what Epiphany does, but arguably introduces a different consistency.
Comment 7 Will Thompson 2008-09-23 09:21:38 UTC
Xavier: yes, I know.  I think I am probably doing it wrong, and should click the window's × button, but thought I should register my surprise at the × going away.  :-)

(I would use ^W, but I avoid pressing it because I can never remember whether it deletes the last word or closes the tab…)