Skip to content

Checkbox (feat): add disable props #267

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

01oseluiz
Copy link
Collaborator

In GitLab by @Wander-lan on Oct 13, 2023, 19:53

Gitlab MR Web Template

Descrição

Adiciona um novo props de "disabled" no componente de checkbox.

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)

Checklist:

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

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