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

Issue 319960043: ui/gtk3: Fix panel CSS format for GTK 3.22 (Closed)

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

ui/gtk3: Fix panel CSS format for GTK 3.22 GtkCssProvider no longer works with the Pango format in GTK 3.22 and need to use the CSS format. BUG=https://github.com/ibus/ibus/issues/1879 Committed: aacf5adbba9b9b34a1ec07005e670d209513fb5b

Patch Set 1 #

Total comments: 4

Patch Set 2 : Updated with message #2 #

Patch Set 3 : Updated with the latest master #

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

Messages

Total messages: 4
fujiwara
7 years, 3 months ago (2017-01-10 08:08:59 UTC) #1
Peng
lgtm with a suggestion. https://codereview.appspot.com/319960043/diff/1/ui/gtk3/panel.vala File ui/gtk3/panel.vala (right): https://codereview.appspot.com/319960043/diff/1/ui/gtk3/panel.vala#newcode554 ui/gtk3/panel.vala:554: string data = data_format.printf(font_family, font_size); ...
7 years, 3 months ago (2017-01-10 14:16:24 UTC) #2
fujiwara
https://codereview.appspot.com/319960043/diff/1/ui/gtk3/panel.vala File ui/gtk3/panel.vala (right): https://codereview.appspot.com/319960043/diff/1/ui/gtk3/panel.vala#newcode554 ui/gtk3/panel.vala:554: string data = data_format.printf(font_family, font_size); On 2017/01/10 14:16:24, Peng ...
7 years, 3 months ago (2017-01-11 03:07:57 UTC) #3
fujiwara
7 years, 3 months ago (2017-01-11 03:17:10 UTC) #4
Message was sent while issue was closed.
Committed patchset #3 (id:40001) manually as
aacf5adbba9b9b34a1ec07005e670d209513fb5b (presubmit successful).
Sign in to reply to this message.

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