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 70295 - no longer Xinerama compatible
no longer Xinerama compatible
Status: RESOLVED DUPLICATE of bug 73475
Product: gnome-core
Classification: Deprecated
Component: panel
1.4.x
Other Linux
: High major
: 1.4.1
Assigned To: Panel Maintainers
Panel Maintainers
: 70096 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2002-02-01 18:28 UTC by Christian Marillat
Modified: 2004-12-22 21:47 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
fixed patch for different sized screens in Xinerama (45.17 KB, patch)
2002-05-03 23:28 UTC, xavier.bestel
none Details | Review

Description Christian Marillat 2002-02-01 18:28:37 UTC
I have a Xinerama setup; two monitors side by side make one big desktop.
The right-hand monitor is the main monitor, the left one is the secondary
monitor.

I have several aligned panels, one of which was anchored to the bottem left
corner of the desktop. Until the latest update of gnome-panel, this
appeared on the bottom left corner of my left-hand monitor. Now this panel
is anchored to the bottom left corner of my right-hand monitor, and I
cannot get the panel to move to the other monitor.

Converting the panel to an edge panel only has it stretch to the full side
of the one right hand monitor, and it no longer stretches across the two
monitors like it would have done before the update.
Comment 1 Luis Villa 2002-02-04 02:52:33 UTC
What version were you upgrading from?
Comment 2 Christian Marillat 2002-02-05 20:27:21 UTC
Since 1.4.0.5
Comment 3 Luis Villa 2002-02-14 14:12:56 UTC
*** Bug 70096 has been marked as a duplicate of this bug. ***
Comment 4 Kjartan Maraas 2002-02-14 20:13:06 UTC
Does setting FAKE_XINERAMA_PANEL=1 help at all?
Comment 5 Christian Marillat 2002-02-15 22:58:43 UTC
Not at all (from the user) :

I can now place the panel on either monitor on any edge (right-hand
edge of left-hand monitor for example). I cannot make an edge panel
that spans the monitors however. Lucky for me I don't need one.:)
Comment 6 Kjartan Maraas 2002-02-15 23:31:54 UTC
I don't think xinerama falls under the "supported" category in 1.4.x.
In  2.0 this support will be at the gtk+ level I think - so that may
be better.
Comment 7 Luis Villa 2002-03-14 17:30:47 UTC
Removing the GNOME2 keyword; if it is still a problem with gtk2/gnome2
then it should be re-added.
Comment 8 Kjartan Maraas 2002-03-14 21:43:14 UTC
Christian, is panel linked to -lXinerama? There was a bug in the
configure checks that made this not work.
Comment 9 Christian Marillat 2002-03-14 23:33:31 UTC
Apparently yes. From panel/Makefile :

XINERAMA_LIBS = -lXinerama

[...]


panel_LDADD = libpanel_applet.la $(GNOME_LIBDIR) $(GNORBA_LIBS)
$(ORB_LIBS) $(GNOMEUI_LIBS)$(INTLLIBS)$(PIXBUF_LIBS) $(XINERAMA_LIBS)

But the configure script don't like xinerama :

checking X11/extensions/Xinerama.h usability... no
checking X11/extensions/Xinerama.h presence... yes
configure: WARNING: X11/extensions/Xinerama.h: present but cannot be
compiled
configure: WARNING: X11/extensions/Xinerama.h: check for missing
prerequisite headers?
configure: WARNING: X11/extensions/Xinerama.h: proceeding with the
preprocessor's result
checking for X11/extensions/Xinerama.h... yes
Comment 10 Kjartan Maraas 2002-03-15 10:11:34 UTC
Could you try compiling from CVS? There was a fix in configure.in wrt
Xinerama recently there.
Comment 11 Christian Marillat 2002-03-15 10:28:36 UTC
checking for X11/extensions/Xinerama.h... yes
checking for XineramaQueryScreens in -lXinerama... no

It is normal ?

I see another problem :

checking for png_init_io in -lpng... ./configure: ]: command not found
yes
Comment 12 Kjartan Maraas 2002-04-27 00:06:37 UTC
Any idea about the WARNINGS for -lXinerama?
Comment 13 jonathankoren 2002-04-30 08:01:14 UTC
You know, this bug and 73475 are duplicates of each other.  
73475 has patch.  It works.  I run it.

The cause of the bug doesn't have anything to do with 
linking librarys or anything like that.  Instead it has everything to
do with the code actually looking to see if xinerama is enabled, if 
the panel should be drawn on the left on the left or right screen,
shift the panel so that its left-most pixel appears at either 0 or
screen_width / 2, and then truncating the panel to screen_width / 2.  

It's all in multiscreen-stuff.c 

You fix it by checking a flag that indicates whether you want
to span monitors or not.  If you do you use gdk_screen_width() 
et. al..  If not, multiscreen_width() et. al..

Oh and the "keep on same screen code" won't work  >2 screens.
Comment 14 Kjartan Maraas 2002-05-02 13:28:37 UTC
Closing as duplicate then
Comment 15 Kjartan Maraas 2002-05-02 13:29:03 UTC

*** This bug has been marked as a duplicate of 73475 ***
Comment 16 xavier.bestel 2002-05-03 23:28:54 UTC
Created attachment 8173 [details] [review]
fixed patch for different sized screens in Xinerama
Comment 17 xavier.bestel 2002-05-03 23:30:28 UTC
The patch didn't work when screens don't have the same size. I made
one which works.
Comment 18 xavier.bestel 2002-05-06 05:39:16 UTC
Oops ... copy/paste error which makes some pop-up menus appear not at
the right place. Here's the fix.


--- panel/basep-widget.c.orig	Sun May  5 23:11:15 2002
+++ panel/basep-widget.c	Sun May  5 23:04:29 2002
@@ -1978,7 +1978,7 @@
 	if (*x < minx)
 
	*x = minx;
 
-
if (*y + maxy)
+
if (*y > maxy)
 
	*y = maxy;
 	if (*y < miny)
 
	*y = miny;
Comment 19 Kjartan Maraas 2002-05-06 15:15:39 UTC
So. We need the patch in the other bugreport + the one here + the last
fix on top?
Comment 20 Christian Marillat 2002-05-06 15:27:59 UTC
Patche (attachment id=8173), plus this little fix, fix this bug.
Comment 21 George Lebl 2002-05-08 15:57:14 UTC
Have you tried middle dragging the panel over to your left monitor?
Comment 22 xavier.bestel 2002-05-09 10:45:36 UTC
My screen0 is the right one.
I tried dragging my panel on another screen, it works. When I reopen
the "all properties" dialog, it's updated with the right defautl screen.
Why, did you see some kind of bug ?
Comment 23 George Lebl 2002-05-15 17:04:37 UTC
The patch in attachment 8173 [details] [review] seems mostly fine though it would have to
have some parts cleaned up.  But I think it is too long to apply to a
stable branch.  It touches too many parts of the code.

I may not be understanding the problem however.  The problem seems
like someone doesn't like panels that are only restrained to one
screen and wants some panels to span multiple screens.  Right?  In
this case this is an enhancement.  I seem confused about this problem
then.  Why is this a bug and why is it major.  It seems to me fairly
cosmetic that I can't have a panel span multiple screens.  Keeping
panels restrained on screen edges seems to me as the most predictable
behaviours and one which will not result in panels appearing offscreen.

If you want pre-Xinerama support style panels then you can just set
the GNOME_PANEL_NO_XINERAMA env variable (need latest CVS, I just
added that)
Comment 24 Luis Villa 2002-05-15 19:18:17 UTC
It's high/major because it's one of those things that works elsewhere
(mostly) so it is right on that knife edge between feature and bug- it
may never have worked in panel, but it works everywhere else, so...
that's why it is where it is. It's not GNOME2 anyway; I don't know if
that helps you clarify where it stands :)
Comment 25 George Lebl 2002-05-16 09:11:51 UTC
I see what it is.  debian applied a patch that pretty much turns off
xinerama support so you get offscreen panels.  But the plan .8 doesn't
do this.  So I'll mark the duplicate bug as WONTFIX unless someone
sends a patch and/or convinces me that the added effort into having
some sort of property so that you may have panels that span screens is
worth it.  Basically debian doesn't support xinerama while gnome does.
 If there is a bug it is with debian, with plain GNOME1.4 nor 2 you
will never get offscreen panels.
Comment 26 Christian Marillat 2002-05-16 09:22:57 UTC
Completely wrong.

1) The configure script is broken and he is unable to find the
xinerama extention. Read the config.log to see what's wrong.

2) The patche in attachment 8173 [details] [review] solve this bug.
Comment 27 George Lebl 2002-05-16 09:39:17 UTC
The configure check is something different and that is not part of
that patch.  I AM interested in such a patch, that's a fix.  Adding
lots of code that adds a feature is not a bugfix.

What I see the code in this patch and the patch in bug #73475 does is
add a new property for panels that lets them span screens.

  1) I have not tested this patch and cannot do that well currently
     with 1.4 gnome
  2) This patch adds a LOT of code and 1.4.1 is way overdue, so making 
     sure such a patch is rock solid would take a bit more
  3) I have some reservations about how corner cases (such as screens
     with different resolutions) are handeled correctly, that is, 
     under no circumstance should ever it be possible for a panel to 
     be offscreen and that is apparently possible with this patch.

GNOME2 is for adding features

marking again as duplicate of 73475

*** This bug has been marked as a duplicate of 73475 ***
Comment 28 George Lebl 2002-05-16 09:48:51 UTC
ok the configure check is fixed