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

Issue 314640043: ui/gtk3: Enable strcasecmp to match emoji annotation (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: Enable strcasecmp to match emoji annotation Users can type capital annotations. Also shows emoji annotations in the status bar if the typing unicode point matches a emoji character. Committed: 50e344afaffc29e626dbc27747a1aeee6cccafdf

Patch Set 1 #

Total comments: 4
Unified diffs Side-by-side diffs Delta from patch set Stats (+65 lines, -32 lines) Patch
M ui/gtk3/emojier.vala View 9 chunks +65 lines, -32 lines 4 comments Download

Messages

Total messages: 3
fujiwara
7 years, 3 months ago (2017-03-16 12:36:57 UTC) #1
Peng
lgtm https://codereview.appspot.com/314640043/diff/1/ui/gtk3/emojier.vala File ui/gtk3/emojier.vala (right): https://codereview.appspot.com/314640043/diff/1/ui/gtk3/emojier.vala#newcode435 ui/gtk3/emojier.vala:435: private string utf8_down(string str) { Add a comment ...
7 years, 3 months ago (2017-03-16 14:31:07 UTC) #2
fujiwara
7 years, 3 months ago (2017-03-17 03:09:00 UTC) #3
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
50e344afaffc29e626dbc27747a1aeee6cccafdf (presubmit successful).
Sign in to reply to this message.

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