MessageBox (feat): Alow Edit Message in component - [merged] #265

Closed 01oseluiz closed 9 months ago

In GitLab by @victorandre957 on Oct 5, 2023, 18:07

Merges feature/alow-edit-message-in-component -> develop

Gitlab MR Web Template

Descrição

Permite que a mensagem no MessageBox seja editada dentro do componente.

Drafts: Colocar o que deve ser finalizado antes do merge

Fixes #(numero da issue)

Tempo de desenvolvimento

Insira aqui o tempo estimado para a tarefa, e após, o quanto foi gasto na realização das tarefas.

Veja como inserir de forma correta em gitlab docs - time tracking.

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)
[ ] Refactor (Alteração que melhora algo que já existente)

Em quais navegadores foi testado?

[X] 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
[X] 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)

Checklist:

Qualidade

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

Documentação

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

Teste

[ ] As mudanças que esta MR implementa possui testes automatizados
[X] 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

MR

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

In GitLab by @DaniloRaposo on Oct 5, 2023, 19:55

Commented on src/lib/components/Messages/Components/MessageBox/MessageBox.svelte line 129

Esse bind faz com que mesmo o usuário clicando no botão de cancelar, se ele mudou o texto, então o valor do texto vai ficar mudado no componente. Esse é o comportamento esperado?

In GitLab by @DaniloRaposo on Oct 5, 2023, 19:56

Ao editar a mensagem, os botões estão indo para o lado a medida que o texto do textarea vai para a direita, ao invés do texto ir para a linha de baixo

In GitLab by @victorandre957 on Oct 5, 2023, 22:45

Commented on src/lib/components/Messages/Components/MessageBox/MessageBox.svelte line 129

changed this line in version 2 of the diff

In GitLab by @victorandre957 on Oct 5, 2023, 22:45

added 1 commit

185ce7c7 - MessageBox (fix): not edit message on cancel

Compare with previous version

In GitLab by @victorandre957 on Oct 5, 2023, 22:46

Tem alguma ideia de como deixar isso ocupando todo o espaço disponível? eu deixei assim para ele crescer até o tamanho da caixa definida por fora.

In GitLab by @DaniloRaposo on Oct 13, 2023, 12:46

requested review from @DaniloRaposo

In GitLab by @DaniloRaposo on Oct 13, 2023, 12:52

Commented on src/lib/components/Messages/Components/MessageBox/MessageBox.svelte line 80

Não sei se é possível, mas o fato dessa variável ser atribuida como zero para o caso em que nenhum item está sendo editado pode dar problema se por ventura uma mensagem possuir o id 0. Acho melhor usar um undefined ou um null tanto aqui quanto na definição da variável

In GitLab by @victorandre957 on Oct 13, 2023, 19:10

added 1 commit

4e2f6a80 - MessageBox (refactor): set fixed size for edit field

Compare with previous version

In GitLab by @DaniloRaposo on Oct 16, 2023, 11:30

Commented on src/lib/components/Messages/Components/MessageBox/MessageBox.svelte line 80

changed this line in version 4 of the diff

In GitLab by @DaniloRaposo on Oct 16, 2023, 11:30

added 1 commit

7712da6f - MessageBox (refactor): use undefined to specify message edit id when any message is editing

Compare with previous version

In GitLab by @DaniloRaposo on Oct 16, 2023, 11:32

resolved all threads

In GitLab by @DaniloRaposo on Oct 16, 2023, 11:39

added 1 commit

2d789163 - changelog (chore): v3.13.0

Compare with previous version

In GitLab by @DaniloRaposo on Oct 16, 2023, 11:42

approved this merge request

In GitLab by @DaniloRaposo on Oct 16, 2023, 11:42

mentioned in commit b2349b1976d516e84c867663f5ba04b25c117f2b

vlgi / uai-components