#846 make smart placement optional (with patch)

[Fons Rademakers]

Hi,

while smart placement is fine, I have to use for some jobs the windows in a repeatable set of places. I made the smart placement optional in the Preference Pane. The patch is attached. Also the patch fixes the issue when using CMD-W to close windows that the shell is not properly closed (close a number of windows a run the "w" command and you'll see the shells are not properly terminated). Hope you can merge this in the svn trunk.

[Fons Rademakers]

[Dan Wilga]

Thank you so much for posting this patch. I have confirmed that it works well, and I hope that it gets incorporated soon.

My only suggestion is that the preference be moved to the Window tab of the prefs pane, since it is a window-related pref.

[Fons Rademakers]

Hi Dan,

thanks for considering the patch. Of course no problem moving the preference to the Window tab. I assume you will do that when applying the patch.

Cheers, Fons.

[R. Lynn Rardin]

I'll add another thank you to Fons for working on this patch. I agree that smart window placement might work well for most users, but, more often than not, the work for which I use iTerm dictates that I need windows to open in certain locations on the display. And I need for the window locations to be preserved across restarts of iTerm. I'll probably stick with the previous release until Fons's patch or something similar is added. Barely a day goes by that I don't use iTerm. Great program!

[R. Lynn Rardin]

I decided to compile iTerm with this patch, rather than wait for an official release that includes it. Works great. Thanks again, Fons.

[George Nachman]

This patch has been integrated into iTerm2 and will be in the alpha-4 and later releases. Please visit the new project at http://iterm2.googlecode.com/

Thanks for the contribution! One question about it: why did you add a kill SIGHUP in closeWindow?