Emoji metrics incorrect on Mac Sierra |
||||||||
Issue descriptionChrome Version : Version 55.0.2883.95 (64-bit) URLs (if applicable) : http://unicode.org/emoji/charts/emoji-list.html#1f473_1f3ff_200d_2642_fe0f Other browsers tested: Add OK or FAIL, along with the version, after other browsers where you have tested this issue: Safari: OK Firefox: IE: What steps will reproduce the problem? (1) Open URL (2) Scan down the "Browser" column to see what Chrome is displaying (other column is images) (3) Metrics are clearly wrong. Sometimes that results in offset glyphs, other times in bad selection boxes. See images. In Chrome the selection box extends too far to the right. What is the expected result? What happens instead? Please provide any additional information below. Attach a screenshot if possible.
,
Dec 18 2016
,
Dec 19 2016
,
Dec 22 2016
Tested this issue in Mac 10.12.2 with chrome version #55.0.2883.95 On navigating to "http://unicode.org/emoji/charts/full-emoji-list.html#1f468_1f3fb_200d_1f393" observed that some emojis are aligned towards left side. Is this is the issue you are experiencing??? Attaching screen-cast for your reference, please look into it and let us know u r observstions.
,
Dec 22 2016
That is part of it; the metrics for other emoji sequences also look wrong, even if they show up centered. Looking at the screen shots I submitted, the bounding box for other sequences extends too far to the right.
,
Dec 29 2016
Thank you for providing more feedback. Adding requester "kkaluri@chromium.org" for another review and adding "Needs-Review" label for tracking. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Jan 2 2017
Looks like an issue with the modifiers having an advance, over to drott for further triage.
,
Jan 3 2017
Emoji data was updated to 4.0 in M56. Mark & kkaluri@ could you try with the M56 beta, if it's not too much trouble. I looked at the beta results and there is no off-center placement, however, we missed fixes for judge and pilot, since our word cache segmentation did split those sequences. Also, the emoji full data page now has VS16 on the emoji keycap sequence base characters 0-9,#,* and our Emoji segmentation step did not take that into account. I uploaded a fix for those last two issues: https://codereview.chromium.org/2603343002 After this fix, the bounding box issue is fixed as well, compare: http://codepen.io/anon/pen/KaKNYr On my fixed version, this outputs identical widths for man and pilot, but it does show a too wide bounding box before the fix. (Making this issue public. Filed by Googler, but does not contain any internal only information.)
,
Jan 3 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/f176e05cf23ff77ceffda2e22caa0a71814bb403 commit f176e05cf23ff77ceffda2e22caa0a71814bb403 Author: drott <drott@chromium.org> Date: Tue Jan 03 15:04:41 2017 Emoji fixes: Two professions and digits with VS16 Fixes for judge and pilot profession sequences as they were split by CachingWordShapeIterator. Also accept VS16 on keycap base characters 0-9,*,# as used on http://unicode.org/emoji/charts/full-emoji-list.html#0023_fe0f_20e3 BUG=628333, 675329 Review-Url: https://codereview.chromium.org/2603343002 Cr-Commit-Position: refs/heads/master@{#441123} [modify] https://crrev.com/f176e05cf23ff77ceffda2e22caa0a71814bb403/third_party/WebKit/Source/platform/fonts/SymbolsIterator.cpp [modify] https://crrev.com/f176e05cf23ff77ceffda2e22caa0a71814bb403/third_party/WebKit/Source/platform/fonts/SymbolsIteratorTest.cpp [modify] https://crrev.com/f176e05cf23ff77ceffda2e22caa0a71814bb403/third_party/WebKit/Source/platform/fonts/shaping/CachingWordShaperTest.cpp [modify] https://crrev.com/f176e05cf23ff77ceffda2e22caa0a71814bb403/third_party/WebKit/Source/platform/text/CharacterPropertyDataGenerator.h
,
Jan 3 2017
,
Jan 3 2017
Mark, if you have time, you may want to have a look at a Canary build of Chrome tomorrow or the day after and see if it's fixed. |
||||||||
►
Sign in to add a comment |
||||||||
Comment 1 by markda...@google.com
, Dec 17 2016