Skip to content

(feat): add new tooltip component#260

Open
01oseluiz wants to merge 5 commits intodevelopfrom
feat/tooltip-component
Open

(feat): add new tooltip component#260
01oseluiz wants to merge 5 commits intodevelopfrom
feat/tooltip-component

Conversation

@01oseluiz
Copy link
Collaborator

In GitLab by @Wander-lan on Sep 11, 2023, 16:15

Gitlab MR Web Template

Descrição

Cria um novo componente de tooltip para ser utilizado como o novo padrão.

Tipo de mudança

  • Bug fix (Correção de um problema que não quebrava algo grande)
  • 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)
  • Refactor (Alteração que melhora algo que já existente)

Em quais navegadores foi testado?

  • Google Chrome
  • Mozilla Firefox
  • IE 11 (Windows 10)
  • Edge
  • Opera
  • Opera GX
  • IOS Safari
  • IOS Chrome
  • Android Chrome
  • Android Firefox

Testei as seguintes resoluções no navegador:

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

Qualidade

  • Segui os padrões de criação da stack em svelte
  • Eu revisei ou reli o meu próprio código
  • Segui o formato de criação e nomenclatura de variáveis CSS

Documentação

  • Documentei onde era mais necessário e difícil de entender no meu código
  • Documentei as variáveis CSS que meu componente externaliza
  • Criei bons exemplos no storybook para meu componente

Teste

  • As mudanças que esta MR implementa possui testes automatizados
  • Executei e garanto que os testes automatizados já criados estão todos passando
  • Executei e garanto que não há erros de lint
  • Executei e garanto que não há erros de typescript

MR

  • Eu conferi a checklist de atividades, e expliquei no meu merge request caso algo não tenha
    sido necessário ou possível

@01oseluiz
Copy link
Collaborator Author

In GitLab by @Wander-lan on Sep 11, 2023, 18:34

requested review from @michaelcrds

@01oseluiz
Copy link
Collaborator Author

In GitLab by @michaelcrds on Sep 12, 2023, 17:18

Commented on src/lib/components/Tooltip/Tooltip.svelte line 6

Já existe um tipo TPosition, se vc procurar por ele vc vai ver que ficou faltando algumas posições

@01oseluiz
Copy link
Collaborator Author

In GitLab by @michaelcrds on Sep 12, 2023, 17:19

image

No Ora a caixa possui uma espécie de seta apontando para o icone

@01oseluiz
Copy link
Collaborator Author

In GitLab by @michaelcrds on Sep 12, 2023, 17:22

image

Faltou opções de customização como:

  • Cor e tamanho do icone
  • Cor e tamanho dos textos
  • Paddings e cor da caixa de texto

Essas são apenas algumas customizações que eu pensei que eventualmente alguém poderia acabar precisando

@01oseluiz
Copy link
Collaborator Author

In GitLab by @michaelcrds on Sep 12, 2023, 17:25

Esse componente não foi desenhado pelo UX/UI então você pode ir atrás deles para validar os meus apontamentos

@01oseluiz
Copy link
Collaborator Author

In GitLab by @michaelcrds on Sep 12, 2023, 17:28

Você vai precisar criar um arquivo .mdx para documentar as variáveis CSS

@01oseluiz
Copy link
Collaborator Author

In GitLab by @Wander-lan on Sep 13, 2023, 20:16

Commented on src/lib/components/Tooltip/Tooltip.svelte line 6

changed this line in version 2 of the diff

@01oseluiz
Copy link
Collaborator Author

In GitLab by @Wander-lan on Sep 13, 2023, 20:16

added 1 commit

  • 9021415 - Tooltip (feat): add new style variables

Compare with previous version

@01oseluiz
Copy link
Collaborator Author

In GitLab by @Wander-lan on Sep 13, 2023, 20:19

added 1 commit

  • 00b84d9 - Tooltip (refactor): run prettier

Compare with previous version

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants