Port the keyboard indicator in gsd to use the app. indicator protocol

[David Barth]

Work item to track the port of gnome-settings-daemon keyboard status icon to use the new app. indicator API.

[David Barth (dbarth)]

Putting that on the a3 radar

[David Barth (dbarth)]

Passing on to jcastro for when the app. indicator icon to code is defined. Marking incomplete as such
mpt: what do you advise we put on the indicator?

[Jorge Castro (jorge)]

Bug #564034 has information on how we fix this in ibus that might be relevant.

[Jorge Castro (jorge)]

David, I don't have permission to assign this bug, can you please assign it to Karl Lattimer, launchpad id is: karl-qdh

[Sebastien Bacher (seb128)]

there has been work done on that previous cycle, see bug #497875, the patch was working but got blocked due to the lack of label support in appindicator and the fact that users want the layout to be indicated on screen

[Sebastien Bacher (seb128)]

see bug #531173 about the issue

[Karl Lattimer (karl-qdh)]

This is an updated version of Jan Arne Petersen's patch in https://bugs.edge.launchpad.net/ubuntu/+source/gnome-settings-daemon/+bug/497875

This is just a straight update so it applies to the 2.30.2 tarball directly, I still need to merge the patch with the other indicator patch and make it use a label for the indicator.

[Karl Lattimer (karl-qdh)]

Updated the patch of gnome-settings-daemon to include both the existing xrandr indicator and the patch supplied with https://bugs.edge.launchpad.net/ubuntu/+source/gnome-settings-daemon/+bug/497875

All that needs to be done now is updating it to use a label rather than a non-descriptive icon

[Karl Lattimer (karl-qdh)]

I have a partially working patch now.

The correct labels are set however they are not correctly set on startup, so I will endeavour to fix that issue and then upload the updated patch.

[Karl Lattimer (karl-qdh)]

This is an incomplete but working patch, currently likely to leak memory.

The deallocation causes a segfault, will discuss this problem with tedg later today.

[Karl Lattimer (karl-qdh)]

Allowing any of the free's or g_object_unrefs will cause a crash.. which is pretty weird

Desperately need to figure out a way to debug this, get some text to print into a terminal and run a valgrind many difficulties related to debugging gnome-settings-daemon.

[Karl Lattimer (karl-qdh)]

As seb128/davidbarth agreed to release this patch in a leaking form a follow up bug has been created for the probable leaking;

https://bugs.launchpad.net/ayatana-ubuntu/+bug/616765

[David Barth (dbarth)]

re-assigning to seb128 so that this can go in for today's upload window

[Launchpad Janitor (janitor)]

This bug was fixed in the package gnome-settings-daemon - 2.30.2-0ubuntu3

---------------
gnome-settings-daemon (2.30.2-0ubuntu3) maverick; urgency=low

  * debian/control.in:
    - requires libindicator-application 0.2.4 for label support
  * debian/patches/06_use_application_indicator.patch:
    - updated version by Karl Lattimer porting the keyboard layout to indicators
      now that indicator-application supports labels (lp: #599844)
  * debian/control.in, debian/rules:
    - use dh-autoreconf
 -- Sebastien Bacher <email address hidden> Tue, 17 Aug 2010 19:21:57 +0200

[Karl Lattimer (karl-qdh)]

Related bugs which have appeared after the fix

https://bugs.launchpad.net/ubuntu/maverick/+source/gnome-settings-daemon/+bug/531290 - FIXED
https://bugs.launchpad.net/ubuntu/+source/gnome-settings-daemon/+bug/620334 - FIXED
https://bugs.launchpad.net/ubuntu/+source/gnome-settings-daemon/+bug/620571 - FIXED
https://bugs.launchpad.net/ubuntu/+source/gnome-settings-daemon/+bug/620334 - FIXED

New patch on the way

[Karl Lattimer (karl-qdh)]

The package will be available shortly from the ppa: https://launchpad.net/~karl-qdh/+archive/ppa

[Sebastien Bacher (seb128)]

gnome-settings-daemon (2.30.2-0ubuntu4) maverick; urgency=low

  * 06_use_application_indicator.patch: update by Karl Lattimer to fix those
    - keyboard indicator does not react to layout changes (lp: #531290)
    - indicator variants in a same layout displayed the same way (lp: #620334)
    - move keyboard layouts out of the submenu (lp: #620571)

[Karl Lattimer (karl-qdh)]

Update to the patch to use the smallest possible indicator size XXX when only 3 letter labels are used.

[Mark Shuttleworth (sabdfl)]

Is anyone working on an Ubuntu Mono style icon for the keyboard indicator?

 affects: ayatana-design

Mark

[Karl Lattimer (karl-qdh)]

Vish has already made one that's currently used by ibus as the default icon, having two identical icons may end up being confusing if someone uses both. We've got a further discussion about the icon here;

https://bugs.launchpad.net/ubuntu/+source/gnome-settings-daemon/+bug/620331

[Sebastien Bacher (seb128)]

Thanks Karl, I will probably wait next week to batch that update with some of the others fixes that might come there

[Karl Lattimer (karl-qdh)]

Currently depends on this bug; https://bugs.launchpad.net/ubuntu/+source/gnome-settings-daemon/+bug/621633

[Karl Lattimer (karl-qdh)]

The last comment refers to a bug which isn't in gsd, apparently according to tedg a release for a fix is due.

[Karl Lattimer (karl-qdh)]

This minor update means that if show_flags is enabled the indicator will only show an icon. This emulates exactly the behavior of the notification-area icon as it currently exists upstream.

The reason for this is that the complaint of too much space being used is perfectly valid, and this makes it much easier for us to drop in a pack of icons, set the gconf key and have exactly what MPT wants as per the screenshot here; http://img243.imageshack.us/img243/7358/screenshotnw.png

which was attached to bug; https://bugs.launchpad.net/ubuntu/+source/gnome-settings-daemon/+bug/623435

[Karl Lattimer (karl-qdh)]

Completing the indicator as specified will only require implementing symbolic svg icons for the "flags" which as previously discussed will not actually be flags, but instead two letters inside of a box.

See the bug regarding this here; https://bugs.launchpad.net/ayatana-ubuntu/+bug/628015