po-angular
po-angular copied to clipboard
fix(list-view): ajusta atributo `visible` da propriedade `p-actions`
Quando a quantidade de actions
era menor ou igual a dois e o atributo visible
da propriedade p-actions
tinha o valor false
atribuído por função, não estava ocultando o botão da action
conforme esperado.
Fixes #1265
List View
1265
PR Checklist [Revisor]
- [x] Padrão de Commit (Coeso, de acordo com o que está sendo realizado)
- [x] Código (Boas práticas, nome de variavéis/métodos, etc.)
- [x] Testes unitários (Cobre a situação implementada e coverage está mantido)
- [ ] Documentação (Clara, objetiva e com exemplos caso necessário)
- [ ] Samples (A implementação possui exemplo no Labs/Caso de uso)
- [ ] Rodado em navegadores suportados (Chrome, FireFox, Edge)
Qual o comportamento atual?
Quando a quantidade de actions
era menor ou igual a dois e o atributo visible
da propriedade p-actions
tinha o valor false
atribuído por função, não estava ocultando o botão da action
conforme esperado.
Qual o novo comportamento?
Independente da quantidade de actions
, o atributo visible
da propriedade p-actions
quando tem o valor false
atribuído por função, está ocultando o botão da action
conforme esperado.
Simulação
Segue o App para simulação.
@wsteixeira você poderia resolver o conflito na branch? acho que basta atualizar com a master via rebase e verificar as alterações
Olá @alinelariguet,
Eu fiz direto pelo site através do botão Resolve conflicts
e depois o botão Commit merge
(acho que era isso, rsrs), parece que deu certo, favor verificar novamente!
Bom dia @wsteixeira , para mim ainda aparece a seguinte mensagem: This branch cannot be rebased due to conflicts
.
Olá @alinelariguet,
Vou fazer então pelo Vscode, mas segue o print que aparece para mim!
Olá @alinelariguet,
Ufa! Depois de três tentativas passou pelo CI
, rsrs!
Atualizei o App para simulação.