Bug 657165

Summary: [gnome3] ibus applet appearing in bottom popup notification area not panel
Product: [Fedora] Fedora Reporter: Jens Petersen <petersen>
Component: gnome-shellAssignee: Owen Taylor <otaylor>
Status: CLOSED CURRENTRELEASE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: i18n-bugs, icj, maxamillion, otaylor, petersen, samkraju, shawn.p.huang, tagoh, tfujiwar, walters
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2011-04-01 02:18:59 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 617261    
Attachments:
Description Flags
Patch for statusIconDispatcher.js
none
Patch for panel.js
none
Patch for statusIconDispatcher.js none

Description Jens Petersen 2010-11-25 05:48:23 UTC
Description of problem:
When running ibus in F15 rawhide with gnome-shell
the ibus status applet seems to appear in
the popup notification area at the bottom
of the desktop used for notification event
and not in the panel where it should be.

I suppose the applet needs some tweaking
to be recognised a permanent status object
and not a temporary event.

How reproducible:
every time

Steps to Reproduce:
1. start ibus in gnome-shell or run it with ibus
  
Actual results:
applet appears in event notification area

Expected results:
applet to be up on the panel

Additional info:
Works ok in F14 gnome-shell

Comment 1 fujiwara 2010-12-02 09:42:37 UTC
The top icons are hard-coded in /usr/share/gnome-shell/js/ui/statusIconDispatcher.js .

We agreed to use "ibus-ui-gtk" officially for the ibus panel icon.
We will set the name in ibus later.
However currently ibus uses pygtk2 and it may be good to replace pygtk2 with gobject-introspection.

When I called gtk_status_icon_set_name() in either gtk2 or gtk3, the WM_CLASS name is not changed.

At the moment, could you register "ibus-ui-gtk" in statusIconDispatcher.js ?

Comment 2 fujiwara 2010-12-02 09:47:43 UTC
Created attachment 464207 [details]
Patch for statusIconDispatcher.js

Attached the patch.

Comment 3 fujiwara 2010-12-10 07:53:46 UTC
I could get a chance to talk with upstream.
My understanding is, I need to provide imports.ui.status.ibus.Indicator for gnome-shell.
I'll investigate it.

Comment 4 fujiwara 2011-02-03 09:47:36 UTC
Created attachment 476733 [details]
Patch for panel.js

I'd like to add the plugin file. The attached patch is able to check the existence of /usr/share/ibus/ui/gjs/ibusindicator.js

Now I rewrote the basic python codes to gjs ones.
The patch is available in my gjs branch.
https://github.com/fujiwarat/ibus/tree/gjs
https://github.com/fujiwarat/ibus/commit/8d057045bd0e31046745d601855418a21e7d5f5d

I also pushed the same /usr/share/ibus/ui/gjs part here:
http://fujiwara.fedorapeople.org/ibus/gnome-shell/gjs.tar.bz2

Comment 5 fujiwara 2011-02-28 01:46:16 UTC
*** Bug 680586 has been marked as a duplicate of this bug. ***

Comment 6 Fedora Update System 2011-03-25 06:25:51 UTC
ibus-1.3.99.20110228-5.fc15 has been submitted as an update for Fedora 15.
https://admin.fedoraproject.org/updates/ibus-1.3.99.20110228-5.fc15

Comment 7 fujiwara 2011-03-25 06:27:07 UTC
Created attachment 487481 [details]
Patch for statusIconDispatcher.js

Reverted the patch of the old shell indicator for GNOME 3.0.

Now the part of ibus was integrated in ibus-1.3.99.20110228-5.fc15:
https://admin.fedoraproject.org/updates/ibus-1.3.99.20110228-5.fc15

Comment 8 Fedora Update System 2011-03-30 05:34:04 UTC
ibus-1.3.99.20110228-6.fc15 has been pushed to the Fedora 15 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 9 fujiwara 2011-04-01 02:18:59 UTC
I think this is fixed by gnome-shell-2.91.93-1.fc15 or later and the latest ibus.