Community
Participate
Working Groups
The automatic adding of GTK's "Input Methods" menu item to context menus can be disabled as described in http://dev.eclipse.org/newslists/news.eclipse.newcomer/msg20408.html . SWT inheirits this behaviour in most cases, but one place where it doesn't is StyledText, where the only criteria used is whether KeyDown or KeyUp is hooked. Showing a context menu in a StyledText (and possibly other emulated controls) should check this property in addition to the KeyDown/KeyUp-is-hooked check when deciding whether to add this item.
I think this is not only wrong in the StyledText widget. We use a GraphViewer from the zest framework and every context menu in the viewer shows the "Input Methods" menu even though this option has been disabled with the gconf-editor.
(In reply to comment #1) > I think this is not only wrong in the StyledText widget. We use a GraphViewer > from the zest framework and every context menu in the viewer shows the "Input > Methods" menu even though this option has been disabled with the gconf-editor. The fix would affect all custom controls.
Created attachment 167648 [details] disable im option as required note: patch does not accept space in filename/directory. I have used the below to build rpm package. Patch200: eclipse-gtk-im-menu-java.patch Patch201: eclipse-gtk-im-os-java.patch pushd plugins/org.eclipse.swt/Eclipse\ SWT/gtk/org/eclipse/swt/widgets/ %patch200 -p0 popd pushd plugins/org.eclipse.swt/Eclipse\ SWT\ PI/gtk/org/eclipse/swt/internal/gtk %patch201 popd
This patch looks correct, but I was not able to verify the case where the menu should show. The input method menu should show in StyledText always when it shows for the native text widget. Do you know how I can change the gtk setting to force the input method menu to show for the native text control ?
Presumably adding the following lines in .gtkrc-2.0 would do it? gtk-show-input-method-menu = 1 gtk-show-unicode-menu = 1
Hi @Felipe Heidrich This should do - https://bugzilla.redhat.com/show_bug.cgi?id=590519#c1
(In reply to comment #6) > Hi @Felipe Heidrich > > This should do - https://bugzilla.redhat.com/show_bug.cgi?id=590519#c1 Thank you, that works. The patch is correct, but it should be easy enough to rewrite the patch to handle dynamic changes in the preference (so eclipse can "see" the change without having to be restarted).
Created attachment 170565 [details] patch After 3.6 (at this point only very critical bugs can be included in the build) Thanks for all the help
Fixed in HEAD > 20100809
