po-angular icon indicating copy to clipboard operation
po-angular copied to clipboard

fix(list-view): ajusta atributo `visible` da propriedade `p-actions`

Open wsteixeira opened this issue 2 years ago • 5 comments

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 avatar Apr 22 '22 22:04 wsteixeira

@wsteixeira você poderia resolver o conflito na branch? acho que basta atualizar com a master via rebase e verificar as alterações

alinelariguet avatar Aug 22 '22 11:08 alinelariguet

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!

wsteixeira avatar Aug 22 '22 12:08 wsteixeira

Bom dia @wsteixeira , para mim ainda aparece a seguinte mensagem: This branch cannot be rebased due to conflicts.

alinelariguet avatar Aug 22 '22 12:08 alinelariguet

Olá @alinelariguet,

Vou fazer então pelo Vscode, mas segue o print que aparece para mim! image

wsteixeira avatar Aug 22 '22 12:08 wsteixeira

Olá @alinelariguet,

Ufa! Depois de três tentativas passou pelo CI, rsrs!

wsteixeira avatar Aug 22 '22 13:08 wsteixeira

Atualizei o App para simulação.

wsteixeira avatar Oct 07 '22 18:10 wsteixeira