<table><tr><td style="">yurchor marked an inline comment as done.<br />yurchor 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/D17120">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/D17120#inline-93149">View Inline</a><span style="color: #4b4d51; font-weight: bold;">tcanabrava</span> wrote in <span style="color: #4b4d51; font-weight: bold;">view.cpp:3691-3712</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">It seems that you are interpolating by hand, why don't you try to use something like a QEasingCurve + a QPropertyAnimation?</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">Because switching to QPropertyAnimation cannot be done without complete rewriting half of KmPlot classes and breaks the whole current structure. There should be implemented some property of View to hold the current viewport to tie it up with QPropertyAnimation, then it should be connected with some timer and updates, and maybe other things, I'm not a skilled Qt programmer... It is easy to use QEasingCurve and QPropertyAnimation when the widget is ready for this. View is a different thing now. It might be the stunning implementation with 6 lines, but it is impossible now.</p>

<p style="padding: 0; margin: 8px;">The proposed patch just works. People wanted this for more than 10 years (21 votes).</p>

<p style="padding: 0; margin: 8px;">Sorry.</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/D17120#inline-93148">View Inline</a><span style="color: #4b4d51; font-weight: bold;">tcanabrava</span> wrote in <span style="color: #4b4d51; font-weight: bold;">view.h:637-640</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">What's TL and BR? Be explicit in the code.</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">"TopLeft" and "BottomRight". Fixed in the new version.</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R334 KmPlot</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D17120">https://phabricator.kde.org/D17120</a></div></div><br /><div><strong>To: </strong>yurchor, KDE Edu, tcanabrava<br /><strong>Cc: </strong>tcanabrava, kde-edu, kde-doc-english, skadinna, narvaez, apol<br /></div>