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 614319 - Search bar shouldn't resize the chat window
Search bar shouldn't resize the chat window
Status: RESOLVED FIXED
Product: empathy
Classification: Core
Component: Chat
2.30.x
Other Linux
: Normal normal
: ---
Assigned To: empathy-maint
: 611855 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2010-03-30 04:32 UTC by Diego Escalante Urrelo (not reading bugmail)
Modified: 2011-04-07 09:39 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
proposed fix (10.64 KB, patch)
2010-11-05 21:11 UTC, Vitaly Minko
reviewed Details | Review
screenshot (38.16 KB, image/jpeg)
2011-04-05 07:59 UTC, Guillaume Desmottes
  Details
another screenshot (35.82 KB, image/png)
2011-04-05 10:37 UTC, Vitaly Minko
  Details

Description Diego Escalante Urrelo (not reading bugmail) 2010-03-30 04:32:04 UTC
Two problems:
1. Find bar is not friendly with the minimalist window width (as narrow as the menubar allows), so it resizes it everytime you activate it.
2. Resized window is not resized back to its original width after using the find bar.

IMO either make the find bar friendlier to narrow windows or make it restore the original width after being closed.
Comment 1 Guillaume Desmottes 2010-03-30 08:43:24 UTC
Actually I'd prefer if the search bar wouldn't resize the Window as the Epiphany one.
Comment 2 Vitaly Minko 2010-10-21 13:24:43 UTC
This issue is a duplicate of https://bugzilla.gnome.org/show_bug.cgi?id=611855
Comment 3 Guillaume Desmottes 2010-10-22 08:42:49 UTC
*** Bug 611855 has been marked as a duplicate of this bug. ***
Comment 4 Vitaly Minko 2010-11-05 21:10:55 UTC
Fixed in
repo: git://vminko.org/empathy
branch: fix-614319
Comment 5 Vitaly Minko 2010-11-05 21:11:26 UTC
Created attachment 173914 [details] [review]
proposed fix
Comment 6 Guillaume Desmottes 2010-11-08 10:05:17 UTC
Review of attachment 173914 [details] [review]:

Is that the way it's done in Epiphany? I tried but it didn't work here, the search bar doesn't appear :\
Comment 7 Vitaly Minko 2010-11-12 16:01:11 UTC
>Is that the way it's done in Epiphany?

Yes.

>I tried but it didn't work here, the search bar doesn't appear :\

I think this issue is not caused by my changes, because I observe the same behavior using the original search bar code. I filed a separate issue for this problem: https://bugzilla.gnome.org/show_bug.cgi?id=634688
Comment 8 Guillaume Desmottes 2010-11-15 09:12:58 UTC
Ok, let's fix bug #634688 first then.
Comment 9 Vitaly Minko 2011-04-03 00:42:11 UTC
This issue is no longer reproducible.
However, the fix is implemented in a different way (Empathy just truncates the search bar, there is no dropdown menu with hidden button).
So please let me know your opinion. If you are OK with the current solution - then I think the bug should be closed. If you prefer my approach - I'll update the patch.
Comment 10 Guillaume Desmottes 2011-04-04 08:56:16 UTC
Yeah I think it would be cool to have the dropdown menu.
Comment 11 Vitaly Minko 2011-04-05 02:27:19 UTC
Well actually there is nothing to update. The original patch works fine for me with empathy-3.0.0.
Comment 12 Guillaume Desmottes 2011-04-05 07:59:43 UTC
Created attachment 185177 [details]
screenshot

I tested your branch and the bar is pretty big here.
Comment 13 Vitaly Minko 2011-04-05 10:37:03 UTC
Created attachment 185185 [details]
another screenshot

Try this branch: fix-614319-v2
It looks quite nice IMHO.
Comment 14 Guillaume Desmottes 2011-04-06 12:12:24 UTC
Author: vminko <vminko@vminko-virtual-machine.(none)>

Time to "git config" :)

Humm I still have the same big button, are you sure you pushed the right patch? (942ddf97dbc275dbfd7df76560a2c3ad5730c1ee)
Comment 15 Vitaly Minko 2011-04-07 07:32:11 UTC
>Time to "git config" :)

Ouch, thanks for letting me know.

>Humm I still have the same big button, are you sure you pushed the right patch?

I pushed the right patch and I can't reproduce this behavior, but I guess that the extra height is caused by the Find Prev/Next buttons, because the icons are displayed on top of the labels. So this time I explicitly specified the correct style: GTK_TOOLBAR_BOTH_HORIZ. Please try the new branch: fix-614319-v3
Comment 16 Guillaume Desmottes 2011-04-07 09:39:13 UTC
I just branched for 3.0, you'll be please to know that your branch is the first 3.2 feature. :)

Merged to master, thanks a lot.

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.