<table><tr><td style="">pino 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/D14360">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #8C98B8;
          color: #6B748C;
          font-style: italic;
          margin: 4px 0 12px 0;
          padding: 8px 12px;
          background-color: #F8F9FC;">
<div style="font-style: normal;
          padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D14360#298208" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D14360#298208</a>, <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> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>pino: I don't why m_iconButton isn't needed(because it is being used by other entries also ), am I right?</p></div>
</blockquote>

<p>I don't understandwhat you mean, can you please rephrase it?</p>

<p>The point is: if the icon must not be edited, then even creating the <tt style="background: #ebebeb; font-size: 13px;">KiconButton</tt> for it is not useful, because it's an unused widget. Even more, Creating it and making it hidden still uses the resources needed to create it, and load the icon for it. So... just do not create it, taking care of handling in the dialog the case when <tt style="background: #ebebeb; font-size: 13px;">m_iconButton</tt> is null.</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/D14360">https://phabricator.kde.org/D14360</a></div></div><br /><div><strong>To: </strong>shubham, ngraham, broulik, Dolphin, Frameworks<br /><strong>Cc: </strong>pino, kde-frameworks-devel, michaelh, ngraham, bruns<br /></div>