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

Issue 320700043: ui/gtk3: Integrate custom rendering to use HarfBuzz glyph info

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

Description

ui/gtk3: Integrate custom rendering to use HarfBuzz glyph info IBusFontSet offers FcFontSet, glyph info with HarfBuzz and rendering on Cairo context. Current Pango changes fonts by emoji variants and draws the separated glyphs [1] but actually the emoji characters with variants can be drawn as one glyph so this class manages Fontconfig fontsets to select a font, HarfBuzz to get glyphs for emoji variants, Cairo to draw glyphs. Need configure --enable-harfbuzz-for-emoji option to enable this feature. [1]: https://bugzilla.gnome.org/show_bug.cgi?id=780669 https://bugzilla.gnome.org/show_bug.cgi?id=781123 BUG=https://bugzilla.gnome.org/show_bug.cgi?id=780669

Patch Set 1 #

Patch Set 2 : Fix a typo #

Patch Set 3 : Show code points for ASCII control chars. #

Patch Set 4 : Fix ln failure in parallel make #

Patch Set 5 : Fixed some memory leaks #

Patch Set 6 : Fix a build failure in ibusfontset.h dependency #

Patch Set 7 : Updated with rhbz#1490733 #

Patch Set 8 : Fix rhbz#1490733 #

Patch Set 9 : Updated for rhel7 #

Patch Set 10 : Fix a performance issue if emoji fonts are not installed #

Patch Set 11 : Fix a typo #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1576 lines, -2 lines) Patch
A bindings/vala/IBusFontSet-1.0.metadata View 1 chunk +1 line, -0 lines 0 comments Download
M bindings/vala/Makefile.am View 1 2 3 4 5 6 7 3 chunks +83 lines, -0 lines 0 comments Download
A bindings/vala/ibus-fontset-1.0.deps View 1 chunk +1 line, -0 lines 0 comments Download
M configure.ac View 1 2 3 4 5 2 chunks +29 lines, -0 lines 0 comments Download
M ui/gtk3/Makefile.am View 1 2 3 4 5 6 7 2 chunks +32 lines, -0 lines 0 comments Download
M ui/gtk3/emojier.vala View 1 2 3 4 5 6 7 8 6 chunks +98 lines, -2 lines 0 comments Download
A ui/gtk3/ibusfontset.h View 1 chunk +302 lines, -0 lines 0 comments Download
A ui/gtk3/ibusfontset.c View 1 2 3 4 5 6 7 8 9 10 1 chunk +1030 lines, -0 lines 0 comments Download

Messages

Total messages: 2
fujiwara
6 years, 10 months ago (2017-06-24 07:53:48 UTC) #1
Peng
6 years, 9 months ago (2017-06-28 02:55:12 UTC) #2
lgtm
Sign in to reply to this message.

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