<table><tr><td style="">ngraham accepted this revision.<br />ngraham added a comment.<br />This revision is now accepted and ready to land.
</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/D13805">View Revision</a></tr></table><br /><div><div><p>Nice, I think this is looking great. +1 on the latest wording. A lovely first patch!</p>

<p>To land it for you, we'll need your full name and email address, so it would be great if you could provide that now. In the future, if you submit your patch using <tt style="background: #ebebeb; font-size: 13px;">arc</tt> (see <a href="https://community.kde.org/Infrastructure/Phabricator#Using_Arcanist_to_post_patches" class="remarkup-link" target="_blank" rel="noreferrer">https://community.kde.org/Infrastructure/Phabricator#Using_Arcanist_to_post_patches</a>) this will happen automatically via the git authorship information, so we won't have to bug you.</p>

<p>Any comments from the <a href="https://phabricator.kde.org/tag/frameworks/" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">#frameworks</a> folks?</p></div></div><br /><div><strong>INLINE COMMENTS</strong><div><div style="margin: 6px 0 12px 0;"><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D13805#inline-72735">View Inline</a><span style="color: #4b4d51; font-weight: bold;">tmarshall</span> wrote in <span style="color: #4b4d51; font-weight: bold;">knewfilemenu.cpp:873</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">I get <tt style="background: #ebebeb; font-size: 13px;">newfilemenu.cpp:873:74: error: ‘Error’ is not a member of ‘QMessageBox’</tt></p>

<p style="padding: 0; margin: 8px;"><tt style="background: #ebebeb; font-size: 13px;">QMessageBox::Critical</tt> seems to work though.</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">Correct: <tt style="background: #ebebeb; font-size: 13px;">QMessageBox::Error</tt> does not exist; it's <tt style="background: #ebebeb; font-size: 13px;">QMessageBox::Critical</tt>. See <a href="https://doc.qt.io/qt-5/qmessagebox.html#Icon-enum" class="remarkup-link" target="_blank" rel="noreferrer">https://doc.qt.io/qt-5/qmessagebox.html#Icon-enum</a></p></div></div></div></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/D13805">https://phabricator.kde.org/D13805</a></div></div><br /><div><strong>To: </strong>tmarshall, Dolphin, Frameworks, ngraham<br /><strong>Cc: </strong>tmarshall, bruns, ngraham, kde-frameworks-devel, michaelh, spoorun, navarromorales, isidorov, firef, andrebarros, emmanuelp<br /></div>