2012-11-27 22:20:00 |
Alberts Muktupāvels |
bug |
|
|
added bug |
2012-11-28 13:20:54 |
Alberts Muktupāvels |
branch linked |
|
lp:~albertsmuktupavels/ubuntu/raring/gnome-panel/fix-for-1083811 |
|
2012-11-29 02:11:34 |
Launchpad Janitor |
branch linked |
|
lp:~albertsmuktupavels/ubuntu/raring/gnome-panel/fix-for-1083811 |
|
2012-11-29 02:12:51 |
Alberts Muktupāvels |
gnome-panel (Ubuntu): assignee |
|
Alberts Muktupāvels (albertsmuktupavels) |
|
2012-11-29 14:35:14 |
Alberts Muktupāvels |
description |
I'm trying to use separate x screens on nvidia card with 3 monitors. I'm expecting one top panel + one bottom panel on each screen, but instead i have got all panels on one screen.
Restarting gnome panel (gnome-panel --replace) it will create two extra (blank) panel pairs on same screen, but new applets will be added on first panel pair. Screenshot ilustrating problem can be found here - http://askubuntu.com/questions/177226/duplicated-menu-panel-indicators-and-taskbar
I tried to manualy set screen key (org->gnome->gnome-panel->layout->toplevels->top-panel->screen) on wich screen panel should appear with dconf Editor. But restarting gnome-panel that key will be reseted.
Downloaded gnome-panel source code (apt-get source gnome-panel), spend some time trying to find problem, and probably i found it in file panel-toplevel.c.
Line 4458: if (!screen || GDK_IS_SCREEN (screen))
Line 4459: screen = gdk_screen_get_default ();
These lines are responsible for resetting screen key. It will reset screen to default if currenct screen exist, but its not what we want. We want reset screen to default if screen dosnt exist, so line 4458 should be - if (!screen || !GDK_IS_SCREEN (screen))
After compiling and installing new package, gnome panel works as expected. I dont now how to make and submit patch, so someone with more knowledge could do it. |
I'm trying to use separate x screens on nvidia card with 3 monitors. I'm expecting one top panel + one bottom panel on each screen, but instead i have got all panels on one screen.
Restarting gnome panel (gnome-panel --replace) it will create two extra (blank) panel pairs on same screen, but new applets will be added on first panel pair. Screenshot ilustrating problem can be found here - http://askubuntu.com/questions/177226/duplicated-menu-panel-indicators-and-taskbar
I tried to manualy set screen key (org->gnome->gnome-panel->layout->toplevels->top-panel->screen) on wich screen panel should appear with dconf Editor. But restarting gnome-panel that key will be reseted.
Downloaded gnome-panel source code (apt-get source gnome-panel), spend some time trying to find problem, and probably i found it in file panel-toplevel.c.
Line 4458: if (!screen || GDK_IS_SCREEN (screen))
Line 4459: screen = gdk_screen_get_default ();
These lines are responsible for resetting screen key. It will reset screen to default if currenct screen exist, but its not what we want. We want reset screen to default if screen dosnt exist, so line 4458 should be - if (!screen || !GDK_IS_SCREEN (screen))
After compiling and installing new package, gnome panel works as expected. |
|
2012-11-29 16:21:16 |
Jeremy Bícha |
bug |
|
|
added subscriber Jeremy Bicha |
2012-11-30 11:32:43 |
Sebastien Bacher |
gnome-panel (Ubuntu): importance |
Undecided |
Low |
|
2012-12-03 22:19:55 |
Launchpad Janitor |
branch linked |
|
lp:~albertsmuktupavels/ubuntu/raring/gnome-panel/fix-1083811 |
|
2012-12-04 11:44:19 |
Alberts Muktupāvels |
bug watch added |
|
https://bugzilla.gnome.org/show_bug.cgi?id=689610 |
|
2012-12-17 23:14:46 |
Dimitri John Ledkov |
bug task added |
|
gnome-panel |
|
2012-12-18 13:46:05 |
Launchpad Janitor |
branch linked |
|
lp:ubuntu/raring-proposed/gnome-panel |
|
2012-12-18 14:09:12 |
Launchpad Janitor |
gnome-panel (Ubuntu): status |
New |
Fix Released |
|
2013-04-28 06:40:27 |
Bug Watch Updater |
gnome-panel: status |
Unknown |
New |
|
2013-04-28 06:40:27 |
Bug Watch Updater |
gnome-panel: importance |
Unknown |
Medium |
|
2013-05-21 05:26:32 |
Bug Watch Updater |
gnome-panel: status |
New |
Fix Released |
|
2013-07-29 21:30:21 |
Alberts Muktupāvels |
attachment added |
|
gnome-panel_3.6.2-0ubuntu12_amd64.deb https://bugs.launchpad.net/ubuntu/+source/gnome-panel/+bug/1083811/+attachment/3753762/+files/gnome-panel_3.6.2-0ubuntu12_amd64.deb |
|
2013-07-29 21:30:44 |
Alberts Muktupāvels |
attachment added |
|
gnome-panel-data_3.6.2-0ubuntu12_all.deb https://bugs.launchpad.net/ubuntu/+source/gnome-panel/+bug/1083811/+attachment/3753763/+files/gnome-panel-data_3.6.2-0ubuntu12_all.deb |
|
2013-07-29 21:53:19 |
Alberts Muktupāvels |
attachment added |
|
gnome-panel_3.4.1-0ubuntu2_amd64.deb https://bugs.launchpad.net/ubuntu/+source/gnome-panel/+bug/1083811/+attachment/3753780/+files/gnome-panel_3.4.1-0ubuntu2_amd64.deb |
|
2013-07-29 21:53:42 |
Alberts Muktupāvels |
attachment added |
|
gnome-panel-data_3.4.1-0ubuntu2_all.deb https://bugs.launchpad.net/ubuntu/+source/gnome-panel/+bug/1083811/+attachment/3753781/+files/gnome-panel-data_3.4.1-0ubuntu2_all.deb |
|
2013-07-29 21:58:15 |
Alberts Muktupāvels |
branch linked |
|
lp:~albertsmuktupavels/gnome-panel/fix-for-1196177 |
|
2013-07-29 22:00:03 |
Launchpad Janitor |
branch linked |
|
lp:~albertsmuktupavels/gnome-panel/separate-x-screens-fix |
|
2013-08-14 14:38:58 |
Dmitry Shachnev |
nominated for series |
|
Ubuntu Precise |
|
2013-08-14 14:38:58 |
Dmitry Shachnev |
bug task added |
|
gnome-panel (Ubuntu Precise) |
|
2013-08-14 19:44:29 |
Alberts Muktupāvels |
description |
I'm trying to use separate x screens on nvidia card with 3 monitors. I'm expecting one top panel + one bottom panel on each screen, but instead i have got all panels on one screen.
Restarting gnome panel (gnome-panel --replace) it will create two extra (blank) panel pairs on same screen, but new applets will be added on first panel pair. Screenshot ilustrating problem can be found here - http://askubuntu.com/questions/177226/duplicated-menu-panel-indicators-and-taskbar
I tried to manualy set screen key (org->gnome->gnome-panel->layout->toplevels->top-panel->screen) on wich screen panel should appear with dconf Editor. But restarting gnome-panel that key will be reseted.
Downloaded gnome-panel source code (apt-get source gnome-panel), spend some time trying to find problem, and probably i found it in file panel-toplevel.c.
Line 4458: if (!screen || GDK_IS_SCREEN (screen))
Line 4459: screen = gdk_screen_get_default ();
These lines are responsible for resetting screen key. It will reset screen to default if currenct screen exist, but its not what we want. We want reset screen to default if screen dosnt exist, so line 4458 should be - if (!screen || !GDK_IS_SCREEN (screen))
After compiling and installing new package, gnome panel works as expected. |
I'm trying to use separate x screens on nvidia card with 3 monitors. I'm expecting one top panel + one bottom panel on each screen, but instead i have got all panels on one screen.
Restarting gnome panel (gnome-panel --replace) it will create two extra (blank) panel pairs on same screen, but new applets will be added on first panel pair. Screenshot ilustrating problem can be found here - http://askubuntu.com/questions/177226/duplicated-menu-panel-indicators-and-taskbar
I tried to manualy set screen key (org->gnome->gnome-panel->layout->toplevels->top-panel->screen) on wich screen panel should appear with dconf Editor. But restarting gnome-panel that key will be reseted.
Downloaded gnome-panel source code (apt-get source gnome-panel), spend some time trying to find problem, and probably i found it in file panel-toplevel.c.
Line 4458: if (!screen || GDK_IS_SCREEN (screen))
Line 4459: screen = gdk_screen_get_default ();
These lines are responsible for resetting screen key. It will reset screen to default if currenct screen exist, but its not what we want. We want reset screen to default if screen dosnt exist, so line 4458 should be - if (!screen || !GDK_IS_SCREEN (screen))
After compiling and installing new package, gnome panel works as expected.
[Impact]
This bug is important for me and from time to time I see bugs where people write that there is problem with separate x screens. This is one of the problems.
I have made working fix and It would be nice if it is included in our current LTS release.
First part of fix is simple. It was missing ! in if statement. This fix is included already in raring and in saucy.
Second part is fix for unique id generation for gnome-panels. Now id is generated with screen number included in its name. It is required to set objects to correct panel.
[Test Case]
* To reproduce the bug you need at least two monitors. Than you need configure x to use each monitor as separate x screen.
* When you will login gnome-panels will be created on first screen. So if you have 2 screens than on first screen will be 4 panels (2 top and 2 bottom). If you have 3 screens than on first screen will be 6 panels (3 top and 2 bottom)
* It is posilbe to change panels settings, changing screen setting to correct screen would allow to put gnome-panel on correct screen. Bet it will be rest next time gnome-panels starts.
[Regression Potential]
* There is not any regression potential I can think of. It simply fix probably typo bug now allowing gnome panels to be created on correct screens. Gnome panel generates new id only when creating new panel, so it should not affect exsisting gnome panels.
* I am using fixed gnome panel. In this bug report is one user who have confirmed that this works. |
|
2013-08-15 22:22:46 |
Brian Murray |
gnome-panel (Ubuntu Precise): status |
New |
Fix Committed |
|
2013-08-15 22:22:50 |
Brian Murray |
bug |
|
|
added subscriber Ubuntu Stable Release Updates Team |
2013-08-15 22:22:51 |
Brian Murray |
bug |
|
|
added subscriber SRU Verification |
2013-08-15 22:23:00 |
Brian Murray |
tags |
|
verification-needed |
|
2013-08-15 22:38:17 |
Launchpad Janitor |
branch linked |
|
lp:ubuntu/precise-proposed/gnome-panel |
|
2013-08-16 13:16:48 |
Alberts Muktupāvels |
tags |
verification-needed |
verification-done |
|
2013-08-29 17:49:00 |
Launchpad Janitor |
gnome-panel (Ubuntu Precise): status |
Fix Committed |
Fix Released |
|
2013-08-29 17:49:05 |
Brian Murray |
removed subscriber Ubuntu Stable Release Updates Team |
|
|
|