Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(197)

Issue 1733014: Fix the space bar to insert spaces in ibus-chewing (Closed)

Created:
10 years, 8 months ago by Zachary Kuznia
Modified:
9 years, 7 months ago
Reviewers:
Yusuke Sato
CC:
chromium-os-reviews_chromium.org
Visibility:
Public.

Description

This will cause the space bar to insert spaces if you aren't selecting a character. It is based on a patch written for ArchLinux: http://aur.archlinux.org/packages.php?ID=20567 BUG=chromium-os:1253 TEST=Set the input method to chewing, and try typing a space while not selecting a candidate. Ensure that a space is output.

Patch Set 1 #

Patch Set 2 : rebase #

Patch Set 3 : Fix preference loading #

Patch Set 4 : Remove preference patch. #

Patch Set 5 : Code Review #

Unified diffs Side-by-side diffs Delta from patch set Stats (+49 lines, -11 lines) Patch
M src/IBusChewingEngine.gob View 1 2 3 9 chunks +23 lines, -6 lines 0 comments Download
M src/IBusChewingEngine-def.c View 2 chunks +2 lines, -2 lines 0 comments Download
M src/IBusChewingEngine-keys.c View 5 chunks +24 lines, -3 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
Zachary Kuznia
10 years, 7 months ago (2010-05-25 09:05:47 UTC) #1
Yusuke Sato
10 years, 7 months ago (2010-05-25 09:16:05 UTC) #2
LGTM

Powered by Google App Engine
This is Rietveld 408576698