Shorter (slightly) reply ;-)<br><br><div class="gmail_quote">2009/6/24 Enrico Tröger <span dir="ltr"><<a href="mailto:enrico.troeger@uvena.de">enrico.troeger@uvena.de</a>></span><br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
On Wed, 24 Jun 2009 16:53:39 +1000, Lex wrote:<br>
<br>
<div class="im"><br>
</div>Great, and thanks for this short mail :).</blockquote><div><br>Thank you very much for reading it, I'm trying to express the essential simplicity of  the system to put in the manual and this is the shortest I've come up with to date, not quite there maybe, but definitely better than previous attempts.<br>
</div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><br>
<div class="im"><br>
<br></div><div class="im">>    1. label: "_Make All", command: "make"<br>
<br>
</div>Maybe we should label it actually "Make" by default but this is really<br>
an unimportant detail for now, can be changed afterwards easily (and<br>
yes, hardcoding 'make all' was very wrong in the first place, we<br>
already talked about my mistake :D).</blockquote><div> </div><div> I was just making it the same as 0.17, as you say it doesn't matter we can change it anytime.<br></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div class="im"><br>
<br>
>    2. label: "Make Custom _Target", command: "make ", but on<br><br>
</div>I think we could add some extra logic here that checks the first<br>
character of the user supplied string to be space and if not, insert<br>
it. Or even better, unconditionally add the space as it won't hurt at<br>
all.<br>
<div class="im"></div></blockquote><div>I actually thought that it was better not to impose that there had to be a space, for all I know someone has a situation where they want to directly catenate the typed input to the command, or more likely to the last argument.  Putting the space in the configurable part means that we don't rule out things just because we can't think of a use right now. <br>
</div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><div class="im"><br>
<br>
>The execute menu item changes to "_Stop" and the icon changes to stop<br>
>when a command initiated from the execute menu item is running and if<br>
>activated the menu item will attempt to kill the executing command.<br>
<br>
</div>I read this as we have only one Run command? This is completely ok and<br>
sufficient IMO. The only thing which bites in is our famous LaTeX<br>
specials, yay. But I would say we could convenience users that usually<br>
one Run command is even enough for LaTeX. The rationale behind the two<br>
We have one Run command and that's fine.</blockquote><div> Yes only one run command at the moment.<br></div><div>Note that the implementation is such that if it *really* is a problem then the extra run command can be added back in reasonably easily (same goes for adding extra commands to any of the configurable sections) by changing some enums and tables.  Since execute commands inherit from filetype, the menu could then inherit two different execute commands for lAtEx from the filetype and the extra menu item would be ignored (and not visible) for everything else (or at least until someone found a use for it ;-)  For now I will stick to one and lets see how much trouble it causes.<br>
 </div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">Of course, this requires you have saved your changes locally already in<br>
a patch or something to not throw away everything you have done so<br>
far :).<br>
</blockquote><div class="h5">Well I was just going to check out the new branch into a different directory and copy over the changed files, I use bazaar for my personal change control so it will tell me which ones to copy.<br>

<br>Cheers<br>Lex<br></div></div><br>