<div dir="ltr"><br><div class="gmail_extra"><div class="gmail_quote">On Mon, Jul 28, 2014 at 3:43 PM, Allan Odgaard <span dir="ltr"><<a href="mailto:mailinglist@textmate.org" target="_blank">mailinglist@textmate.org</a>></span> wrote:<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">

For showing diffs from a sheet, if we set TM_PROJECT_UUID to a newly generated UUID for the TM_MATE process then the diffs should open in its own window (might be good to generate one UUID per commit window session so all diffs go into same window).<br>
</blockquote><div> </div><div>Thanks. I thought this was possible but did not know which variable to set for TM_MATE. ;) Showing the diffs in a separate window, I think, is better in general so I will probably submit it as a separate pull request.</div>
<div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
I don’t think moving the commit window to the dialog plug-in is giving us any advantages other than maybe remove some code redundancy in that both the dialog plug-in and commit window controller sets up a connection for communicating between shell and TM, but this could maybe be factored out into reusable code.<br>


<br>
It might make more sense to migrate the dialog plug-in code into the main TextMate code base. The reason it was made as a plug-in was partly to allow it to be F/OSS (before TM’s core) and partly for separation of concerns.<br>
</blockquote><div><br></div><div>No problem, I just recalled you initially mentioning making the commit window part of dialog plug-in. I will start prepping the code base (style fixes, update to ARC, etc.)  for inclusion into TM's core, if that's OK.</div>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">

______________________________<u></u>_________________<br>
textmate-dev mailing list<br>
<a href="mailto:textmate-dev@lists.macromates.com" target="_blank">textmate-dev@lists.macromates.<u></u>com</a><br>
<a href="http://lists.macromates.com/listinfo/textmate-dev" target="_blank">http://lists.macromates.com/<u></u>listinfo/textmate-dev</a></blockquote></div><br></div></div>