<table><tr><td style="">rjvbb 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/D17794">View Revision</a></tr></table><br /><div><div><p>Yes, I'm hoping that others will chime in here on this aspect 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;"><div class="remarkup-code-block" style="margin: 12px 0;" data-code-lang="text" data-sigil="remarkup-code-block"><pre class="remarkup-code" style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; padding: 12px; margin: 0; background: rgba(71, 87, 120, 0.08);">I think that this functionality should be optional</pre></div></blockquote>

<p>AFAIC it's enough that use of the functionality is optional (which it is, just like the rest of the customdefinesand* plugin).<br />
Consider that by default all plugins are loaded, so a new plugin will show up in the project config dialog's sidebar and you have to open another dialog to disable the plugin if you don't want it. (And then you never really know in my experience how plugin selection carries over to other and/or new sessions). OTOH, if you just add a page to the customdefines plugin, e.g. as a "pkgconfig" tab between the "Defines" and the "C/C++ Parser" tabs that has much less chance of getting in anyone's way (horizontal screen space is always less at a premium than vertical space) while still being visible enough and IMHO in a logical location.</p>

<p>Bonus: no need to worry about what icon to use ;)</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R32 KDevelop</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D17794">https://phabricator.kde.org/D17794</a></div></div><br /><div><strong>To: </strong>blackwarthog, KDevelop<br /><strong>Cc: </strong>rjvbb, arrowd, kdevelop-devel, KDevelop, glebaccon, hase, antismap, iodelay, vbspam, geetamc, Pilzschaf, akshaydeo, surgenight<br /></div>