Refactor/Adjust close button z index modal - [merged]

[01oseluiz]

In GitLab by @DaniloRaposo on Jun 29, 2023, 20:07

Merges refactor/adjust-close-button-z-index-modal -> develop

Gitlab MR Web Template

Descrição

Mudança no mixin de borda que ocasiona resolução de problema relacionado ao z-index do botão no header do modal

Fixes #133

Tempo de desenvolvimento

1 hora

Tipo de mudança

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

• [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)
• [X] 1024 x 768 (iPad horizontal)
• [X] 375 x 812 (iPhone X)
• [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 Jul 5, 2023, 17:23

approved this merge request

[01oseluiz]

changed target branch from master to develop

[01oseluiz]

added 3 commits

• 50a02cf3 - 1 commit from branch develop
• e657eb92 - Merge remote-tracking branch 'origin/develop' into refactor/adjust-close-button-z-index-modal
• 01a2e3fa - changelog (refactor): add this fix

Compare with previous version

[01oseluiz]

approved this merge request

[01oseluiz]

mentioned in commit 8083b5660047e6959d58935f8b341dce4ad57af3