Hi all, I packaged the shell extension with ibus for the approach 4. The package is available from obs://home:ftake:branches:M17N/openSUSE_12.3 Could you test it? After installing it, please enable the shell extension by your self and log out. I will submit it to M17N and forward to Factory if it is OK. Is there any possibility to add "enabled-extension"? If we do, users cannot disable the shell extension. So we need to consider carefully. # We can put other selected icons including fcitx on the top panel in the same way. Fuminobu TAKEYAMA (2013/02/13 4:32), Fuminobu TAKEYAMA wrote:

Hello, it's 4 a.m. in Japan,

I updated my shell extension. It should work fine with ibus on *RC1 DVD*. On 12.3 repo, there is ibus with hillwood's patch, which does not work with my extension because the patch removes code for the indicator. https://github.com/ftake/ibus_indicator-14

To test this, please run $ cd ~/.local/share/gnome-shell/extensions/ $ git clone git://github.com/ftake/ibus_indicator-14.git

and enable it by gnome-tweak-tool

...

Now, you need to change the default settings for gnome-shell.

See enabled-extensions key in https://build.opensuse.org/package/view_file?expand=1&file=glib2-branding.gschema.override.in&package=glib2-branding-openSUSE&project=GNOME%3AFactory

Thanks!

Now, I have five ideas: 1. Go with the current (hillwood's patched) ibus - Its icon is managed notification daemon (on the bottom of *Activity*) - A bit difficult to use (too small menu, not shown always) 2. Enable the extension by modifying glib2-branding-openSUSE 3. Put a part of the extension (only 17 lines of code) into gnome-shell's script by a patch (see below) - This *was* the best solution if we could notice this problem earlier beta 4. Users enable the extension manually by gnome-tweak-tool, and we update hillwood's patch to work with the extension

5. Call "gnome-shell-extension-tools -e ibus_indicator-14" from the ibus launch script to enable the extension

Fuminobu TAKEYAMA

geeko@linux-jpk4:~> diff -u /usr/share/gnome-shell/js/ui/notificationDaemon.js.back /usr/share/gnome-shell/js/ui/notificationDaemon.js --- /usr/share/gnome-shell/js/ui/notificationDaemon.js.back 2013-02-13 03:55:29.996264488 +0900 +++ /usr/share/gnome-shell/js/ui/notificationDaemon.js 2013-02-13 04:19:39.961930850 +0900 @@ -500,6 +500,21 @@

_onTrayIconAdded: function(o, icon) { let wmClass = icon.wm_class ? icon.wm_class.toLowerCase() : ''; + if (wmClass == 'ibus-ui-gtk') { + try { + var IBus = imports.gi.IBus + if (!('new_async' in IBus.Bus)) { + // This IBus does not support GNOME 3.6 + icon.height = imports.ui.panel.PANEL_ICON_SIZE + this.ibusIndicator = new imports.ui.panelMenu.Button() + this.ibusIndicator.actor.add_actor(icon) + Main.panel.addToStatusArea('ibus-gtk-ui', this.ibusIndicator, 0, 'right') + } + } catch(e) { + IBus = null + } + } + if (STANDARD_TRAY_ICON_IMPLEMENTATIONS[wmClass] !== undefined) return;

@@ -510,6 +525,8 @@ let source = this._lookupSource(null, icon.pid, true); if (source) source.destroy(); + if (icon.wm_class == 'ibus-ui-gtk' && this.ibusIndicator) + this.ibusIndicator.destroy(); } }); -- To unsubscribe, e-mail: opensuse-m17n+unsubscribe@opensuse.org To contact the owner, e-mail: opensuse-m17n+owner@opensuse.org

Hi Hillwood, I also think Topicons is a great shell extension but it has problems that it put all of icons on the top, which breaks design principle of GNOME. # although the current notification daemon is not good enough. So I want to make another option to enable only IBus icon. IBus should be usable just after installation as ever, but it may be time to give up it. Some users will feel openSUSE (or IBus) is broken. Fuminobu TAKEYAMA (2013/02/16 14:08), Hillwood Yang wrote:

Hi ftake,

Actually, there is a extension named topicons also can do this work (https://extensions.gnome.org/extension/495/topicons/). It can work in Gnome 3.6 and move all tray icons to top, include fcitx ,scim ,gcin, banshee and others. And all features of openSUSE 12.3 have been frozen, I don't think your extension would be auto enable. of course, you can try. Maybe we should ask users to install and enable topicons or your extension themselves.

PS: Topicons works fine with my patch.

https://fbcdn-sphotos-h-a.akamaihd.net/hphotos-ak-frc1/858577_10200777261453...

Thanks!

Hillwood Yang

在 2013-02-16六的 01:59 +0900，Fuminobu TAKEYAMA写道：

...

Hi all,

I packaged the shell extension with ibus for the approach 4. The package is available from obs://home:ftake:branches:M17N/openSUSE_12.3

Could you test it?

After installing it, please enable the shell extension by your self and log out.

I will submit it to M17N and forward to Factory if it is OK.

Is there any possibility to add "enabled-extension"? If we do, users cannot disable the shell extension. So we need to consider carefully.

# We can put other selected icons including fcitx on the top panel in the same way.

Fuminobu TAKEYAMA

(2013/02/13 4:32), Fuminobu TAKEYAMA wrote:

...

Hello, it's 4 a.m. in Japan,

I updated my shell extension. It should work fine with ibus on *RC1 DVD*. On 12.3 repo, there is ibus with hillwood's patch, which does not work with my extension because the patch removes code for the indicator. https://github.com/ftake/ibus_indicator-14

To test this, please run $ cd ~/.local/share/gnome-shell/extensions/ $ git clone git://github.com/ftake/ibus_indicator-14.git

and enable it by gnome-tweak-tool

...

Now, you need to change the default settings for gnome-shell.

See enabled-extensions key in https://build.opensuse.org/package/view_file?expand=1&file=glib2-branding.gschema.override.in&package=glib2-branding-openSUSE&project=GNOME%3AFactory

Thanks!

Now, I have five ideas: 1. Go with the current (hillwood's patched) ibus - Its icon is managed notification daemon (on the bottom of *Activity*) - A bit difficult to use (too small menu, not shown always) 2. Enable the extension by modifying glib2-branding-openSUSE 3. Put a part of the extension (only 17 lines of code) into gnome-shell's script by a patch (see below) - This *was* the best solution if we could notice this problem earlier beta 4. Users enable the extension manually by gnome-tweak-tool, and we update hillwood's patch to work with the extension

5. Call "gnome-shell-extension-tools -e ibus_indicator-14" from the ibus launch script to enable the extension

Fuminobu TAKEYAMA

geeko@linux-jpk4:~> diff -u /usr/share/gnome-shell/js/ui/notificationDaemon.js.back /usr/share/gnome-shell/js/ui/notificationDaemon.js --- /usr/share/gnome-shell/js/ui/notificationDaemon.js.back 2013-02-13 03:55:29.996264488 +0900 +++ /usr/share/gnome-shell/js/ui/notificationDaemon.js 2013-02-13 04:19:39.961930850 +0900 @@ -500,6 +500,21 @@

_onTrayIconAdded: function(o, icon) { let wmClass = icon.wm_class ? icon.wm_class.toLowerCase() : ''; + if (wmClass == 'ibus-ui-gtk') { + try { + var IBus = imports.gi.IBus + if (!('new_async' in IBus.Bus)) { + // This IBus does not support GNOME 3.6 + icon.height = imports.ui.panel.PANEL_ICON_SIZE + this.ibusIndicator = new imports.ui.panelMenu.Button() + this.ibusIndicator.actor.add_actor(icon) + Main.panel.addToStatusArea('ibus-gtk-ui', this.ibusIndicator, 0, 'right') + } + } catch(e) { + IBus = null + } + } + if (STANDARD_TRAY_ICON_IMPLEMENTATIONS[wmClass] !== undefined) return;

@@ -510,6 +525,8 @@ let source = this._lookupSource(null, icon.pid, true); if (source) source.destroy(); + if (icon.wm_class == 'ibus-ui-gtk' && this.ibusIndicator) + this.ibusIndicator.destroy(); } });

-- To unsubscribe, e-mail: opensuse-m17n+unsubscribe@opensuse.org To contact the owner, e-mail: opensuse-m17n+owner@opensuse.org