Ubuntu
ibus package

Support Application Indicators

Bug #497878 reported by Jorge Castro on 2009-12-17

This bug affects 1 person

Affects		Status	Importance	Assigned to	Milestone
	ibus	Unknown	Unknown	ibus #780
	ibus (Ubuntu)	Fix Released	Wishlist	Jan Arne Petersen

Bug Description

This application should be investigated to be ported to use Application
Indicators for Lucid - https://edge.launchpad.net/indicator-application

affects ubuntu/ibus

assignee canonical-dx-team
tag indicator-application

Tags:

Related branches

lp:ubuntu/lucid/ibus

Jan Arne Petersen (jpetersen) on 2010-02-11

Changed in ibus (Ubuntu):
status:	New → In Progress
assignee:	Canonical Desktop Experience Team (canonical-dx-team) → Jan Arne Petersen (jpetersen)

Revision history for this message

Jan Arne Petersen (jpetersen) wrote on 2010-02-11:

Basic support for app indicator. Edit (5.7 KiB, text/x-diff)

There is still a problem updating the menu when an input field get focus out by opening the menu.

Revision history for this message

Jan Arne Petersen (jpetersen) wrote on 2010-02-12:

Updated patch, which make it work as good as possible without focus-set=false on the menu Edit (7.1 KiB, text/plain)

tags:

added: patch

Revision history for this message

Jan Arne Petersen (jpetersen) wrote on 2010-02-12:

In addition we would need support for icon set by filename.

Jan Arne Petersen (jpetersen) on 2010-02-12

Changed in ibus (Ubuntu):
assignee:	Jan Arne Petersen (jpetersen) → Ken VanDine (ken-vandine)
status:	In Progress → Fix Committed

Jorge Castro (jorge) on 2010-02-12

Changed in ibus (Ubuntu):
assignee:	Ken VanDine (ken-vandine) → Jorge O. Castro (jorge)
assignee:	Jorge O. Castro (jorge) → Canonical Desktop Team (canonical-desktop-team)

Sense Egbert Hofstede (sense) on 2010-02-13

Changed in ibus (Ubuntu):
importance:	Undecided → Wishlist

Revision history for this message

Martin Pitt (pitti) wrote on 2010-02-15:

Taking for sponsoring.

Changed in ibus (Ubuntu):
assignee:	Canonical Desktop Team (canonical-desktop-team) → Martin Pitt (pitti)

Revision history for this message

Martin Pitt (pitti) wrote on 2010-02-15:

$ patch -p1 < debian/patches/05_appindicator.dpatch
patching file ui/gtk/panel.py
Hunk #7 FAILED at 366.
Hunk #8 succeeded at 417 (offset -16 lines).
Hunk #9 succeeded at 439 (offset -16 lines).
Hunk #10 succeeded at 458 (offset -16 lines).
1 out of 10 hunks FAILED -- saving rejects to file ui/gtk/panel.py.rej

Jan, can you please update it to the current lucid package?

Changed in ibus (Ubuntu):
assignee:	Martin Pitt (pitti) → Jan Arne Petersen (jpetersen)
status:	Fix Committed → Incomplete

Revision history for this message

Martin Pitt (pitti) wrote on 2010-02-15:

debdiff (with broken patch) Edit (9.1 KiB, text/plain)

For the record, this is a complete debdiff which adds the patch to the package and updates the package accordingly.

Revision history for this message

Jan Arne Petersen (jpetersen) wrote on 2010-02-15:

Updated patch (debdiff) Edit (8.7 KiB, text/plain)

Revision history for this message

Launchpad Janitor (janitor) wrote on 2010-02-15:

This bug was fixed in the package ibus - 1.2.0.20091215-1ubuntu1

---------------
ibus (1.2.0.20091215-1ubuntu1) lucid; urgency=low

  * Add 05_appindicator.dpatch: Add support for Application indicators. Patch
    by Jan Arne Petersen, thanks! (LP: #497878)
  * debian/control: Add python-appindicator recommends.
-- Martin Pitt <email address hidden> Mon, 15 Feb 2010 12:24:28 +0100

Changed in ibus (Ubuntu):
status:	Incomplete → Fix Released

Revision history for this message

Jerry Chong (zanglang) wrote on 2010-03-15:

Seems that the new indicator icon doesn't respect a 'Show icon in system tray' preference introduced in bug #422253. I've filed this as a new bug #539057.