<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/D14610">View Revision</a></tr></table><br /><div><div><p><a href="https://phabricator.kde.org/p/shubham/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@shubham</a> Thanks for helping out with <a href="https://phabricator.kde.org/T9297" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">T9297</a>!</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>reuse the above code that creates a QLabel</p></blockquote>

<p><a href="https://phabricator.kde.org/p/dfaure/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@dfaure</a> Thanks for the review! Any advice on how to handle <tt style="background: #ebebeb; font-size: 13px;">KPropertiesDialog::setFileNameReadOnly</tt>, which is used in <a href="https://lxr.kde.org/source/kde/workspace/plasma-workspace/applets/icon/iconapplet.cpp#0528" class="remarkup-link" target="_blank" rel="noreferrer">Plasma's <tt style="background: #ebebeb; font-size: 13px;">IconApplet</tt></a> and would still show the <tt style="background: #ebebeb; font-size: 13px;">KLineEdit</tt> if I understand your proposal correctly?</p>

<p>Also, <tt style="background: #ebebeb; font-size: 13px;">KFilePropsPlugin::setFileNameReadOnly</tt> checks for <tt style="background: #ebebeb; font-size: 13px;">m_bFromTemplate</tt>. Would this be relevant for your suggestion too?</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>the patch description is unclear</p></blockquote>

<p>The main motivation for the patch is to also indicate visually that an item's name cannot be changed. Currently users only notice that by chance due to typing into the line edit not being allowed.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R241 KIO</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D14610">https://phabricator.kde.org/D14610</a></div></div><br /><div><strong>To: </strong>shubham, rkflx, dfaure, ngraham<br /><strong>Cc: </strong>kde-frameworks-devel, michaelh, ngraham, bruns<br /></div>