<table><tr><td style="">ahmadsamir updated this revision to Diff 78241.<br />ahmadsamir retitled this revision from "Move/port KFontDialog from KDELibs4Support to KWidgetAddons" to "Copy KFontDialog from KDELibs4Support to KWidgetAddons, now KFontChooserDialog".<br />ahmadsamir edited the summary of this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-ejdrmg7ykzwfaf2/">(Show Details)</a><br />ahmadsamir edited the test plan for this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-jcll4guz3ffbx54/">(Show Details)</a><br />ahmadsamir removed a reviewer: dfaure.<br />ahmadsamir removed subscribers: dfaure, kossebau.<br />ahmadsamir 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/D28122">View Revision</a></tr></table><br /><div><div><p>Address comments:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">Drop the fontList parameter in the ctor, it's not useful, 99% of the times users of this class call the static getFont() methods</li>
<li class="remarkup-list-item">Make the parent QWidget * the last param. in the ctor, a la Qt API</li>
<li class="remarkup-list-item">Change stripRegularStyleName() to be file-static</li>
<li class="remarkup-list-item">Add <span class="phabricator-remarkup-mention-unknown">@since</span></li>
<li class="remarkup-list-item">Don't call setModal(true), exec() takes care of that</li>
<li class="remarkup-list-item">More const</li>
</ul></div></div><br /><div><strong>CHANGES TO REVISION SUMMARY</strong><div><div style="white-space: pre-wrap; color: #74777D;">- Drop relative size bits, seems not that <span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">useful or widely </span>used<div style="padding: 8px 0;">...</div>- Port to QDialog<br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);"><br />
See https://phabricator.kde.org/D27808 for more details about the reason</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">- Drop the fontList parameter in the ctor, it's not useful, 99% of the times<br />
  users of this class called the static getFont() method</span>s<br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">to make this port</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">- Make the parent QWidget * the last param. in the ctor, a la Qt API<br />
<br />
See https://phabricator.kde.org/D27808 for more details</span>.</div></div></div><br /><div><strong>CHANGES TO TEST PLAN</strong><div><div style="white-space: pre-wrap; color: #74777D;">it builds and kfont<span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">chooser</span>dialog works (see test app)</div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R236 KWidgetsAddons</div></div></div><br /><div><strong>CHANGES SINCE LAST UPDATE</strong><div><a href="https://phabricator.kde.org/D28122?vs=78153&id=78241">https://phabricator.kde.org/D28122?vs=78153&id=78241</a></div></div><br /><div><strong>BRANCH</strong><div><div>l-kfontdlg (branched from master)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D28122">https://phabricator.kde.org/D28122</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>docs/pics/kfontchooserdialog.png<br />
src/CMakeLists.txt<br />
src/kfontchooserdialog.cpp<br />
src/kfontchooserdialog.h<br />
tests/CMakeLists.txt<br />
tests/kfontchooserdialogtest.cpp</div></div></div><br /><div><strong>To: </strong>ahmadsamir, Frameworks, davidedmundson, cfeck, broulik, ervin, meven, bport, dfaure<br /><strong>Cc: </strong>kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, ngraham, bruns, dfaure, kossebau<br /></div>