<table><tr><td style="">murveit added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D25645">View Revision</a></tr></table><br /><div><div><p>Thanks again for testing and checking out the interface.</p>

<p>I have fixed the issue with floating input. Please re-test.</p>

<p>To address some of the other comments:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">Buttons vs CheckBox: I can happily move to buttons. FYI, the reason I did it as I did was this way it is obvious whether stretching is on or off.</li>
</ul>

<p>With buttons, I'm  not sure whether a highlighted button means on or off. Further, if I change the label to say "Stretch" or "Unstretched", which <br />
would you use when it is stretched? It could mean the current state, or it could mean the state you want after the button is pushed. <br />
Anyway, let me know what you want specifically and I'll implement that.</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">Grainy while using the slider: Yes, I reduce the resolution (see e.g. fitstab.cpp, line 223, view->setSampling(4))</li>
</ul>

<p>while dragging to speed up the computation. Otherwise it can get laggy. I could make the sampling a function of the <br />
image size and # of channels, but perhaps that's fine tuning that should be done in a follow-up submission.</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">Auto vs Stretch: I see that can be confusing. They really are two different things, and you should be able to have one</li>
</ul>

<p>checked without the other. E.g. If you have the auto-stretch setting on, then when you load an image, you should see both<br />
checked.If you move the sliders, then Auto should go unchecked, as you've manually overridden the automatically generated <br />
parameters. If you then uncheck Stretch, then both would be unchecked and no stretching done. If you re-check stretch, <br />
then auto should stay unchecked, as it will remember the previous manual setting. Let me know if it doesn't work that way <br />
for you.  It is true, though, that if Stretch is unchecked, and you check Auto, both checks will appear, as setting Auto, to me, <br />
implies that you want to Stretch. Again, if this is confusing, let me know if you see a better way.</p>

<p>Please let me know if you see other issues or ways to improve.<br />
Hy</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R321 KStars</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D25645">https://phabricator.kde.org/D25645</a></div></div><br /><div><strong>To: </strong>murveit, mutlaqja, lancaster, wreissenberger<br /><strong>Cc: </strong>kde-edu, narvaez, apol<br /></div>