<blockquote>
<p>Also shouldn't the use of <code>vte_terminal_im_append_menuitems()</code> be just dropped?</p>
</blockquote>

<p>I'm not sure.  At least on GTK2 I think widgets were supposed to display a way to select the input method -- just right click on any GtkEntry from a GTK2 app and you'll see the item.  Apparently it was <a href="https://developer.gnome.org/gtk3/stable/GtkSettings.html#GtkSettings--gtk-show-input-method-menu">removed from GTK 3.10 somehow</a>, but I have no idea hos it works now -- well, nor before, I don't know anything about input methods.  But if it's something the widget has to implement another way (say, an interface so GTK can do some magic on its side), we'd need to be sure not to remove the element from a VTE that doesn't have that other thing.</p>

<p>Basically, as said in my first comment on <a href="https://github.com/geany/geany/issues/336" class="issue-link js-issue-link" data-url="https://github.com/geany/geany/issues/336" data-id="43783222" data-error-text="Failed to load issue title" data-permission-text="Issue title is private">#336</a> I don't really know hot to deal with this removal, and while it seems that on recent GTK3 version it's not needed (or not displayed?), I wouldn't remove it on other VTE versions unless I actually <em>knew</em> what I was changing.<br>
And no, commit that removes this on the VTE repo isn't clear: <a href="https://git.gnome.org/browse/vte/commit/?id=2da7058c57fd27bb8a8955b50a7a5b2ae8e6d50f">2da7058c57fd27bb8a8955b50a7a5b2ae8e6d50f deprecated it</a> and <a href="https://git.gnome.org/browse/vte/commit/?id=985fd375c25345933bcd864d4c8be302c3a54eb8">985fd375c25345933bcd864d4c8be302c3a54eb8 removed it</a>.</p>

<p>Writing this, I dug in GTK docs, and I'd think that since 3.10 widgets aren't supposed to show a way of switching the input method, and I guess instead the DE is supposed to handle that globally somehow.  But that's still a guess.</p>

<p>Possibly we should show this item on GTK < 3.10, and surely not show it if <a href="https://developer.gnome.org/gtk3/stable/GtkSettings.html#GtkSettings--gtk-show-input-method-menu"><code>gtk-show-input-method-menu</code></a> is set to <code>FALSE</code>.</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/1181#issuecomment-241398561">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/ABDrJ-nJTdJX-gJf6A5tDSQGt6EANNtQks5qiZbZgaJpZM4Jog38">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/ABDrJ1BuO1V18gFMZT_iyBWEQTZJdbfZks5qiZbZgaJpZM4Jog38.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/1181#issuecomment-241398561"></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":"@b4n in #1181: \u003e Also shouldn't the use of `vte_terminal_im_append_menuitems()` be just dropped?\r\n\r\nI'm not sure.  At least on GTK2 I think widgets were supposed to display a way to select the input method -- just right click on any GtkEntry from a GTK2 app and you'll see the item.  Apparently it was [removed from GTK 3.10 somehow](https://developer.gnome.org/gtk3/stable/GtkSettings.html#GtkSettings--gtk-show-input-method-menu), but I have no idea hos it works now -- well, nor before, I don't know anything about input methods.  But if it's something the widget has to implement another way (say, an interface so GTK can do some magic on its side), we'd need to be sure not to remove the element from a VTE that doesn't have that other thing.\r\n\r\nBasically, as said in my first comment on #336 I don't really know hot to deal with this removal, and while it seems that on recent GTK3 version it's not needed (or not displayed?), I wouldn't remove it on other VTE versions unless I actually *knew* what I was changing.\r\nAnd no, commit that removes this on the VTE repo isn't clear: [2da7058c57fd27bb8a8955b50a7a5b2ae8e6d50f deprecated it](https://git.gnome.org/browse/vte/commit/?id=2da7058c57fd27bb8a8955b50a7a5b2ae8e6d50f) and [985fd375c25345933bcd864d4c8be302c3a54eb8 removed it](https://git.gnome.org/browse/vte/commit/?id=985fd375c25345933bcd864d4c8be302c3a54eb8).\r\n\r\nWriting this, I dug in GTK docs, and I'd think that since 3.10 widgets aren't supposed to show a way of switching the input method, and I guess instead the DE is supposed to handle that globally somehow.  But that's still a guess.\r\n\r\nPossibly we should show this item on GTK \u003c 3.10, and surely not show it if [`gtk-show-input-method-menu`](https://developer.gnome.org/gtk3/stable/GtkSettings.html#GtkSettings--gtk-show-input-method-menu) is set to `FALSE`."}],"action":{"name":"View Pull Request","url":"https://github.com/geany/geany/pull/1181#issuecomment-241398561"}}}</script>