<blockquote>
<blockquote>
<p>Ok, well, unless <a href="https://github.com/shiftee" class="user-mention">@shiftee</a> wants to make it so, thats not part of this PR<br>
Why is dealing with bad interactions between features not part of a PR unless the submitter (solely) wants to make the change?</p>
</blockquote>
</blockquote>

<p>The submitter can always say they don't want to do something, thats always their choice.  Not doing something may affect the chances of the PR being accepted, but its always the submitter's choice.</p>

<p>I have said I don't think it should affect the chances, and you have said you disagree, so its up to somebody else to determine then.</p>

<blockquote>
<blockquote>
<p>But until its done the default should be off.<br>
I agree, but it can be done as part of this PR if we want to, by anyone, and is far simpler to do than the testing of this feature with all the related features/options will be before merging, so if nobody beats me to it, I might just do it myself while testing.</p>
</blockquote>
</blockquote>

<p>Of course you can make another PR with the extra functionality, thats the point of open source. :)</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/1246#issuecomment-249753296">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/ABDrJz9RrK7O6pARFNcPm2cpkkj8q3ZOks5quISxgaJpZM4KGa-x">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/ABDrJ4wiICHB8kNbaI5xJ0iRgQ7ThhNqks5quISxgaJpZM4KGa-x.gif" width="1" /></p>
<div itemscope itemtype="http://schema.org/EmailMessage">
<div itemprop="action" itemscope itemtype="http://schema.org/ViewAction">
  <link itemprop="url" href="https://github.com/geany/geany/pull/1246#issuecomment-249753296"></link>
  <meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>

<script type="application/json" data-scope="inboxmarkup">{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/geany/geany","title":"geany/geany","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/geany/geany"}},"updates":{"snippets":[{"icon":"PERSON","message":"@elextr in #1246: \u003e \u003e Ok, well, unless @shiftee wants to make it so, thats not part of this PR\r\n\u003e Why is dealing with bad interactions between features not part of a PR unless the submitter (solely) wants to make the change?\r\n\r\nThe submitter can always say they don't want to do something, thats always their choice.  Not doing something may affect the chances of the PR being accepted, but its always the submitter's choice.\r\n\r\nI have said I don't think it should affect the chances, and you have said you disagree, so its up to somebody else to determine then.\r\n\r\n\u003e \u003e But until its done the default should be off.\r\n\u003e I agree, but it can be done as part of this PR if we want to, by anyone, and is far simpler to do than the testing of this feature with all the related features/options will be before merging, so if nobody beats me to it, I might just do it myself while testing.\r\n\r\nOf course you can make another PR with the extra functionality, thats the point of open source. :)"}],"action":{"name":"View Pull Request","url":"https://github.com/geany/geany/pull/1246#issuecomment-249753296"}}}</script>