Skip to content

fix: 252 Corrigindo padding desnecessário nos cards de distribuidora

Descrição

Foram removidos os paddings desnecessários dos cards de distribuidora.

Motivação e contexto

Os espaços em branco na parte superior e inferior dos cards de distribuidora foram eliminados. O espaço superior estava relacionado ao reaproveitamento de um componente que contém um ícone, o qual não é necessário para os cards de distribuidora.

Issue relacionada

#252

Principais implementações

Lista das principais implementações ou mudanças realizadas neste merge request.

  • O componente Box estava fora da lógica relacionada ao shouldShowFavoriteIconButton, o que resultava em um espaço em branco desnecessário para o card. Para corrigir isso, o componente Box foi movido para dentro da lógica do shouldShowFavoriteIconButton.
  • Aparentemente existe um espaço em branco desnecessário na parte inferior do card de distribuidora. Para corrigir isso, a propriedade minHeight do componente CardWrapper, que é externo ao componente Card, foi removida. (consulte as notas adicionais sobre essa implementação)

Como isso foi testado?

Essa alteração foi testada localmente e no navegador, executando a aplicação.

Tipos de mudança

Marque com um "x" as caixas correspondentes:

  • Correção de bug (não-interrompção de mudança que corrige um problema)
  • Nova funcionalidade (mudança que adiciona funcionalidade)
  • Mudança de quebra (alteração que interrompe o comportamento anterior)
  • Melhoria de desempenho
  • Mudança de refatoração (alteração que não adiciona nova funcionalidade nem corrige bugs)
  • Outra mudança (descreva abaixo):

Checklist

Marque com um "x" as caixas correspondentes:

  • O código segue as diretrizes de estilo deste projeto
  • Foram adicionados testes para cobrir as novas funcionalidades ou mudanças
  • Todos os testes passam
  • As alterações foram revisadas por pelo menos um colega de equipe
  • A documentação foi atualizada, se necessário
  • As alterações não quebram o fluxo de trabalho de nenhum outro usuário ou sistema

Notas adicionais

Pelo que eu vi no código, o componente Card está sendo utilizado em muitos lugares, consequentemente, o CardWrapper também está. Então, a mudança feita retirando a propriedade minHeight do CardWrapper pode afetar outras páginas além da distribuidora. Tentei verificar se houve mudanças em outras páginas, porém posso ter deixado passar alguma.

Merge request reports