<table><tr><td style="">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/D19511">View Revision</a></tr></table><br /><div><div><p>So, then I assume you would be pro this patch.</p>

<p>I notice one behavior that would be against this patch but not pro the meaning of "Read Only Mode". When you set this setting and issue a save request you get the "Save as" dialog. Such behavior is good in any case. But such option, working as the current one, should then be called like, "Don't Overwrite File".</p>

<p>When you open as normal user a not writeable file, like /etc/fstab, I would somehow expect to see some warning and/or to enter the "Read Only Mode". But that's not the case. When you issue a save request you will be promted for admin rights, which is good in that example, no doubts.</p>

<p>Then exist the case that an "dangerous/odd" file, e.g. with too long lines, will set to "Read Only Mode". So far I remember is that also only be done to protect the orig file to be overwritten, but not for some hard needs which couldn't be handled by KTextEditor, it's more some odd protection against bad edit performance. So here is also a "Don't Overwrite File" setting more needful at the long end.</p>

<p>So, have have the feeling "Read Only Mode" should be renamed and this patch trashed, and the lock icon patch presumably too :-/</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R39 KTextEditor</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D19511">https://phabricator.kde.org/D19511</a></div></div><br /><div><strong>To: </strong>loh.tar, KTextEditor, cullmann<br /><strong>Cc: </strong>ngraham, cullmann, kwrite-devel, kde-frameworks-devel, KTextEditor, gennad, domson, michaelh, bruns, demsking, sars, dhaumann<br /></div>