Unity launches selected application on commit pre-edit texts with ibus
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Nux |
Fix Released
|
Undecided
|
Koichi Akabe | ||
Unity |
Fix Released
|
Undecided
|
Unassigned | ||
nux (Ubuntu) |
Fix Released
|
Undecided
|
Unassigned | ||
unity (Ubuntu) |
Fix Released
|
Undecided
|
Unassigned |
Bug Description
Unity launches an application on commit pre-edit texts with ibus.
Step to reproduce:
1. Open Unity Dath
2. Enable iBus
3. Input texts using iBus (ibus-anthy, ibus-mozc, etc...)
4. Press Enter to commit pre-edit texts.
Actual result:
launch selected application
Expected result:
commit pre-edit texts
ProblemType: Bug
DistroRelease: Ubuntu 12.04
Package: libnux-2.0-0 2.6.0-0ubuntu1 [modified: usr/lib/
Uname: Linux 3.3.0-rc6-amd64 x86_64
ApportVersion: 1.94.1-0ubuntu2
Architecture: amd64
Date: Wed Mar 14 19:23:50 2012
DistUpgraded: Log time: 2012-02-10 23:15:24.498418
DistroCodename: precise
DistroVariant: ubuntu
DkmsStatus: vboxhost, 4.1.8, 3.2.0-18-generic, x86_64: installed
GraphicsCard:
Intel Corporation Core Processor Integrated Graphics Controller [8086:0046] (rev 12) (prog-if 00 [VGA controller])
Subsystem: Toshiba America Info Systems Device [1179:ff40]
InstallationMedia: Ubuntu 11.10 "Oneiric Ocelot" - Release amd64 (20111012)
MachineType: TOSHIBA dynabook CXE/47LE
ProcEnviron:
TERM=xterm
LANG=ja_JP.UTF-8
SHELL=/bin/bash
ProcKernelCmdLine: BOOT_IMAGE=
SourcePackage: nux
UpgradeStatus: Upgraded to precise on 2012-03-12 (1 days ago)
dmi.bios.date: 01/28/2010
dmi.bios.vendor: American Megatrends Inc.
dmi.bios.version: 1.60
dmi.board.
dmi.board.name: JPTR
dmi.board.vendor: TOSHIBA
dmi.board.version: 1.60
dmi.chassis.
dmi.chassis.type: 9
dmi.chassis.vendor: TOSHIBA
dmi.chassis.
dmi.modalias: dmi:bvnAmerican
dmi.product.name: dynabook CXE/47LE
dmi.product.
dmi.sys.vendor: TOSHIBA
version.compiz: compiz 1:0.9.7.
version.ia32-libs: ia32-libs N/A
version.libdrm2: libdrm2 2.4.30-1ubuntu1
version.
version.
version.
version.
version.
version.
version.
version.
Related branches
- Brandon Schaefer (community): Approve
-
Diff: 57 lines (+9/-7)1 file modifiedNux/TextEntry.cpp (+9/-7)
Changed in nux (Ubuntu): | |
status: | New → In Progress |
Changed in nux: | |
status: | New → In Progress |
assignee: | nobody → Koichi Akabe (vbkaisetsu) |
Changed in nux (Ubuntu): | |
assignee: | Koichi Akabe (vbkaisetsu) → nobody |
status: | In Progress → New |
Changed in nux: | |
status: | In Progress → Fix Committed |
Changed in unity: | |
status: | New → Fix Committed |
milestone: | none → 5.8.0 |
Changed in nux: | |
status: | Fix Committed → Fix Released |
Changed in unity: | |
status: | Fix Committed → Fix Released |
Changed in unity (Ubuntu): | |
status: | New → Fix Released |
I have a question about this patch, in L.2428, QueueRefresh(false, true) is called in original code.
When (ime_-> FilterKeyEvent( event) = TRUE) :
Original Code:
"(keysym == NUX_VK_ENTER || keysym == NUX_KP_ENTER) " case: activated.emit() + return
"! (keysym == NUX_VK_ENTER || keysym == NUX_KP_ENTER) " case: QueueRefresh + return
This patch:
always return. without QueueRefresh.
QueueRefresh Is really skippable?