Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(3184)

Issue 3807047: Support enable/disable hotkeys (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
13 years, 6 months ago by Yusuke Sato
Modified:
13 years, 4 months ago
Reviewers:
shawn.p.huang, Peng
Base URL:
git@github.com:ibus/ibus.git@master
Visibility:
Public.

Description

Support enable/disable hotkeys that enable or disable ibus unconditionally (i.e. not toggle.) I'll update ibus/po/*po files if the change looks good to you. BUG=http://code.google.com/p/ibus/issues/detail?id=1173 TEST=manually

Patch Set 1 : rebase #

Unified diffs Side-by-side diffs Delta from patch set Stats (+195 lines, -33 lines) Patch
M bus/ibusimpl.c View 5 chunks +54 lines, -8 lines 0 comments Download
M ibus/common.py View 2 chunks +4 lines, -6 lines 0 comments Download
M setup/main.py View 1 chunk +22 lines, -0 lines 0 comments Download
M setup/setup.ui View 5 chunks +115 lines, -19 lines 0 comments Download

Messages

Total messages: 2
Yusuke Sato
Hi, Would it be possible to review this change? The change is not related to ...
13 years, 5 months ago (2011-01-30 13:44:13 UTC) #1
Peng
13 years, 4 months ago (2011-02-10 20:42:00 UTC) #2
On 2011/01/30 13:44:13, Yusuke Sato wrote:
> Hi,
> Would it be possible to review this change?
> 
> The change is not related to Chromium OS at all :), but very important for
> Japanese keyboard users.

LGTM. Sorry for reply late.
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b