<table><tr><td style="">rkflx 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/D8838" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>Sorry for the delay, finally could do some testing. First I was quite enthusiastic about what I saw, but then some oddities came up so I cannot draw a conclusion on what I would recommend to do here yet.</p>
<p>Any final patch (i.e. not fulfilled by current patch version) should ensure IMO:</p>
<ul class="remarkup-list">
<li class="remarkup-list-item">Thumbnails are less important than the main view and therefore should not delay its rendering.</li>
<li class="remarkup-list-item">If we can currently render a fast document without any delay, we should not regress on that.</li>
</ul>
<p>Here is what seems odd:</p>
<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>files after okular is open don't suffer this</p></blockquote>
<p>IIUC, your explanation is about resize events when the window is opening. However, adding debug output to <tt style="background: #ebebeb; font-size: 13px;">resizeEvent</tt> (for <tt style="background: #ebebeb; font-size: 13px;">isEmpty()</tt> and for <tt style="background: #ebebeb; font-size: 13px;">!isEmpty()</tt>) as well as <tt style="background: #ebebeb; font-size: 13px;">slotRequestVisiblePixmaps</tt> shows that this is also happening for a simple <span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">Reload</span></span></span>. This suggests those events are entirely controlled by Okular code and may be fixable there?</p>
<p>Next, I reverted both <tt style="background: #ebebeb; font-size: 13px;">delayResizeEventTimer</tt> related changes to evaluate your fix to the second problem (tested with <tt style="background: #ebebeb; font-size: 13px;">dublin-center-street.pdf</tt> @ <span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">200% zoom</span></span></span>, with T: thumbnails render, M: main view renders, D: weird delay of several seconds):</p>
<div class="remarkup-table-wrap"><table style="border-collapse: separate;
border-spacing: 1px;
background: #d3d3d3;
margin: 12px 0;">
<tr><th style="background: #ffffff;
padding: 3px 6px;"></th><th style="background: #ffffff;
padding: 3px 6px;">master</th><th style="background: #ffffff;
padding: 3px 6px;">patch</th><th style="background: #ffffff;
padding: 3px 6px;">patch w/o timer</th></tr>
<tr><td style="background: #ffffff;
padding: 3px 6px;">start normal</td><td style="background: #ffffff;
padding: 3px 6px;">M-D-M-T-M</td><td style="background: #ffffff;
padding: 3px 6px;">T-D-M</td><td style="background: #ffffff;
padding: 3px 6px;">M-D-T-M</td></tr>
<tr><td style="background: #ffffff;
padding: 3px 6px;">reload normal</td><td style="background: #ffffff;
padding: 3px 6px;">M-D-M-T-M</td><td style="background: #ffffff;
padding: 3px 6px;">T-D-M</td><td style="background: #ffffff;
padding: 3px 6px;">M-D-T-M</td></tr>
<tr><td style="background: #ffffff;
padding: 3px 6px;">start rotated</td><td style="background: #ffffff;
padding: 3px 6px;">M-D-M-T-M</td><td style="background: #ffffff;
padding: 3px 6px;">T-D-M</td><td style="background: #ffffff;
padding: 3px 6px;">M-D-T-M</td></tr>
<tr><td style="background: #ffffff;
padding: 3px 6px;">reload rotated</td><td style="background: #ffffff;
padding: 3px 6px;">M-D-M-T-M</td><td style="background: #ffffff;
padding: 3px 6px;">T-D-M</td><td style="background: #ffffff;
padding: 3px 6px;">M-D-T-M</td></tr>
<tr></tr>
</table></div>
<p>Note:</p>
<ul class="remarkup-list">
<li class="remarkup-list-item">For normal documents the last M in -M-T-M would sometimes be omitted when choosing another initial window size.</li>
<li class="remarkup-list-item">For rotated documents the last M would sometimes affect only parts of the window.</li>
<li class="remarkup-list-item">T-D-M would sometimes turn into D-T-M when repeating the same test.</li>
<li class="remarkup-list-item">I still do not understand what D is about (in the other Diff you said "There's no text generation for main view and one for thumbnail, there is no reason to generate text twice", but this does not really answer the question I asked?).</li>
<li class="remarkup-list-item">Ideal log: M-T (or rather in parallel ;)</li>
</ul>
<p>In conclusion, I'm not convinced that the standalone fix for the second problem is completely working either. This is all very strange, maybe you could have another look?</p>
<p>I suspect there is still something else going on, so perhaps we are not yet seeing the root cause? Especially for rotated pages in some cases reloading results in more redraws compared to normal pages and removing window focus can provoke blank pages.</p>
<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>Ideally here the solution is page rendering cancellation</p></blockquote>
<p>Yeah, that would be great. Given the other solutions also have problems, this could be worth thinking about?</p>
<hr class="remarkup-hr" />
<p>Refs: <a href="https://phabricator.kde.org/F5477455" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: none;" rel="noreferrer">F5477455</a>, <a href="https://phabricator.kde.org/D8379#164630" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: line-through;" rel="noreferrer">D8379#164630</a></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/D8838" rel="noreferrer">https://phabricator.kde.org/D8838</a></div></div><br /><div><strong>To: </strong>aacid<br /><strong>Cc: </strong>rkflx, Okular, gassaf, ngraham, aacid<br /></div>