Xref: utzoo comp.windows.x:33044 comp.windows.x.motif:1982
Path: utzoo!utgpu!news-server.csri.toronto.edu!cs.utexas.edu!uunet!mcsun!i2unix!esacs!ferrara
From: ferrara@esacs.UUCP (Giancarlo Ferrara)
Newsgroups: comp.windows.x,comp.windows.x.motif
Subject: --Table little fix for defaultButton.
Keywords: Table Motif1.1
Message-ID: <105@esacs.UUCP>
Date: 20 Feb 91 15:23:36 GMT
Followup-To: comp.windows.x
Organization: Esa Software s.r.l. Rimini (FO), ITALY
Lines: 48


The Motif version of the table widget (as in Wcl 1.05) contains two bugs.

One I have fixed:
- Table is not able to dispatch return (osfActivate) to the default
button if you have one and you are not on the default button. The
parent_process function is not inhered.
The fix is simple and a diff follow: it's just
to replace of parent_process field from NULL to XmInheritParentProcess.

The other is that table is not able to draw the shadowing. It is not
so simple to fix;

First the expose procedure must be inhered (now is _XmRedisplayGadgets());

Second all the geometry policy must consider that the size of the table
is smaller of 2*shadowThickness (child must have x+shadowThickness AND
all request to parent and query_geometry must ask/suggest +2*shadowThickness)
Also all the action that result in a redisplay must clear the old shadow
and redisplay the new updating the dedicated fields (old_shadow*).

Fixing the geometry policy in table have also changed a lot this code and is
not so trivial for me to follow the code.

May be someone have already fixed???
Giancarlo

PS:

*** Table.c	Wed Feb 20 11:13:13 1991
--- Xmp/Table.c	Tue Feb 19 17:04:36 1991
***************
*** 194,200 ****
      /* num_syn_resources  */	0,
      /* syn_cont_resources */	NULL,
      /* num_syn_cont_resources */ 0,
!     /* parent_process     */    XmInheritParentProcess,
      /* extension	  */	NULL
  #else
      /* translations 	  */	(XtTranslations) _XtInherit,
--- 195,201 ----
      /* num_syn_resources  */	0,
      /* syn_cont_resources */	NULL,
      /* num_syn_cont_resources */ 0,
!     /* parent_process     */    NULL,
      /* extension	  */	NULL
  #else
      /* translations 	  */	(XtTranslations) _XtInherit,