GNU bug report logs - #2546
NS: toolbar height miscalculated

Previous Next

Packages: emacs, ns;

Reported by: David Reitter <david.reitter <at> gmail.com>

Date: Mon, 2 Mar 2009 23:35:03 UTC

Owned by: David Reitter <david.reitter <at> gmail.com>

Severity: normal

Done: David Reitter <david.reitter <at> gmail.com>

Bug is archived. No further changes may be made.

Full log


View this message in rfc822 format

From: David Reitter <david.reitter <at> gmail.com>
To: Adrian Robert <adrian.b.robert <at> gmail.com>, 2546 <at> debbugs.gnu.org
Subject: bug#2546: toolbar bug
Date: Fri, 6 Mar 2009 10:42:21 -0500
[Message part 1 (text/plain, inline)]
On 6 Mar 2009, at 09:56, Adrian Robert wrote:

> The calculation method that was commented out in x_set_window_size()  
> works here, and I've committed it to CVS.  As mentioned in the  
> comment, this would sometimes fail previously if the toolbar had not  
> been displayed yet.  I believe the relevant call would be [toolbar  
> setVisible: true] and this normally happens in nsmenu.c:  
> ns_update_toolbar().  When a new frame is created, it is initially  
> set to NOT visible in EmacsView-initFrameFromEmacs, because it will  
> be turned on if needed and after it is ready by  
> ns_update_toolbar().  However, if this gets called AFTER  
> x_set_window_size(), it could cause the failure.  If you still see  
> it in your environment, could you try throwing in a call to [[view  
> toolbar] setVisible: true] in the 'if (tb)' section before calling  
> frameRectForContentRect, and see if that helps?

OK, I've seen your patch now; this works for me.

The resizing problems persist.  Resizing happens only for each full  
column or row, but the it's jumping around badly, as if the whole  
window content is drawn one row too low first.


[smime.p7s (application/pkcs7-signature, attachment)]

This bug report was last modified 16 years and 80 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.