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

Issue 9081043: Defalut triggers key is changed to <Super>space. (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
10 years, 12 months ago by fujiwara
Modified:
10 years, 8 months ago
Reviewers:
shawn.p.huang, Peng
CC:
shawn.p.huang_gmail.com
Base URL:
git://github.com/ibus/ibus.git@master
Visibility:
Public.

Description

Defalut triggers key is changed to <Super>space. We have been used <Control>space as the default triggers key. GNOME3 and Windows7 applied <Super>space as the default trigger key and IBus follows it for the consistent key between desktops. BUG=RH#953404

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+4 lines, -4 lines) Patch
M data/ibus.schemas.in View 1 chunk +1 line, -1 line 0 comments Download
M setup/main.py View 1 chunk +1 line, -1 line 0 comments Download
M ui/gtk3/panel.vala View 2 chunks +2 lines, -2 lines 0 comments Download

Messages

Total messages: 4
fujiwara
10 years, 12 months ago (2013-05-01 14:49:33 UTC) #1
Peng
On 2013/05/01 14:49:33, fujiwara wrote: I think change it to <Supper>Space is OK to me. ...
10 years, 12 months ago (2013-05-01 15:14:40 UTC) #2
fujiwara
On 2013/05/01 15:14:40, Peng wrote: > On 2013/05/01 14:49:33, fujiwara wrote: > > I think ...
10 years, 12 months ago (2013-05-03 02:05:29 UTC) #3
Peng
10 years, 12 months ago (2013-05-03 16:28:09 UTC) #4
On 2013/05/03 02:05:29, fujiwara wrote:
> On 2013/05/01 15:14:40, Peng wrote:
> > On 2013/05/01 14:49:33, fujiwara wrote:
> > 
> > I think change it to <Supper>Space is OK to me. But I am thinking how to let
> > user know this change. Otherwise we will get a lot complain about hotkey
does
> > not work.
> 
> How about another patch adding a configure option to customize ibus.schemas.in
> later?
> When ibus 1.5.3 is released, probably we can inform of this.

LGTM. I think we should find a good way to let end-user know it.
Sign in to reply to this message.

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