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 642865 - Chat window close button is huge
Chat window close button is huge
Status: RESOLVED FIXED
Product: empathy
Classification: Core
Component: General
unspecified
Other Linux
: Normal normal
: ---
Assigned To: empathy-maint
empathy-maint
Depends on:
Blocks:
 
 
Reported: 2011-02-21 12:00 UTC by Ignacio Casal Quinteiro (nacho)
Modified: 2011-02-21 15:14 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
gedit-close-button: relicense to LGPL (2.08 KB, patch)
2011-02-21 14:28 UTC, Guillaume Desmottes
none Details | Review
http://git.collabora.co.uk/?p=user/cassidy/empathy;a=shortlog;h=refs/heads/gedit-button-642865 (8.23 KB, patch)
2011-02-21 15:00 UTC, Guillaume Desmottes
none Details | Review

Description Ignacio Casal Quinteiro (nacho) 2011-02-21 12:00:21 UTC
Here you have one that works only with gtk3: http://git.gnome.org/browse/gedit/tree/gedit/gedit-close-button.c
and another that works with 2 - 3:
http://git.gnome.org/browse/gnome-terminal/tree/src/terminal-close-button.c
Comment 1 Guillaume Desmottes 2011-02-21 13:22:03 UTC
Are you refering about the close button in chat tabs?

Also, we are trying to avoid introducing new GPL code to Empathy (the goal being to be able to move code to a future telepathy-gtk library); any chance to have a LGPL version of gedit's code?
Comment 2 Ignacio Casal Quinteiro (nacho) 2011-02-21 13:38:32 UTC
yes, the close button in the tabs.
You have my agree to relicense it as lgpl, although you need it also from paolo
Comment 3 Paolo Borelli 2011-02-21 13:39:37 UTC
I am fine with relicensing to lgpl
Comment 4 Guillaume Desmottes 2011-02-21 14:03:50 UTC
Great, thanks guys. Can you please update the license in gedit first so Empathy can just ship a bare copy of it?
Comment 5 Guillaume Desmottes 2011-02-21 14:23:51 UTC
We need the approval of those people as well:

cassidy@cass-x200s:~/gnome/gedit $ git shortlog -s gedit/gedit-close-button.c
     1  Garrett Regier
     5  Ignacio Casal Quinteiro
     1  Jesse van den Kieboom
     1  Paolo Borelli
     1  Steve Frécinaux
Comment 6 Guillaume Desmottes 2011-02-21 14:28:38 UTC
Created attachment 181460 [details] [review]
gedit-close-button: relicense to LGPL

If/when all the authors agreed, I already have a patch ready for the relicensing.
Comment 7 Guillaume Desmottes 2011-02-21 14:39:31 UTC
Ignacio told me it was ok so I pushed the change.
Comment 8 Guillaume Desmottes 2011-02-21 15:00:05 UTC
Created attachment 181463 [details] [review]
http://git.collabora.co.uk/?p=user/cassidy/empathy;a=shortlog;h=refs/heads/gedit-button-642865

 src/Makefile.am           |    2 +
 src/empathy-chat-window.c |   25 ++------------
 src/gedit-close-button.c  |   80 +++++++++++++++++++++++++++++++++++++++++++++
 src/gedit-close-button.h  |   60 +++++++++++++++++++++++++++++++++
 4 files changed, 146 insertions(+), 21 deletions(-)
Comment 9 Guillaume Desmottes 2011-02-21 15:12:32 UTC
merged to master.

This problem has been fixed in the development version. The fix will be available in the next major software release. Thank you for your bug report.
Comment 10 Steve Frécinaux 2011-02-21 15:14:01 UTC
For the record, I'm ok with the license change.