<div dir="ltr"><div>Hi<br><br>- Add a semicolon for JAR compression level<br></div>- I would set warning text to " " / "Available:xxx"  when limit is disabled (hence, not hiding neither jumping, but not showing wrong/invalid information)<br><br><div>Ok for me.<br><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Sep 4, 2014 at 8:39 PM, Lukasz Dracz <span dir="ltr"><<a href="mailto:ldracz@redhat.com" target="_blank">ldracz@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hello,<br>
<br>
Okay so this patch, uses disabling/enabling instead of hiding/showing. Also put the "Java stores application ..." description to always be at the top where as the rest of the components will be centered.<br>
Also changed from powers of two to increments of 10 but not jumps of 1,10,100,1000,10000 etc. but instead to<br>
0,1,2,3,...,7,8,9,10,20,30...80,90,100,200,300...800,900,1000,2000,3000 etc. I think this way is best as it does not jump to the larger values too quickly.<br>
<br>
As for my opinion, I think disabling is better than hiding, as it still shows the user the possibilities in the panel even though they are not active. I also liked the powers of two spinner much more personally but from a user's point of view I agree that increments of 10 are more intuitive and easier to understand.<br>
<br>
Thank you,<br>
Lukasz Dracz</blockquote></div><br></div>