<table><tr><td style="">loh.tar updated this revision to Diff 53296.<br />loh.tar retitled this revision from "DocumentPrivate: Auto reload in read-only mode" to "DocumentPrivate: Add option "Enable Auto Reload" to ModOnHdPrompt".<br />loh.tar edited the summary of this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-74lrzy5tg25t7st/">(Show Details)</a><br />loh.tar edited the test plan for this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-zjitxcmdugdb4dq/">(Show Details)</a><br />loh.tar 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/D19517">View Revision</a></tr></table><br /><div><div><ul class="remarkup-list">
<li class="remarkup-list-item">Add option "Enable Auto Reload" to ModOnHdPrompt</li>
<li class="remarkup-list-item">Allow also auto reload for not read-only files</li>
<li class="remarkup-list-item">Only auto reload when doc is unmodified</li>
<li class="remarkup-list-item">Only reload when no reload is in progress, may that not enough to avoid mentioned lock up?</li>
</ul>

<p>Issues:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">The new button is slightly dangerous placed for my taste but with intend not nearby the normal reload button</li>
<li class="remarkup-list-item">No other way to disable that setting than to close/open the file</li>
<li class="remarkup-list-item">The reload is only triggered when the application has the focus, but that may be an intended behavior</li>
</ul>

<p>TODO(?)</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">Add getter function so that e.g. the status bar can show a suitable icon to indicate that mode and the ModOnHdPrompt can hide that button in case of an already active setting</li>
<li class="remarkup-list-item">Add some option/action to the view so that the view scrolls down after an auto reload</li>
</ul></div></div><br /><div><strong>CHANGES TO REVISION SUMMARY</strong><div><div style="white-space: pre-wrap; color: #74777D;">This patch a<span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">dd an a</span>uto reload <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">the document onl</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">functionalit</span>y when <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">it was not already<br />
modified and it is in read-only mode</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">the user answers the ModOnHdPrompt accordingly</span><br />
<br />
BUG:375361<div style="padding: 8px 0;">...</div></div></div></div><br /><div><strong>CHANGES TO TEST PLAN</strong><div><div style="white-space: pre-wrap; color: #74777D;"><div style="padding: 8px 0;">...</div>384384 Say "no tail -f" but it sound so<br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">Should it be accepted in general I would look to add such extra option before landing.</span></div></div></div><br /><div><strong>CHANGES SINCE LAST UPDATE</strong><div><a href="https://phabricator.kde.org/D19517?vs=53132&id=53296">https://phabricator.kde.org/D19517?vs=53132&id=53296</a></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D19517">https://phabricator.kde.org/D19517</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>src/dialogs/katedialogs.cpp<br />
src/dialogs/katedialogs.h<br />
src/document/katedocument.cpp<br />
src/document/katedocument.h</div></div></div><br /><div><strong>To: </strong>loh.tar, KTextEditor, cullmann<br /><strong>Cc: </strong>cullmann, kwrite-devel, kde-frameworks-devel, KTextEditor, gennad, domson, michaelh, ngraham, bruns, demsking, sars, dhaumann<br /></div>