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

Issue 9407043: Let users know the default hotkey is Super+space with libnotify. (Closed)

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

Description

Let users know the default hotkey is Super+space with libnotify.

Patch Set 1 #

Total comments: 4

Patch Set 2 : Updated with message #2. #

Patch Set 3 : Updated copyright. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+141 lines, -6 lines) Patch
M configure.ac View 1 2 3 chunks +18 lines, -2 lines 0 comments Download
M data/ibus.schemas.in View 1 chunk +14 lines, -0 lines 0 comments Download
M ui/gtk3/Makefile.am View 1 2 2 chunks +17 lines, -2 lines 0 comments Download
M ui/gtk3/panel.vala View 1 2 chunks +92 lines, -2 lines 0 comments Download

Messages

Total messages: 5
fujiwara
11 years ago (2013-05-14 06:26:35 UTC) #1
Peng
https://codereview.appspot.com/9407043/diff/1/configure.ac File configure.ac (right): https://codereview.appspot.com/9407043/diff/1/configure.ac#newcode157 configure.ac:157: ]) Is it easy to make this feature configurable? ...
11 years ago (2013-05-14 14:51:41 UTC) #2
fujiwara
https://codereview.appspot.com/9407043/diff/1/configure.ac File configure.ac (right): https://codereview.appspot.com/9407043/diff/1/configure.ac#newcode157 configure.ac:157: ]) On 2013/05/14 14:51:41, Peng wrote: > Is it ...
11 years ago (2013-05-15 15:27:41 UTC) #3
Peng
lgtm
11 years ago (2013-05-16 04:49:16 UTC) #4
fujiwara
11 years ago (2013-05-17 03:11:33 UTC) #5
Updated copyright.
Sign in to reply to this message.

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