vue3-emoji-picker icon indicating copy to clipboard operation
vue3-emoji-picker copied to clipboard

Published 20 hours ago • verified publisher icon delowardev

Update emoji list to emoji v15.0.

Open lushangkan opened this issue 1 year ago • 8 comments

Context

Update emoji list to emoji version. Add option to customize emoji list.

Summary

Update emoji list to emoji v15.0. Change emoji grouping to official grouping. Add option to customize emoji list. Change default group icon to lucide icon. Modify Header Padding to accommodate new grouping size. Modify Header flex option to fix off-center icons. Updating npm dependencies. Fix some bugs. And more...

To-do

  • [ ] Due to the change in emoji grouping, users may need to be reminded of the change in the documentation in the future.

User-facing changes

Too much changes. Comment it if you need.

Checklist

  • [x] I have tested this code to the best of my abilities
  • [x] I have added documentation where necessary

lushangkan avatar Aug 16 '23 16:08 lushangkan

Review or Edit in CodeSandbox

Open the branch in Web Editor • VS Code • Insiders
Open Preview

codesandbox[bot] avatar Aug 16 '23 16:08 codesandbox[bot]

Hi @lushangkan Thank you for the PR; I will review the changes and merge it as soon as possible.

delowardev avatar Aug 17 '23 03:08 delowardev

Seems like I didn't handle skin tones well, I'll fix that later

lushangkan avatar Aug 17 '23 08:08 lushangkan

Fixed unsupported skin color issue, zwj_emoji not supported for now (except Other group)

lushangkan avatar Aug 19 '23 09:08 lushangkan

Hi @lushangkan, thanks again for your incredible effort on this.

PR looks good, just two minor changes needed:

  1. You may want to update the emoji cdn URL as well in constant.ts#L3, the new src should be https://cdn.jsdelivr.net/npm/[email protected]/img/apple/64/

https://github.com/delowardev/vue3-emoji-picker/blob/c430646adea6727f0b31349ec33629b72a7715f2/src/constant.ts#L3-L4

  1. In emojis.json#10461, could you please update 2642-fe0f to 2641-fe0f? I noticed the male sign emoji doesn't show with the previous code for some reason.

https://github.com/delowardev/vue3-emoji-picker/blob/c430646adea6727f0b31349ec33629b72a7715f2/src/data/emojis.json#L10461

Thank you for checking it out! I've noticed some new issues in the meantime, I'll get back to you when I'm done fixing them. Also, emoji added a lot of zwj emoji to switch the gender of the character within the emoji, etc., do you have plans to adapt it?

lushangkan avatar Aug 20 '23 09:08 lushangkan

I think zwj emoji will be a good feature, I will need to check the possibilities.

delowardev avatar Aug 20 '23 11:08 delowardev

  1. 2641-fe0f

Is the 2641 the ♁ symbol you are referring to? The symbol is earth symbol，deprecated in Unicode 1.1, I switched 2642 to 2641 in my device (Windows 11) and it shows up as image, and I didn't find this icon in the official list of Unicode either

1 2

Hi @lushangkan, thanks again for your incredible effort on this.

PR looks good, just two minor changes needed:

  1. You may want to update the emoji cdn URL as well in constant.ts#L3, the new src should be https://cdn.jsdelivr.net/npm/[email protected]/img/apple/64/

https://github.com/delowardev/vue3-emoji-picker/blob/c430646adea6727f0b31349ec33629b72a7715f2/src/constant.ts#L3-L4

  1. In emojis.json#10461, could you please update 2642-fe0f to 2641-fe0f? I noticed the male sign emoji doesn't show with the previous code for some reason.

https://github.com/delowardev/vue3-emoji-picker/blob/c430646adea6727f0b31349ec33629b72a7715f2/src/data/emojis.json#L10461

Is the 2641 the ♁ symbol you are referring to? The symbol is earth symbol, deprecated in Unicode 1.1, I switched 2642 to 2641 in my device (Windows 11) and it shows up as image, and I didn't find this icon in the official list of Unicode either

lushangkan avatar Aug 20 '23 14:08 lushangkan

Hi @lushangkan, thanks again for your incredible effort on this.

PR looks good, just two minor changes needed:

  1. You may want to update the emoji cdn URL as well in constant.ts#L3, the new src should be https://cdn.jsdelivr.net/npm/[email protected]/img/apple/64/

https://github.com/delowardev/vue3-emoji-picker/blob/c430646adea6727f0b31349ec33629b72a7715f2/src/constant.ts#L3-L4

  1. In emojis.json#10461, could you please update 2642-fe0f to 2641-fe0f? I noticed the male sign emoji doesn't show with the previous code for some reason.

https://github.com/delowardev/vue3-emoji-picker/blob/c430646adea6727f0b31349ec33629b72a7715f2/src/data/emojis.json#L10461

I see the request https://fastly.jsdelivr.net/npm/[email protected]/img/apple/64/2642-fe0f.png failed message in the console, which could be a problem with the emoji-datasource-apple repository

lushangkan avatar Aug 20 '23 14:08 lushangkan