<blockquote>
<p>Requiring the user to discover and enable plugins for even the most basic enhancements degrades Geany itself, and makes the plugin manager become a mess.</p>
</blockquote>
<p>You're making an argument that the current UX for plugins sucks, not that this PR specifically shouldn't be a plugin.</p>
<blockquote>
<p>Plus, those "just add one tiny enhancement" plugins are often fire-and-forget and become orphaned and unmaintained.</p>
</blockquote>
<p>So dumping all of those unmaintained fire-and-forget codes into core is better?</p>
<blockquote>
<p>Plugins should add complex functionality that the core devs aren't up to maintain. Having tons of micro plugins is worse overall than just adding 50-100 LOCs to the core (ideally along with some unit tests).</p>
</blockquote>
<p>Not IMO. Aside from the friction to install/update/remove plugins, there's not really any good reason not to have nearly everything in plugins. As mentioned, this isn't a reason to bring a bunch of plugins into core functionality, it's a reason to improve the plugin experience.</p>
<p>Anyway, I'm not opposed to doing what the PR title says ("Replace search dialog with search bar"), I'm just opposed to adding <em>another</em> search UI in addition to what exists, hidden behind a preference. If the search bar is truly superior to everyone, then we should remove the search dialog and replace it with the search bar, otherwise, it ought to be a plugin, IMO.</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/geany/geany/pull/2220?email_source=notifications&email_token=AAIOWJZ2LO7XGRNFTTCRP2LQBY7JTA5CNFSM4IEKN7D2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD27KU5A#issuecomment-515811956">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AAIOWJYDTSBEY7KH2XCT6QTQBY7JTANCNFSM4IEKN7DQ">mute the thread</a>.<img src="https://github.com/notifications/beacon/AAIOWJ2XRHY6AD4F6K7W5X3QBY7JTA5CNFSM4IEKN7D2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD27KU5A.gif" height="1" width="1" alt="" /></p>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/geany/geany/pull/2220?email_source=notifications\u0026email_token=AAIOWJZ2LO7XGRNFTTCRP2LQBY7JTA5CNFSM4IEKN7D2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD27KU5A#issuecomment-515811956",
"url": "https://github.com/geany/geany/pull/2220?email_source=notifications\u0026email_token=AAIOWJZ2LO7XGRNFTTCRP2LQBY7JTA5CNFSM4IEKN7D2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD27KU5A#issuecomment-515811956",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>