-
Notifications
You must be signed in to change notification settings - Fork 6
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
FIX - indicativo de resultados da search #326
The head ref may contain hidden characters: "109-espa\u00E7os-indicativo-de-quantidade-de-resultados-encontrados-parece-um-bot\u00E3o"
FIX - indicativo de resultados da search #326
Conversation
@fns21 poderia revisar, por favor? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Segue meu feedback de revisão:
O código foi revisado e constatado que as mudanças de fato funcionam. O componente background-color: var(--mc-white) foi removido da classe "foundResults" e isso resolveu o problema.
O método de avaliação foi clonar a branch e fazer a verificação local.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
esse arquivo precisa ser removido do commit pois ele é gerado no processo de build da aplicação.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
tá bem dificil de revisar o que mudou nos arquivos.
vale configurar o seu editor para seguir o que está descrito no .editorconfig ou se lá estiver errado, vale abrir um PR para corrigir.
modificar a formatação do arquivo junto com outras modificações são mais propenas a erros.
removido alterações desnecessárias |
586927c
to
51d5157
Compare
Descrição
Altera o estilo do texto que mostra a quantidade de itens mostrados na página (simplesmente remove o fundo braco).
Validação
Issues relacionadas
[ESPAÇOS] Indicativo de quantidade de resultados encontrados parece um botão #109