On Sun, Jan 14, 2007 at 07:55:19PM +0100, Dominik Vogt wrote:
> On Sun, Jan 14, 2007 at 06:43:48PM +0000, Thomas Adam wrote:
> > On Sun, Jan 14, 2007 at 07:36:02PM +0100, Dominik Vogt wrote:
> > > On Sun, Jan 14, 2007 at 06:25:15PM +0000, Thomas Adam wrote:
> > > > On Sun, Jan 14, 2007 at 07:13:04PM +0100, Dominik Vogt wrote:
> > > > > The reason is that fvwm thinks the character size is 1x1.  This
> > > > > happens explicitly with the ResizeHintOverride style (I've
> > > > > committed a patch for that).  There are three possible reasons:
> > > > > 
> > > > >  1) The user set ResizeHintOverride
> > > > > 
> > > > >     ==> Try to set "style * !ResizeHintOverride" explicitly
> > > > > 
> > > > >     Note: This style does *not* work if the window is already
> > > > >           mapped!
> > > > > 
> > > > >  2) The application did not set a character size (or any size
> > > > >     hints at all).
> > > > > 
> > > > >     As xwininfo reports the proper size, this is not the case.
> > > > > 
> > > > >  3) The application set an invalid character size (<= 0)
> > > > > 
> > > > >     ==> look for a message "The application window ... has broken
> > > > >         size hints (..._inc)" on the console.
> > > > 
> > > > Well, in this instance, 3) gave no messages whatsoever, but it does fix
> > > > FvwmIdent in this case.
> > > 
> > > *What* did fix FvwmIdent?
> > 
> > Your lastest patches do.  All I was saying is that for the three
> > options you listed above, point number three (since neither point 1 or
> > point 2 were true in this instance) gave no output about about broken size
> > hints, but for all the windows I have since tried FvwmIdent on
> > (including the test windows I was using to help you in this diagnosis)
> > it now reports the correct geometry.  I couldn't tell you *what* exactly
> > fixed what, all I know is that it now works.  :)
> >  
> > > Do you mean that the problem goes away with !ResizeHintOverride?
> > 
> > Yes.  And now even with ResizeHintOverride set, the correct geometry is
> > reported via FvwmIdent.
> 
> Aha, so the ResizeHintOverride must have been set.
> 
> > > > The only problem now is that it appears the ResizeHintOverride
> > > > style is broken.  :)
> > > 
> > > The ResizeHintOverride is already fixed in CVS.
> > > 
> > > Note:  For some reason the opposite style of ResizeHintOverride
> > > was NoResizeOverride (not NoResizeHintOverride).
> > 
> > Heh.  Many thanks.
> 
> I have backed out the patch and will think about a different
> solution.

I've committed a different patch.  Can you try that please?

Ciao

Dominik ^_^  ^_^

 --
Dominik Vogt, [EMAIL PROTECTED]

Attachment: signature.asc
Description: Digital signature

Reply via email to