Bug 33888

Summary:	RFE: GNOME3 support
Product:	UIM	Reporter:	Akira TAGOH <akira>
Component:	helper: toolbar	Assignee:	uim-bugs
Status:	RESOLVED FIXED	QA Contact:
Severity:	normal
Priority:	medium
Version:	unspecified
Hardware:	Other
OS:	All
Whiteboard:
i915 platform:		i915 features:

Description Akira TAGOH 2011-02-03 19:26:37 UTC

a lot of APIs has been changed/obsoleted in GNOME3. there should be a kind of tasks to accomplish that in uim:

* immodule built against gtk3
* get rid of libgnome/libgnomeui dependencies
  http://live.gnome.org/LibgnomeMustDie
* update the applet code to get rid of bonobo deps
  libpanel-applet doesn't provide an interface for bonobo anymore.

Comment 1 Muneyuki Noguchi 2011-02-04 17:17:51 UTC

I removed the dependency of libgnomeui in trunk (r6932 and r6933).

Comment 2 Muneyuki Noguchi 2011-02-14 05:49:24 UTC

I added "--with-gtk3" option to ./configure in trunk, and ported the GTK+ 2 applications and the immodule to GTK+ 3.

To-Dos
 * helper/eggtrayicon.c (uim-toolbar-gtk3-systray)
   - Port egg_tray_icon_expose() and egg_tray_icon_set_colormap()
 * gtk/gtk-im-uim.c (im-uim.so, or GTK+ 3 immodule)
   - handle_key_on_toplevel() without accessing the need_im_reset variable directly

Comment 3 Muneyuki Noguchi 2011-02-18 14:06:46 UTC

>  * helper/eggtrayicon.c (uim-toolbar-gtk3-systray)
>    - Port egg_tray_icon_expose() and egg_tray_icon_set_colormap()
Done though uim_toolbar_trayicon_new() (helper/toolbar-common-gtk.c) must be rewritten because gtk_rc_parse_string() is called and this RC function doesn't have any effect in GTK+ 3.

>  * gtk/gtk-im-uim.c (im-uim.so, or GTK+ 3 immodule)
>    - handle_key_on_toplevel() without accessing the need_im_reset variable
> directly
I can't fix this because I didn't take part in the past discussion on the implementation of the GTK+ 2 immodule and don't know the reason need_im_reset is set to TRUE here.

Comment 4 Muneyuki Noguchi 2011-02-18 22:31:42 UTC

> Done though uim_toolbar_trayicon_new() (helper/toolbar-common-gtk.c) must be
> rewritten because gtk_rc_parse_string() is called and this RC function doesn't
> have any effect in GTK+ 3.
> 
Fix in r6970.

AFAIK, only "need_im_reset" problem in GTK+ 3 immodule is left as a remaining big issue in GTK+ 3/GNOME 3 porting.

Comment 5 Akira TAGOH 2011-02-21 00:02:49 UTC

http://sourceforge.jp/projects/anthy/lists/archive/dev/2006-January/002805.html

I suppose the above link was a mail on the list when binding the key event handler onto the toplevel widget was available. I guess you could check the behavior on firefox for that issue?

Comment 6 Muneyuki Noguchi 2011-05-05 05:22:44 UTC

(In reply to comment #5)
> http://sourceforge.jp/projects/anthy/lists/archive/dev/2006-January/002805.html
> 
> I suppose the above link was a mail on the list when binding the key event
> handler onto the toplevel widget was available. I guess you could check the
> behavior on firefox for that issue?

Though I haven't tested uim-gtk3 on GTK+3 version of Firefox (https://bugzilla.mozilla.org/show_bug.cgi?id=627699), "need_im_reset = TRUE" in uim-gtk2 isn't executed on Firefox 3.6. uim-gtk3 may not have to set the need_im_reset variable.

Comment 7 Muneyuki Noguchi 2011-05-06 21:52:44 UTC

I conformed that uim-toolbar-applet-gnome3 and GTK+3 immodule in trunk (r7016) work in Fedora 15 beta.

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.