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

Issue 219520043: ibus-ui-gtk3: Draw gray color on PropertyPanel handle (Closed)

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

Description

ibus-ui-gtk3: Draw gray color on PropertyPanel handle Users can move the position of IBus PropertyPanel with the mouse but currently it is too hard to find the handle on the panel. Now the handle is drawn by the gray color for the visibility. TEST=ui/gtk3/ibus-ui-gtk3

Patch Set 1 #

Total comments: 2

Patch Set 2 : Updated with message #3. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+14 lines, -1 line) Patch
M ui/gtk3/handle.vala View 1 2 chunks +14 lines, -1 line 0 comments Download

Messages

Total messages: 4
fujiwara
9 years, 1 month ago (2015-03-31 09:56:55 UTC) #1
Peng
On 2015/03/31 09:56:55, fujiwara wrote: lgtm
9 years, 1 month ago (2015-03-31 15:11:04 UTC) #2
Peng
https://codereview.appspot.com/219520043/diff/1/ui/gtk3/handle.vala File ui/gtk3/handle.vala (right): https://codereview.appspot.com/219520043/diff/1/ui/gtk3/handle.vala#newcode39 ui/gtk3/handle.vala:39: Please add a comment for below code.
9 years, 1 month ago (2015-03-31 15:11:10 UTC) #3
fujiwara
9 years, 1 month ago (2015-04-01 02:41:39 UTC) #4
https://codereview.appspot.com/219520043/diff/1/ui/gtk3/handle.vala
File ui/gtk3/handle.vala (right):

https://codereview.appspot.com/219520043/diff/1/ui/gtk3/handle.vala#newcode39
ui/gtk3/handle.vala:39: 
On 2015/03/31 15:11:10, Peng wrote:
> Please add a comment for below code.

Done.
Sign in to reply to this message.

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