I'll take care of these issues. I put 2 in one row because there is a limit as to how many rows can be in the general tab and we've hit that limit :-/ I'm not sure how that is being handled or if we want to increase the # but if I increase the row by one it shoots out an error for me. Let me know how to proceed, I'll deal with the other two issues as soon as possible. Thanks for the comments<div>
<br></div><div>Joel<br><br><div class="gmail_quote">On Mon, Aug 6, 2012 at 11:28 AM, Ivan Timofeev <span dir="ltr"><<a href="mailto:timofeev.i.s@gmail.com" target="_blank">timofeev.i.s@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Joel,<div class="im"><br>
<br>
On 02.08.2012 21:57, Joel Madero wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Here is an updated patch.<br>
</blockquote>
<br></div>
not that it is essential, but it is good if the commit message is wrapped so that it does not exceed a reasonable width (say 70-80 characters). (Yes, my e-mail client seems unable to do that, ugh.)<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
 Please enter the commit message for your changes. Lines starting<br>
 with '#' will be ignored, and an empty message aborts the commit.<br>
</blockquote>
<br>
Something wrong with your editor? These lines should start with '#' and be ignored, why they are in the commit message again?<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Text [ en-US ] = "Optional features" ;<br>
</blockquote>
<br>
uhm, IMHO "optional" means "not necessary in most cases, but sometimes might be helpful". So a user would think "Ok, let's enable all these, won't hurt anyway." But this features aren't stable and somewhat dangerous to enable, and<br>

<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
-        Text [ en-US ] = "Enable experimental (unstable) features";<br>
+        Text [ en-US ] = "Enable experimental features";<br>
</blockquote>
<br>
makes it sound even more safe. Maybe "Unstable features" as a header then?<br>
<br>
Another nit-picking:<br>
you placed two labels in one row, not good IMHO: consider long translations. But then the last row could go out of the visible area, it would be great to check that - play with "#define ROW.." constants in<br>
optgdlg.hrc, remove my ugly +6 in ROW14 etc... :)<br>
<br>
Thanks!<span class="HOEnZb"><font color="#888888"><br>
<br>
Ivan<br>
</font></span></blockquote></div><br></div>