1
0
mirror of https://github.com/twitter/twemoji.git synced 2024-12-28 06:18:30 +00:00
Commit Graph

61 Commits

Author SHA1 Message Date
Andrea Giammarchi
8e58ae4745 fixed SVG smiles face alignment 2015-04-09 11:57:30 +01:00
Andrea Giammarchi
974c86d072 Merge pull request #66 from twitter/bug/consistent-face-sizes
Make size and position of face emoji more consistent
2015-04-09 11:29:43 +01:00
Joen Asmussen
898d403387 Tweak 1f630 and 1f622
Remove extra teardrop from crying face emoji. Fixes #63

Reposition sweat-drop on face with open mouth and cold sweat emoji. Fixes #61
2015-04-09 12:03:00 +02:00
Andrea Giammarchi
264989b2df removed duplicated viewBox problem with more updated Inkscape 2015-04-09 10:18:05 +01:00
Joen Asmussen
18f4beeb76 Make size and position of face emoji more consistent
This pull request starts work on #60. Should it be approved we'd want to export new PNGs and SVGs.

**Observations during work**

All twemoji are drawn on a 38x38 canvas in the .ai source files. All SVGs exported from these .ai files are scaled down to a 24x24px canvas. This explains the weirdly un-rounded size values such as 22,737px in the current SVGs.

Sizing and positioning issues are most noticeable with the "yellow faces". About half the "yellow face" emoji have a head-size of 36x36px inside the 38px canvas, and the other half had a smaller 34x34px head size.

**Specifics for the 1f60a to 1f62f range**

Some of those yellow face emoji are deliberately sized down, so as to make room for artwork. For example "Face With Stuck-Out Tongue And Winking Eye" has a 34x34px head size to allow for the tongue to stick out 2px below. Compare: https://cloudup.com/c0rSvWfAh5i. Since this is a deliberate creative choice, there's nothing to fix here, except push the emoji down 1px.

1f62c had a 34px head size. I compared the eye position with 1f62e and sized up the head of 1f62c. Compare: https://cloudup.com/cD6vRRdaJGJ

**Specifics for the 1f600 to 1f637 range**

There's a fair bit more variance in sizing and positioning of these yellow faces compared to the previous range. Head sizes range from 34px to 36px, and many are off centre. Compare: https://cloudup.com/ckZltnxmCy1

A lot of the smileys in this range almost hug the top of the canvas. I'm tempted to think this may have been in an effort to better align them with text, whose glyphs are traditionally shifted upwards. However since it's not consistent, and the base head size rule is broken often to make room for other creative choices, I think properly centering is probably the more desirable outcome here.

The devil horn smileys are shifted downwards and have 34px base heads, to make room for their horns. There's not much to fix here.

Some of the shades of yellow varied ever so slightly in these emoji. But this could be a creative choice, since some of the "lighter" yellow faces were angry. Such as 1f616 and 1f612.

**Summary and fixes applied**

As a summary, the set is in pretty good shape. Among the problematic yellow-face emoji, some of the nonstandard sizes were a creative choice (devil horns), in those cases I've done nothing.

The two most standard head sizes were 34px and 36px inside the 38px canvas. Specifically in the 1f600 to 1f637 range, it became clear that the 34px head size was both the most flexible (allowed more artwork like hearts and tears to go outside of the base shape), but also the most balanced in eye-to-forehead ratio.

For that reason, I've done the following to standardize:

- In cases where this size was off (such as 36px), I've standardized on the 34px head size
- In cases where emoji were off centre for no seemingly good reason, I've centered them.
- In a very few cases I had to move an eyebrow so as to keep the balanced with the slightly smaller head size, f.x. 1f62f

Where I've performed these fixes I've taken great care in ensuring that eyes, mouth and expressions didn't suddenly fall off the pixel grid, and made sure to compare with eye-sizes of similar emoji.

Here are some before/after's: https://cloudup.com/cCyeVpudNRd
2015-04-09 10:05:44 +02:00
Andrea Giammarchi
990fa702a2 added ex-Twitter under my name 2015-04-08 15:19:38 +01:00
Andrea Giammarchi
5ea434fd0a updated minified version 2015-03-29 17:23:28 +02:00
Andrea Giammarchi
d7388f126d added SELECT to the list of ignored nodes 2015-03-13 10:26:31 +00:00
Andrea Giammarchi
4dd82d4c9e added SELECT to the list of ignored nodes 2015-03-13 10:25:20 +00:00
Andrea Giammarchi
c42a15bca1 modified reset during the loop 2015-03-12 23:30:00 +00:00
Andrea Giammarchi
fe1c85aef0 updated tags 2015-03-12 20:50:22 +00:00
Andrea Giammarchi
92d5bea1ab do not change nodes if not needed 2015-03-12 20:41:20 +00:00
Andrea Giammarchi
39a091da4b added TEXTAREA to the ignore list + fixed test for Firefox too 2015-02-16 11:33:43 +00:00
Andrea Giammarchi
1011e6b114 Merge pull request #42 from notlaforge/readme-comment-fix
Fix names of ©, ®, and ™ in code sample comment
2015-02-12 09:23:12 +01:00
Jordi Bunster
23d51907f1 Update README.md
Fix names of ©, ®, and ™ in code sample comment.
2015-02-11 16:01:20 -08:00
Andrea Giammarchi
39ebd1ca2f Merge pull request #37 from unindented/patch-1
Allow for configurable `className`
2015-01-13 22:07:50 +00:00
Daniel Perez Alvarez
e6e5610e5b Allow for configurable className 2015-01-13 16:37:56 +00:00
Andrea Giammarchi
b8a92ab762 Merge pull request #36 from JuanitoFatas/patch-1
Fix GitHub spelling. [ci skip]
2015-01-08 18:53:47 +00:00
Juanito Fatas
3e127f59fc Fix GitHub spelling. [ci skip] 2015-01-09 01:16:23 +08:00
Chris Aniszczyk
9c263d1cf9 Merge pull request #35 from msurguy/patch-1
Typo
2015-01-02 16:06:34 -06:00
Maksim Surguy
1dc4a90c91 Typo
Fixing a typo
2015-01-02 14:04:33 -08:00
Andrea Giammarchi
c23dc3f27d improved the example to exclude ®, © and ™ 2014-12-19 14:56:16 +00:00
Chris Aniszczyk
d82fa62004 Add Tom Wuttke as a maintainer 2014-12-17 09:14:09 -06:00
Andrea Giammarchi
3123cfe176 added extra invariants and invalid chars tests 2014-12-02 17:09:36 +00:00
Andrea Giammarchi
82d698e4ec added more tests for weird graphic cases 2014-12-02 16:52:19 +00:00
Andrea Giammarchi
a467816397 fixed other chars too 2014-12-02 08:18:22 +00:00
Andrea Giammarchi
0a3a60d138 fixed non standard OSX behavior with variants 2014-12-01 23:28:19 +00:00
Andrea Giammarchi
2b9415791c bumped version after ignoring svg.sh 2014-12-01 10:36:21 +00:00
debian
c9094ba05d updated svg flags too 2014-11-21 18:41:12 -05:00
Andrea Giammarchi
9a2200f93f updated few .ai flags removing the red border 2014-11-21 23:31:02 +00:00
Andrea Giammarchi
edf294cd61 updated all wrong abs.twimg references in the documentation 2014-11-19 12:58:46 +00:00
debian
989b6e46b4 exported the right 1f567.svg 2014-11-14 20:00:50 -05:00
Andrea Giammarchi
c91d390318 fixed wrong 1f567 2014-11-15 00:42:35 +00:00
Andrea Giammarchi
2501eaa1ac optimized SVGs size + added responsive viewport 2014-11-13 10:48:03 +00:00
Andrea Giammarchi
6d82d6a6bf fixed duplicated chinese flag 2014-11-11 20:07:52 +00:00
Andrea Giammarchi
f7d309cda9 Merge pull request #24 from andypiper/patch-1
add a verb to "Attribution Requirements"
2014-11-11 18:16:31 +00:00
Andy Piper
a5b334b2b7 add a verb to "Attribution Requirements"
Closes #23
2014-11-11 17:38:14 +00:00
Andrea Giammarchi
ad86b740c4 Merge pull request #22 from tomByrer/patch-1
del  -o-user-select: none;
2014-11-11 16:20:47 +00:00
Tom Byrer
29e62e425e del -o-user-select: none;
Opera's `user-select` came with Blink: http://caniuse.com/#feat=user-select-none
2014-11-11 09:15:42 -07:00
Andrea Giammarchi
92b06540c0 Merge pull request #21 from pmk25/patch-1
Update README.md
2014-11-11 16:12:08 +00:00
pmk25
fdc6f224fc Update README.md
Minor grammar/formatting tweaks.
2014-11-11 11:07:21 -05:00
Chris Aniszczyk
91efae9f3a Add "Community Projects" section 2014-11-11 12:50:52 +09:00
Chris Aniszczyk
416e7fafde Add a section on Attribution
Closes #18
2014-11-11 11:38:57 +09:00
Andrea Giammarchi
d5f9775816 removed duplicated entries in non standard related regexp 2014-11-09 20:25:25 +00:00
Andrea Giammarchi
f5d1318b97 Merge pull request #6 from thii/component
Add component.json
2014-11-08 00:31:11 +00:00
Andrea Giammarchi
42d27cd00a added folder support for svg case 2014-11-08 00:21:51 +00:00
Doan Truong Thi
f4c8d33349 Add component's build directory as file to ignore 2014-11-08 09:14:27 +09:00
Doan Truong Thi
193671788b Add component.json as file to ignore to .npmignore and bower.json 2014-11-08 09:13:06 +09:00
debian
5c0297d324 created svg script and folder 2014-11-07 18:59:05 -05:00
Andrea Giammarchi
0450e3e5e3 added className object property 2014-11-07 23:36:32 +00:00