<table><tr><td style="">tobiasdeiminger 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/D16941">View Revision</a></tr></table><br /><div><div><p><a href="https://phabricator.kde.org/p/aacid/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@aacid</a> When is the last chance to get something into 18.12 release? Already passed?</p>

<p>I'm asking because we have two interfering issues here, where the first should be reasonably fixed with this diff (test still missing), but the second may take longer. Maybe we can split it, and get the first in?</p>

<ol class="remarkup-list">
<li class="remarkup-list-item">inconsistent calculation of viewport placement (fixed now)</li>
<li class="remarkup-list-item">viewport displaced a bit again, when <tt style="background: #ebebeb; font-size: 13px;">finished</tt> signals arrive from pixmap generation thread asynchronously, causing <tt style="background: #ebebeb; font-size: 13px;">notifyPageChanged( page, DocumentObserver::BoundingBox )</tt> and <tt style="background: #ebebeb; font-size: 13px;">PageView::slotRelayoutPages</tt></li>
</ol>

<p>If there's a chance to get 1 in, I'll focus on it (i.e., implement a test).</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R223 Okular</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D16941">https://phabricator.kde.org/D16941</a></div></div><br /><div><strong>To: </strong>tobiasdeiminger, Okular<br /><strong>Cc: </strong>sander, aacid, okular-devel, ngraham<br /></div>