Bug 541492 - ibus needs to support some xkb layout switching
Summary: ibus needs to support some xkb layout switching
Alias: None
Product: Fedora
Classification: Fedora
Component: ibus
Version: rawhide
Hardware: All
OS: Linux
Target Milestone: ---
Assignee: fujiwara
QA Contact: Fedora Extras Quality Assurance
Depends On:
TreeView+ depends on / blocked
Reported: 2009-11-26 03:41 UTC by Jens Petersen
Modified: 2010-10-08 12:38 UTC (History)
3 users (show)

Fixed In Version: ibus-1.3.7-11.fc14
Doc Type: Enhancement
Doc Text:
Clone Of:
Last Closed: 2010-10-08 12:38:44 UTC
Type: ---

Attachments (Terms of Use)
ibus Makefile.am, enginexkbc.py, main.py, varsxkbc.py, setup.ui (15.78 KB, patch)
2010-01-21 05:49 UTC, fujiwara
no flags Details | Diff

Description Jens Petersen 2009-11-26 03:41:25 UTC
Description of problem:
Currently we have a mixture of xkb layout config and input methods
on the desktop: even some input methods maps are just made to
avoid xkb setting/configuration issues etc.  And there is a
gnome keyboard indicator/switcher applet for changing layout.
This makes for a confusing UX and UI.  If ibus could handle
this better it might even make sense to run ibus by default
for xkb switching too at least.

Comment 1 Peng Huang 2009-12-10 04:59:00 UTC
It is scratch build for ibus. The srpm is provided by Linpus Engineer. It supports switch xkb layout in ibus systray menu. Please try it.

Comment 2 fujiwara 2009-12-11 07:30:43 UTC
OK, I'll check this asap.
However originally I thought another package ibus-xkb besides ibus.

Comment 3 fujiwara 2010-01-21 05:49:39 UTC
Created attachment 385850 [details]
ibus Makefile.am, enginexkbc.py, main.py, varsxkbc.py, setup.ui

Another idea.

Comment 4 fujiwara 2010-02-04 08:28:20 UTC
Integrating ibus-xkbc is ok however there is a design issue, which layer should be fixed. 
I'll back to GDM again to resolve the original issue.

Comment 5 Jens Petersen 2010-06-25 06:03:26 UTC
Reassigning - not sure if this still needs any more attention?

Comment 6 fujiwara 2010-07-29 06:21:40 UTC
> http://github.com/phuang/ibus/commit/02ea48ca5df2818fc467805cfe612a23632ed79f#commitcomment-119162

Yesterday I saw a commit in a branch.
I'd suggest not to integrate this patch in master. It has many critical problems.
I think the xkb settings should be in the engine only.
I applied the patch:

Also it's better to disable global input method by default.
Now I'm thinking to integrate ibus-xkb-layouts in Fedora.

Comment 7 fujiwara 2010-07-30 02:00:31 UTC
Yesterday I talked with james.su.
He thinks IMs and layouts should be consolidated and currently no plan to release ibus-xkb-layouts.
I suggested to remove his patch from the branch since I pretty sure it's a wrong patch but not sure.
His bad way is to set 'us' layout forcibly and use IBusEngineDesc.layout in case of use_system_layout.

He suggested Mac OS X has the better implementation, interested.

But currently there are no European keyboards even though IBus will can handle XKB.
I would think it's an idea to use ibus-xkb-layouts for European keyboards but he pointed my approach was also wrong. Using Ctrl + Space is wrong and an engine should have the fixed layout.
Probably I need to work on this to provide the better patch.

Comment 8 Fedora Admin XMLRPC Client 2010-08-02 06:17:38 UTC
This package has changed ownership in the Fedora Package Database.  Reassigning to the new owner of this component.

Comment 9 fujiwara 2010-08-11 01:15:48 UTC
Added libxklavier feature.

After talked with the maintainer, I'll separate the library dependencies to another binaries.

Comment 11 fujiwara 2010-08-16 01:35:03 UTC
Updated GUI to change the default xkb layout:

Comment 12 fujiwara 2010-08-17 07:47:30 UTC
Updated default loaded layouts to check ~/.config/ibus/xkb/xkblayoutconfig.xml in observed-paths.

Comment 13 fujiwara 2010-08-19 02:33:50 UTC
Created scratch build for f13:

Today's gcc is old for f14/f15:

Comment 14 fujiwara 2010-08-26 06:26:02 UTC
Rebuild the scratch build with this feature for f13:

It includes the fix of bug 627107.

Now this feature is available in f14 and f15.

Comment 18 fujiwara 2010-09-03 09:46:50 UTC
Rebased patch:

Minor chages:
 - Layout typo: s/ar/ara/
 - Moved "Cofigure Layout" button on ibus-setup to "Advanced" tab


Comment 20 Fedora Update System 2010-09-14 11:54:17 UTC
ibus-1.3.7-5.fc14 has been submitted as an update for Fedora 14.

Comment 21 fujiwara 2010-09-14 11:56:57 UTC
Revised the patch to support xkb variant and option

Comment 22 Fedora Update System 2010-09-16 03:44:48 UTC
ibus-1.3.7-5.fc14 has been pushed to the Fedora 14 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 23 fujiwara 2010-10-05 10:35:27 UTC
Revised the patch:

 - Fixed a bug when changed the system layout "jp(kana)" to "jp".
 - Redesigned xkb setting dialog
 - Applied 37ed695bfcaa764c6fd80d4f3d29f3df9032f83f
 - Removed "cn,kr,jp" from the default layouts in input methods.
 - Support group layouts in the system layouts.
 - Put "jp(kana)" into non-latin layouts. i.e. "jp(kana),us"
 - Customize the title in ibus.EngineCombobox and ibus.EngineTreeView.

Comment 24 fujiwara 2010-10-06 04:31:35 UTC
Trimmed spaces:

F13 scratch build:

I integrated the latest patch in rawhide.

Currently f14 has a build problem in gtk3:

Comment 25 Fedora Update System 2010-10-07 07:35:08 UTC
ibus-1.3.7-11.fc14 has been submitted as an update for Fedora 14.

Comment 26 Fedora Update System 2010-10-07 19:53:34 UTC
ibus-1.3.7-11.fc14 has been pushed to the Fedora 14 testing repository.  If problems still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update ibus'.  You can provide feedback for this update here: https://admin.fedoraproject.org/updates/ibus-1.3.7-11.fc14

Comment 27 Fedora Update System 2010-10-08 12:38:35 UTC
ibus-1.3.7-11.fc14 has been pushed to the Fedora 14 stable repository.  If problems still persist, please make note of it in this bug report.

Note You need to log in before you can comment on or make changes to this bug.