Feature/Remove autofocus from button - [merged]

[01oseluiz]

In GitLab by @DaniloRaposo on Jun 20, 2023, 12:46

Merges feature/remove-autofocus-from-button -> develop

Gitlab MR Web Template

Descrição

Remover autofocus do botão de close do modal

Fixes #121

Tempo de desenvolvimento

20 minutos

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

[01oseluiz]

In GitLab by @victorandre957 on Jun 21, 2023, 18:09

approved this merge request

[01oseluiz]

changed target branch from master to develop

[01oseluiz]

added 5 commits

• 0b701a50...e510d18e - 4 commits from branch develop
• 24420986 - Merge remote-tracking branch 'origin/develop' into feature/remove-autofocus-from-button

Compare with previous version

[01oseluiz]

added 3 commits

• 24420986...62813edb - 2 commits from branch develop
• 46b40b0d - Merge remote-tracking branch 'origin/develop' into feature/remove-autofocus-from-button

Compare with previous version

[01oseluiz]

added 1 commit

• bcfed82f - changelog (refactor): add this fix

Compare with previous version

[01oseluiz]

mentioned in commit 7021eb50dbb05460d364fdf6b50f958dca092025