Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Window List - stretched buttons on vertical panel #269

Closed
usv299792458 opened this issue Dec 22, 2014 · 15 comments
Closed

Window List - stretched buttons on vertical panel #269

usv299792458 opened this issue Dec 22, 2014 · 15 comments

Comments

@usv299792458
Copy link

usv299792458 commented Dec 22, 2014

In Gnome 2 I used to have this layout.
gnome

Unfortunately, in MATE panel, in Window List, window buttons are stretched vertically.
mate

My setup:
MATE version 1.8.1 (Debian Jessie).
Steps to reproduce:
Select TraditionalOK theme.
Remove all panels except one, make it right-sided, set width to 30 pixels, run an application.

@mihai-bica
Copy link

I had the same problem and I run "mate-panel --reset" (which resets the panel layout) and the vertical windows list mate applet got fixed. I think that this is a configuration problem.

@usv299792458
Copy link
Author

Hello mihai-bica,

Thanks for sharing your results.

Unfortunately,

mate-panel --reset

does not fix the issue for me: it just makes top and bottom panels as by default. When I reconfigure right panel, buttons are stretched again.

Igor.

@mihai-bica
Copy link

Hi, you are right. mate-panel --reset does not fix the problem.

However, I downloaded the source code and did this modification:
mihai@mihai-vb ~/git/mate-panel $ git diff

diff --git a/applets/wncklet/window-list.c b/applets/wncklet/window-list.c
index 721aef2..53db860 100644
--- a/applets/wncklet/window-list.c
+++ b/applets/wncklet/window-list.c
@@ -126,6 +126,7 @@ static void applet_change_orient(MatePanelApplet* applet, MatePanelAppletOrient
        wnck_tasklist_set_orientation (tasklist->tasklist, new_orient);
 #endif
 #endif
+       wnck_tasklist_set_orientation (tasklist->tasklist, new_orient); //MIHAI
        tasklist_update(tasklist);
 }

@@ -451,6 +452,8 @@ gboolean window_list_applet_fill(MatePanelApplet* applet)
 #endif
 #endif

+       wnck_tasklist_set_orientation (tasklist->tasklist, tasklist->orientation); //MIHAI
+
        wnck_tasklist_set_icon_loader(WNCK_TASKLIST(tasklist->tasklist), icon_loader_func, tasklist, NULL);

        g_signal_connect(G_OBJECT(tasklist->tasklist), "destroy", G_CALLBACK(destroy_tasklist), tasklist);

mihai@mihai-vb ~/git/mate-panel $

and built it and the pannel got fixed. I am having a fixed panel right now on my linux mint 17 mate.

Sorry for just pasting the code here. I don't have time to commit it ... maybe I will commit at the end of this week.

Thanks.

screenshot0001

@usv299792458
Copy link
Author

Hi again,
Hope your fix will get to debian testing soon.
There is one more problem with window list. I'm not sure if it's better to open a separate issue about it. Anyway, did it happen to you to observe "trembling" of window buttons sometimes, when the panel is on the side? (don't know when and for what reason it happens) It seems that buttons are reorganized to one-column and two-column layout in fast intervals and it causes this ugly effect, which makes window list completely unusable.

@mihai-bica
Copy link

Hi, I observed the trembling when I had many windows open and it completely
disappeared after this fix. Now I can have like 100 windows shown in
taskbar without problem both in horizontal and vertical orientation.

However, I don't know how it behaves on other platforms. I tested only on
mint mate 17.

On Apr 27, 2015 12:16, "usv299792458" notifications@github.com wrote:

Hi, again.
Hope your fix will get to debian testing soon.
There is one more problem with window list. I'm not sure if it's better
to open a separate issue about it. Anyway, did you happen to observe
"trembling" of window buttons sometimes, when the panel is on the side?
(don't know when and for what reason it happens) It seems that buttons are
reorganized to one-column and two-column layout in fast intervals and it
causes this ugly effect, which makes window list completely unusable.


Reply to this email directly or view it on GitHub.

@usv299792458
Copy link
Author

I will test on Debian 8 and write you my results in a few days.

@usv299792458
Copy link
Author

Hello Mihai,
Thank you.
Confirm that your patch works perfectly in Debian 8.
Igor.

@mihai-bica
Copy link

Hi, I'm glad to hear this. What would be the next step? Do I have to commit this code fix?

I refactored a bit the code. Removed the ifdef WNCK_CHECK_VERSION. I searched for WNCK_CHECK_VERSION and it wasn't defined anywhere.

mihai@mihai-vb ~/git/mate-panel $ git diff

diff --git a/applets/wncklet/window-list.c b/applets/wncklet/window-list.c
index 721aef2..78a47d2 100644
--- a/applets/wncklet/window-list.c
+++ b/applets/wncklet/window-list.c
@@ -121,11 +121,7 @@ static void applet_change_orient(MatePanelApplet* applet, MatePanelAppletOrient

        tasklist->orientation = new_orient;

-#ifdef WNCK_CHECK_VERSION
-#if WNCK_CHECK_VERSION (3, 4, 6)
        wnck_tasklist_set_orientation (tasklist->tasklist, new_orient);
-#endif
-#endif
        tasklist_update(tasklist);
 }

@@ -445,11 +441,7 @@ gboolean window_list_applet_fill(MatePanelApplet* applet)
        tasklist->tasklist = wnck_tasklist_new(NULL);
 #endif

-#ifdef WNCK_CHECK_VERSION
-#if WNCK_CHECK_VERSION (3, 4, 6)
        wnck_tasklist_set_orientation (tasklist->tasklist, tasklist->orientation);
-#endif
-#endif

        wnck_tasklist_set_icon_loader(WNCK_TASKLIST(tasklist->tasklist), icon_loader_func, tasklist, NULL);

mihai@mihai-vb ~/git/mate-panel $

Thanks,
Mihai

@usv299792458
Copy link
Author

I would advise you to contact the developers.

@usv299792458
Copy link
Author

Hello mihai-bica,

Did you write to Debian mate-panel maintainers that they remove '#ifdef WNCK_CHECK_VERSION' around 'wnck_tasklist_set_orientation' ?

BTW, did you see this? I have stumbled upon it just now: #58

@mihai-bica
Copy link

Hi usv299792458, I didn't write to Debian pate-panel maintainers. If you have the time, can you please ask them to remove the '#ifdef WNCK_CHECK_VERSION (3, 4, 6)' around 'wnck_tasklist_set_orientation' ? (I don't know who to email/ask and don't have time to search.)

'BTW, did you see this? I have stumbled upon it just now: #58', no I didn't see it. I also know that it was working in the past back to gnome 2 and it got broken recently around mint 17. The bug was introduced by adding this 'ifdef' and not upgrading 'libwnck' to version 3.4.6 in Debian.

Upgrading 'libwnck' to version 3 could be problematic because it is based on GTK 3 which is experimental and that will not happen soon (see latest comment #300).

IMHO, the best fix for us, (until we will have GTK3 mate support) is to revert the 'ifdef' in the Debian mate-panel.

@usv299792458
Copy link
Author

This issue is posted to Debian BTS: https://bugs.debian.org/785272

@usv299792458
Copy link
Author

Hi mihai-bica,

In case you have some information for Debian mate-panel maintainers, simply send it to 785272@bugs.debian.org

@monsta monsta changed the title Window List - stretched buttons Window List - stretched buttons on vertical panel Jan 7, 2016
@monsta monsta added this to the Gtk+3 milestone Jan 7, 2016
@raveit65 raveit65 removed this from the Gtk+3 milestone Aug 13, 2016
@mckaygerhard
Copy link

i discovered that when i resising the panel to inpar size numbers, the icons list dont filker, but in numeric par size filck always for same amount of windows list, lest ilustrate:

i have 8 windows opened in the mate desktop

the panel are in 42 pixels, so windows will got crazy if i open one more program, but if panel has 82 as size, the new window willplace in new line up to down .. event are more space available:
panel-two-windowslist-line

@monsta
Copy link
Contributor

monsta commented Jul 27, 2017

This should be fixed now in 1.18 because of migration to GTK+3.

@monsta monsta closed this as completed Jul 27, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants