David Reitter <[EMAIL PROTECTED]> writes: > I've been having trouble with the old version of the patch when > changing font settings. If you change the font (e.g., with M-x set- > frame-font, or with the font panel), the frame resizes, and it is not > full-screen any longer. > > I haven't installed the new patch yet. It would be interesting to see > if things work better with that.
David, I had noticed that problem, too (and my patch did not address it). I tried right now to call fullscreen_hook from x_set_font (see patch below), and it seems to work. I think it's a little ugly, however. It would be better to intercept the lower-level code that actually changes the size (in pixels) of the Mac window, but I couldn't find it in the time I have available today. What do you think? Vebjorn --- ../../old/emacs.orig/src/frame.c 2008-06-02 16:25:50.000000000 -0400 +++ frame.c 2008-06-09 14:57:20.000000000 -0400 @@ -3142,6 +3142,9 @@ else abort (); + if (fullscreen_hook != NULL) + fullscreen_hook (f); + do_pending_window_change (0); /* Don't call `face-set-after-frame-default' when faces haven't been --~--~---------~--~----~------------~-------~--~----~ Carbon Emacs User Group http://groups.google.com/group/carbon-emacs?hl=en -~----------~----~----~----~------~----~------~--~---