<table><tr><td style="">lancaster 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>That does look much better.  I am uncertain as to whether highlights is needed or not, we should try it on some bright planet or moon images before getting rid of it.</p>

<p>One way to alleviate some confusion between auto and stretch would be to have the "stretch" button enable/disable all the other controls, then one can get a visual cue that the stretch is gone other than just a checkbox.  The controls can still remember their values, but you can't change them until you enable stretch.</p>

<p>Another thing that could help would be to have the controls be at an absolute position and hitting auto will visually make them go to that spot in the slider.  Also moving them to a spot will make the slider stay at that spot.  I understand the utility of having the slider work the way you have done it for more dynamic control, but it is a little disconcerting at first to move the slider, have it change the image as you go, and then it snaps back to center when you let go.</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>