Emoji Picker - [merged] #228

Closed 01oseluiz closed 10 months ago

In GitLab by @ismaelpamplona on Jan 18, 2023, 14:36

Merges feature/emoji-picker -> master

Gitlab MR Web Template

Descrição

Foi criado um componente novo chamado Emoji Picker
A função dele é mostrar ao usuário uma coleção de emojis para que ele possa selecionar um e adicionar a um input ou textarea
Foram adicionadas duas bibliotecas:
1. fuse.js: serve para fazer uma boa pesquisa com a lógica fuzzy e voltar ao usuário, neste caso, os emojis que dão mais match com a busca feita (https://fusejs.io/)
2. @emoji-mart/data: dados de emojis (https://github.com/missive/emoji-mart)

Tempo de desenvolvimento

8 horas

Tipo de mudança

[ ] Bug fix (Correção de um problema que não quebrava algo grande)
[X] New feature (Mudança que adiciona novas funcionalidades)
[ ] Breaking change (correção ou recurso que faria com que a funcionalidade existente não funcionasse como esperado)
[ ] Improvement (Alteração que melhora o desempenho de alguma função já existente)

Em quais navegadores foi testado?

[X] Google Chrome
[X] Mozilla Firefox
[ ] IE 11 (Windows 10)
[X] Edge
[ ] Opera
[ ] Opera GX
[ ] IOS Safari
[ ] IOS Chrome
[ ] Android Chrome
[ ] Android Firefox
[X] Brave

Testei as seguintes resoluções no navegador:

[X] 2520 x 1080
[X] 1920 x 1080
[X] 1400 x 790
[X] 1280 x 720
[X] 1024 x 1366 (iPad Pro vertical)
[X] 1366 x 1024 (iPad Pro horizontal)
[X] 768 x 1024 (iPad vertical)
[X] 1024 x 768 (iPad horizontal)
[X] 375 x 812 (iPhone X)
[X] 320 x 568 (iPhone 5/SE)

Checklist:

[X] Meu código segue a estilização padrão da página
[X] Eu revisei ou reli o meu próprio código
[X] Documentei onde era mais necessário e difícil de entender no meu código
[X] Segui os formatos de nomenclaturas e normas previstos pelo time
[X] Minhas mudanças não causaram mais avisos de erro
[ ] Eu conferi a checklist de atividades, e expliquei no meu merge request caso algo não tenha sido necessário ou possível
[ ] As mudanças que esta MR implementa possui testes automatizados
[ ] Executei e garanto que os testes automatizados já criados estão todos passando
[X] Executei e garanto que não há erros de lint
[X] Executei e garanto que não há erros de typescript

In GitLab by @ismaelpamplona on Jan 18, 2023, 21:04

added 1 commit

a69c0c0a - checkIfItemIsInArray functions improvement

Compare with previous version

In GitLab by @DaniloRaposo on Jan 27, 2023, 11:49

requested review from @DaniloRaposo

In GitLab by @DaniloRaposo on Jan 27, 2023, 12:39

added 1 commit

f5fe6c81 - EmojiPicker (fix): adjust import

Compare with previous version

In GitLab by @DaniloRaposo on Jan 27, 2023, 13:11

Commented on src/components/EmojiPicker/EmojiPicker.stories.mdx line 24

Para deixar de acordo com a padronização na nossa documentação, tire esses var e deixa apenas a variável css

| --szot-emojis-picker-border-radius               | --theme-small-shape        | Container border radius          |
| --szot-emojis-picker-box-shadow                  | --theme-medium-shadow      | Container box shadow             |
| --szot-emojis-picker-height                      | 315px                      | Container height                 |
| --szot-emojis-picker-width                       | 315px                      | Container width                  |
| --szot-emojis-picker-category-title-border-color | --theme-light              | Category title border color      |
| --szot-emojis-picker-category-title-font-color   | --theme-dark               | Category title font color        |
| --szot-emojis-picker-emoji-background-color      | transparent                | Emoji button background color    |
| --szot-emojis-picker-tabs-border-color           | --theme-light              | Category tab button border color |
| --szot-emojis-picker-tabs-icon-color             | --theme-dark               | Category tab icon color          |

In GitLab by @DaniloRaposo on Jan 27, 2023, 13:40

Commented on src/components/EmojiPicker/EmojiPicker.svelte line 78

Colocar um nome mais legível para a constante

In GitLab by @DaniloRaposo on Jan 27, 2023, 14:36

Commented on src/components/EmojiPicker/EmojiPicker.svelte line 262

Ficou pouco legível por causa das constantes com nome c e d

In GitLab by @DaniloRaposo on Jan 27, 2023, 14:57

Commented on src/components/EmojiPicker/EmojiPicker.svelte line 353

O input não está pegando o focus quando se escolhe a skin do emoji com skin, ao contrário do que ocorre com o emoji sem skin. Acredito que esse não seja o comportamento desejado

In GitLab by @ismaelpamplona on Jan 27, 2023, 16:13

Commented on src/components/EmojiPicker/EmojiPicker.stories.mdx line 24

Ok.

In GitLab by @ismaelpamplona on Jan 27, 2023, 16:17

Commented on src/components/EmojiPicker/EmojiPicker.svelte line 78

OK

In GitLab by @ismaelpamplona on Jan 27, 2023, 16:17

Commented on src/components/EmojiPicker/EmojiPicker.svelte line 262

Ok.

In GitLab by @ismaelpamplona on Jan 27, 2023, 16:26

resolved all threads

In GitLab by @ismaelpamplona on Jan 27, 2023, 18:17

Commented on src/components/EmojiPicker/EmojiPicker.stories.mdx line 24

changed this line in version 4 of the diff

In GitLab by @ismaelpamplona on Jan 27, 2023, 18:17

Commented on src/components/EmojiPicker/EmojiPicker.svelte line 78

changed this line in version 4 of the diff

In GitLab by @ismaelpamplona on Jan 27, 2023, 18:17

Commented on src/components/EmojiPicker/EmojiPicker.svelte line 262

changed this line in version 4 of the diff

In GitLab by @ismaelpamplona on Jan 27, 2023, 18:17

added 1 commit

de8bb816 - MR Review

Compare with previous version

In GitLab by @ismaelpamplona on Jan 27, 2023, 18:49

added 1 commit

e2c2db41 - MR Review

Compare with previous version

In GitLab by @ismaelpamplona on Jan 27, 2023, 19:30

added 1 commit

388cb1ad - MR Review

Compare with previous version

In GitLab by @DaniloRaposo on Jan 27, 2023, 19:41

approved this merge request

added 62 commits

388cb1ad...2c007b08 - 61 commits from branch master
3330faf1 - Merge remote-tracking branch 'origin/master' into feature/emoji-picker

Compare with previous version

changed this line in version 8 of the diff

added 2 commits

b73c2831 - emoji picker (refactor): improve types and change icons to use iconify
641e32f3 - emoji picker (fix): fix skin picker not showing on dropdown format

Compare with previous version

added 1 commit

c854a99c - (feat): export emoji picker

Compare with previous version

mentioned in commit 335b71edc3a78f83873303e622a7a0be27f89d7e

vlgi / uai-components