<html><head><link itemprop="url" href="https://github.com/geany/geany/pull/672#issuecomment-152685983" /><meta itemprop="name" content="View Pull Request" /><meta itemprop="description" content="View this Pull Request on GitHub" /></head><body><div class="gmail_quote">Am 31. Oktober 2015 02:52:01 MEZ, schrieb Matthew Brush <notifications@github.com>:<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<blockquote>
<p>[...] so unless there are actual problems I don't see an evidence to keep the off default.</p>
</blockquote>
<p>I mentioned a number of issues with it in the mailing list thread linked from the PR you made to disable it by default.</p>
<p>I don't think we should cram this in just because of string freeze, especially with the open issues, the UI issues, and that we will be releasing again in a few months. I'm not opposed to the feature, I'm opposed to the implementation (re-using an existing feature to do something else, something unexpected of the existing feature, as well as the infobar without a checkbox that can be hidden from the user without ever seeing it).</p>
<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />Reply to this email directly or <a href="https://github.com/geany/geany/pull/672#issuecomment-152685983">view it on GitHub</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/AAidKMXvLSxK_3Ixj8GNj3yjHMx5VID0ks5pBBZBgaJpZM4GHluE.gif" width="1" /></p>
<div itemscope="itemscope" itemtype="http://schema.org/EmailMessage">
<div itemprop="action" itemscope="itemscope" itemtype="http://schema.org/ViewAction">
</div>
</div>
</blockquote></div><br clear="all">Carrying this discussion to the ML.<br>
<br>
Can you elaborate on the issues you see?<br>
<br>
As to the UI. It's important that we get any UI for 1.26, therefore we had to manage before string freeze. We can introduce a better UI fir 1.27. I'm not opposed to a check box but would be more effort. And it would presumably checked by default anyway, no?</body></html>