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

Issue 4235064: Use XkbKeysymToModifiers to check AltGr modifier. (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
13 years, 2 months ago by Daiki Ueno
Modified:
13 years, 1 month ago
Reviewers:
shawn.p.huang
Base URL:
git@github.com:ibus/ibus-m17n.git@master
Visibility:
Public.

Description

Use XkbKeysymToModifiers to check AltGr modifier. Currently, ibus-m17n assumes that AltGr is mapped to MOD5. However, it may be mapped to different non-virtual modifiers. This patch uses XkbKeysymToModifiers to check which modifier AltGr is mapped to. BUG=none TEST=manual

Patch Set 1 #

Patch Set 2 : simplified #

Unified diffs Side-by-side diffs Delta from patch set Stats (+46 lines, -22 lines) Patch
configure.ac View 1 chunk +17 lines, -0 lines 0 comments Download
M src/Makefile.am View 1 chunk +5 lines, -0 lines 0 comments Download
M src/engine.c View 1 5 chunks +24 lines, -22 lines 0 comments Download

Messages

Total messages: 3
Daiki Ueno
13 years, 2 months ago (2011-03-09 08:09:51 UTC) #1
Peng
On 2011/03/09 08:09:51, Daiki Ueno wrote: Is it necessary? It is not good to let ...
13 years, 2 months ago (2011-03-09 14:05:58 UTC) #2
Daiki Ueno
13 years, 1 month ago (2011-03-18 07:14:52 UTC) #3
On 2011/03/09 14:05:58, Peng wrote:
> On 2011/03/09 08:09:51, Daiki Ueno wrote:
> Is it necessary? It is not good to let engine depend on X, and it also make
> increase the start up time.

Sorry for late response.  After discussion with the people really wanting AltGr,
it turned out that more comprehensive support of AltGr in ibus side seems
necessary.  I'll close this and file a bug report with the full story later. 
Anyway thanks for taking a look of this.
Sign in to reply to this message.

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