Feature/camera components #242

Closed 01oseluiz closed 8 months ago

In GitLab by @VitorLamego on Jun 2, 2023, 21:53

Gitlab MR Web Template

Descrição

A issue em questão tem como objetivo adicionar o componente FileSelector, responsável por adicionar arquivos a partir de um botão. A ideia de colocar o componente no szot-ui é para que o componente possa ser inserido no contexto do Form.

Tempo de desenvolvimento

1h30

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?

[ ] Google Chrome
[ ] Mozilla Firefox
[ ] IE 11 (Windows 10)
[ ] Edge
[ ] Opera
[ ] Opera GX
[x] IOS Safari
[ ] IOS Chrome
[ ] Android Chrome
[ ] Android Firefox
[x] iOS Simulator

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)
[x] 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 @VitorLamego on Jun 2, 2023, 21:54

added 1 commit

faf4f395 - FileSelector (refactor): change path of FileSelector

Compare with previous version

In GitLab by @VitorLamego on Jun 2, 2023, 21:55

added 1 commit

afc19bc3 - FileSelector (refactor): change path of FileSelector

Compare with previous version

In GitLab by @VitorLamego on Jun 7, 2023, 17:24

added 5 commits

afc19bc3...7e15de38 - 4 commits from branch master
c9271ad6 - Merge branch 'master' of gitlab.com:team-tecnologia/public-pkgs/szot-ui into...

Compare with previous version

In GitLab by @VitorLamego on Jun 7, 2023, 17:32

added 1 commit

314f07c1 - stories (fix): remove errors from merge

Compare with previous version

In GitLab by @VitorLamego on Jun 7, 2023, 17:38

marked this merge request as ready

In GitLab by @DaniloRaposo on Jun 9, 2023, 13:13

Commented on src/lib/components/formFields/FileSelector/FileSelector.stories.ts line 8

Esse é um componente usado no forms, então a documentação dele tem que está no formfields

In GitLab by @DaniloRaposo on Jun 9, 2023, 13:14

Commented on src/lib/components/formFields/FileSelector/FileSelector.stories.ts line 8

Mudar nos outros locais da documentação para estar tudo dentro da pasta formfields

In GitLab by @DaniloRaposo on Jun 9, 2023, 13:27

Commented on src/lib/components/formFields/FileSelector/FileSelector.svelte line 16

Acho que vale mudar o nome dessa variável para um nome melhor. A variável tem mais haver com os tipos de arquivo do que com o proprio aceitar

In GitLab by @DaniloRaposo on Jun 9, 2023, 13:36

Commented on src/lib/components/formFields/FileSelector/FileSelector.svelte line 31

Essas variáveis deveriam estar sendo exportadas, não?

In GitLab by @DaniloRaposo on Jun 9, 2023, 13:38

Commented on src/lib/components/formFields/FileSelector/FileSelector.svelte line 35

Essas aqui tbm não deveriam estar sendo expostas?

In GitLab by @DaniloRaposo on Jun 9, 2023, 13:49

Commented on src/lib/components/formFields/FileSelector/FileSelector.svelte line 70

    return (_value != null);

In GitLab by @DaniloRaposo on Jun 9, 2023, 13:51

Commented on src/lib/components/formFields/FileSelector/FileSelector.svelte line 85

Essa segunda reatividade não já está sendo feita na primeira não?

In GitLab by @DaniloRaposo on Jun 9, 2023, 13:55

Commented on src/lib/components/formFields/FileSelector/FileSelector.svelte line 11

Onde isso está sendo usado?

In GitLab by @DaniloRaposo on Jun 9, 2023, 13:59

Commented on src/lib/components/formFields/FileSelector/FileSelectorExample.svelte line 11

Acho que apenas label ou buttonLabel ficaria um nome melhor para a label do botão, não?

In GitLab by @DaniloRaposo on Jun 9, 2023, 14:00

Acho que seria legal documentar o uso dessa props accept, já que o que ela recebe é meio que uma string especifica

In GitLab by @VitorLamego on Jun 9, 2023, 18:50

Commented on src/lib/components/formFields/FileSelector/FileSelector.svelte line 11

É a tipagem do restProps. Para eu conseguir passar o $restProps dentro do Button eu preciso fazer essa tipagem pro usuário conseguir enviar os argumentos do Button também.

In GitLab by @VitorLamego on Jun 9, 2023, 19:44

Commented on src/lib/components/formFields/FileSelector/FileSelector.stories.ts line 8

changed this line in version 6 of the diff

In GitLab by @VitorLamego on Jun 9, 2023, 19:44

Commented on src/lib/components/formFields/FileSelector/FileSelector.svelte line 16

changed this line in version 6 of the diff

In GitLab by @VitorLamego on Jun 9, 2023, 19:44

Commented on src/lib/components/formFields/FileSelector/FileSelector.svelte line 31

changed this line in version 6 of the diff

In GitLab by @VitorLamego on Jun 9, 2023, 19:44

Commented on src/lib/components/formFields/FileSelector/FileSelector.svelte line 35

changed this line in version 6 of the diff