<table><tr><td style="">TallFurryMan planned changes to this revision.<br />TallFurryMan added inline comments.
</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/D23892">View Revision</a></tr></table><br /><div><strong>INLINE COMMENTS</strong><div><div style="margin: 6px 0 12px 0;"><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D23892#inline-135052">View Inline</a><span style="color: #4b4d51; font-weight: bold;">mutlaqja</span> wrote in <span style="color: #4b4d51; font-weight: bold;">focus.ui:9</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">Width must be less than 800 pixels to conform with the rest of the Ekos interface.</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">Actually, this has no impact on the final UI rendering. That width is only used in the designer, the final width in the application is constrained by the layout rules in the manager. As it happens, the whole view in the designer does not reflect the view in the app, it simply tries its best as it depends on the theme and personalization.</p></div></div><br /><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D23892#inline-135054">View Inline</a><span style="color: #4b4d51; font-weight: bold;">mutlaqja</span> wrote in <span style="color: #4b4d51; font-weight: bold;">fitsview.cpp:946</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">That seems a bit too large when it is subframed. Also, the number is clipped. Maybe it should be Above the star? or maybe it should be smaller in this case?</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">Originally, I made the font size independent from the zoom level. But noise when zoomed made the string unreadable. So I chose to make the font zoom dependent, but with the generic 1280x1024 resolution, the font was far too small in the 800x600 focus UI :) therefore I multiplied by this arbitrary 3.</p>

<p style="padding: 0; margin: 8px;">But I think what you really mean is that the number is clipped when subframing. Thanks, I'll remove HFR printout when subframing, that's not useful with a single star. Correct understanding?</p></div></div><br /><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D23892#inline-135053">View Inline</a><span style="color: #4b4d51; font-weight: bold;">mutlaqja</span> wrote in <span style="color: #4b4d51; font-weight: bold;">fitsview.cpp:952</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">for(const auto &center: imageData->getStarCenters())</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">Will fix. I'm not a fan of auto variables though, they have a tendency to leave unverified code behind if items change their type, and new functions change their meaning.</p></div></div></div></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/D23892">https://phabricator.kde.org/D23892</a></div></div><br /><div><strong>To: </strong>TallFurryMan, mutlaqja, alexcherney, wreissenberger<br /><strong>Cc: </strong>kde-edu, narvaez, apol<br /></div>