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

Issue 308320043: Fix warning messages during panel resizes (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
7 years, 7 months ago by fujiwara
Modified:
7 years, 7 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

Fix warning messages during panel resizes Move gtk_window_resize(). Seems calling gtk_window_resize() is not good during gtk_widget_get_preferred_width/height() is called. Committed: af66ace771fc16a23e7a5841d2f0bd79789e009d

Patch Set 1 #

Total comments: 4

Patch Set 2 : Updated with message #3. #

Total comments: 4

Patch Set 3 : Updated with message #6 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+19 lines, -23 lines) Patch
M ui/gtk3/candidatepanel.vala View 1 2 3 chunks +10 lines, -11 lines 0 comments Download
M ui/gtk3/propertypanel.vala View 1 2 2 chunks +9 lines, -12 lines 0 comments Download

Messages

Total messages: 8
fujiwara
7 years, 7 months ago (2016-09-06 11:17:45 UTC) #1
Peng
https://codereview.appspot.com/308320043/diff/1/ui/gtk3/candidatepanel.vala File ui/gtk3/candidatepanel.vala (right): https://codereview.appspot.com/308320043/diff/1/ui/gtk3/candidatepanel.vala#newcode243 ui/gtk3/candidatepanel.vala:243: * in gtk_widget_size_allocate_with_baseline() */ Could you give me a ...
7 years, 7 months ago (2016-09-09 10:54:42 UTC) #2
Peng
https://codereview.appspot.com/308320043/diff/1/ui/gtk3/candidatepanel.vala File ui/gtk3/candidatepanel.vala (right): https://codereview.appspot.com/308320043/diff/1/ui/gtk3/candidatepanel.vala#newcode135 ui/gtk3/candidatepanel.vala:135: m_toplevel.resize(1, 1); Can we call this in update()?
7 years, 7 months ago (2016-09-09 11:07:16 UTC) #3
fujiwara
https://codereview.appspot.com/308320043/diff/1/ui/gtk3/candidatepanel.vala File ui/gtk3/candidatepanel.vala (right): https://codereview.appspot.com/308320043/diff/1/ui/gtk3/candidatepanel.vala#newcode243 ui/gtk3/candidatepanel.vala:243: * in gtk_widget_size_allocate_with_baseline() */ On 2016/09/09 10:54:41, Peng wrote: ...
7 years, 7 months ago (2016-09-12 10:28:24 UTC) #4
fujiwara
https://codereview.appspot.com/308320043/diff/1/ui/gtk3/candidatepanel.vala File ui/gtk3/candidatepanel.vala (right): https://codereview.appspot.com/308320043/diff/1/ui/gtk3/candidatepanel.vala#newcode135 ui/gtk3/candidatepanel.vala:135: m_toplevel.resize(1, 1); On 2016/09/09 11:07:16, Peng wrote: > Can ...
7 years, 7 months ago (2016-09-13 10:27:34 UTC) #5
Peng
lgtm https://codereview.appspot.com/308320043/diff/20001/ui/gtk3/candidatepanel.vala File ui/gtk3/candidatepanel.vala (right): https://codereview.appspot.com/308320043/diff/20001/ui/gtk3/candidatepanel.vala#newcode231 ui/gtk3/candidatepanel.vala:231: public override void get_preferred_width(out int minimum_width, out int ...
7 years, 7 months ago (2016-09-13 13:16:45 UTC) #6
fujiwara
https://codereview.appspot.com/308320043/diff/20001/ui/gtk3/candidatepanel.vala File ui/gtk3/candidatepanel.vala (right): https://codereview.appspot.com/308320043/diff/20001/ui/gtk3/candidatepanel.vala#newcode231 ui/gtk3/candidatepanel.vala:231: public override void get_preferred_width(out int minimum_width, out int natural_width) ...
7 years, 7 months ago (2016-09-14 04:34:30 UTC) #7
fujiwara
7 years, 7 months ago (2016-09-14 04:34:51 UTC) #8
Message was sent while issue was closed.
Committed patchset #3 (id:40001) manually as
af66ace771fc16a23e7a5841d2f0bd79789e009d (presubmit successful).
Sign in to reply to this message.

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