<p>Ah, yeah, it probably is fired twice in the case when</p>
<ol>
<li>the plugin doesn't consume the event, and</li>
<li>Geany doesn't consume the event</li>
</ol>
<p>Basically it means it's fired twice when the plugin isn't interested in the event which doesn't really matter for the vimode plugin but it definitely isn't nice. So yeah, we are back where we were and I don't know how to handle this with the single "key-press-event" used both for Geany and the plugin. You'd really need to change the order in which the signal is connected but I don't see a way how this could be done.</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/1829#issuecomment-383276093">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/ABDrJ-XvlEBqD2W-dsKL9efJP5DTqFulks5tquaGgaJpZM4TWsgF">mute the thread</a>.<img src="https://github.com/notifications/beacon/ABDrJyFwnouTI1x4IFcbtKCcIFJV7vIdks5tquaGgaJpZM4TWsgF.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/1829#issuecomment-383276093"></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":"@techee in #1829: Ah, yeah, it probably is fired twice in the case when\r\n\r\n1. the plugin doesn't consume the event, and\r\n2. Geany doesn't consume the event\r\n\r\nBasically it means it's fired twice when the plugin isn't interested in the event which doesn't really matter for the vimode plugin but it definitely isn't nice. So yeah, we are back where we were and I don't know how to handle this with the single \"key-press-event\" used both for Geany and the plugin. You'd really need to change the order in which the signal is connected but I don't see a way how this could be done."}],"action":{"name":"View Pull Request","url":"https://github.com/geany/geany/pull/1829#issuecomment-383276093"}}}</script>