<blockquote>
<p>The context doesn't work at that, of course. It should not be enabled: 1) when no text is selected</p>
</blockquote>
<p>IIUC its a deliberate feature to select the word under the cursor if no selection exists, so this will need an option defaulting to off so it won't affect the current behaviour unless the user wants it to.</p>
<blockquote>
<ol start="2">
<li>even more important: when there is no context action set.</li>
</ol>
</blockquote>
<p>Making the menu item insensitive in this case makes sense.</p>
<blockquote>
<p>This commit introduces %f, %d, %e, %p, %l placeholders into the context string, i.e. the same ones as in Compile/ Build/ Run.</p>
</blockquote>
<p>Seems a worthwhile improvement.</p>
<blockquote>
<p>the set context action is active when there is no selected text or a word under caret.</p>
</blockquote>
<p>Erm, didn't your item 1. just propose disabling it in this case?</p>
<p>Finally, this will <strong>NOT</strong> be accepted without the manual being modified to reflect the changes.</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/1841#issuecomment-384107256">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/ABDrJ02Yf7rvHA-6i6wh4HgEdounfbJ0ks5tr6-MgaJpZM4Thz7Y">mute the thread</a>.<img src="https://github.com/notifications/beacon/ABDrJ5yjlgN9b6lXZU_l2JRQplvsR11Nks5tr6-MgaJpZM4Thz7Y.gif" height="1" width="1" alt="" /></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/1841#issuecomment-384107256"></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 #1841: \u003e The context doesn't work at that, of course. It should not be enabled: 1) when no text is selected\r\n\r\nIIUC its a deliberate feature to select the word under the cursor if no selection exists, so this will need an option defaulting to off so it won't affect the current behaviour unless the user wants it to.\r\n\r\n\u003e 2) even more important: when there is no context action set.\r\n\r\nMaking the menu item insensitive in this case makes sense.\r\n\r\n\u003e This commit introduces %f, %d, %e, %p, %l placeholders into the context string, i.e. the same ones as in Compile/ Build/ Run.\r\n\r\nSeems a worthwhile improvement.\r\n\r\n\u003e  the set context action is active when there is no selected text or a word under caret. \r\n\r\nErm, didn't your item 1. just propose disabling it in this case?\r\n\r\nFinally, this will __NOT__ be accepted without the manual being modified to reflect the changes.\r\n\r\n"}],"action":{"name":"View Pull Request","url":"https://github.com/geany/geany/pull/1841#issuecomment-384107256"}}}</script>