<table><tr><td style="">rkflx created this revision.<br />rkflx added a reviewer: Frameworks.<br />Restricted Application added a project: Frameworks.
</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/D7859" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>Besides <tt style="background: #ebebeb; font-size: 13px;">Information</tt>, <tt style="background: #ebebeb; font-size: 13px;">Warning</tt> and <tt style="background: #ebebeb; font-size: 13px;">Critical</tt>, the <tt style="background: #ebebeb; font-size: 13px;">QMessageBox::Icon</tt><br />
enum supports icons of type <tt style="background: #ebebeb; font-size: 13px;">Question</tt>. Instead of falling back to an<br />
"information" icon, use a "question mark" icon for every<br />
<tt style="background: #ebebeb; font-size: 13px;">KMessageBox::question*()</tt>.</p>

<p>Note: Using the Oxygen icon set still displays an "information" icon<br />
(may need investigation).</p></div></div><br /><div><strong>TEST PLAN</strong><div><ul class="remarkup-list">
<li class="remarkup-list-item"><tt style="background: #ebebeb; font-size: 13px;">kmessageboxtest</tt> now shows question mark icon for some dialogs</li>
<li class="remarkup-list-item">cursory look at lxr suggest most usage is a proper question indeed</li>
<li class="remarkup-list-item">question mark icon supported in Breeze and Adwaita icon set</li>
</ul></div></div><br /><div><strong>REPOSITORY</strong><div><div>R236 KWidgetsAddons</div></div></div><br /><div><strong>BRANCH</strong><div><div>master</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D7859" rel="noreferrer">https://phabricator.kde.org/D7859</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>src/kmessagebox.cpp</div></div></div><br /><div><strong>To: </strong>rkflx, Frameworks<br /></div>