Skip to content
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(page-default): ajusta propriedade visible do PoPageAction #1861

Merged
merged 1 commit into from
Oct 25, 2024

Conversation

wsteixeira
Copy link
Contributor

@wsteixeira wsteixeira commented Oct 31, 2023

A propriedade visible do PoPageAction não estava funcionando como função, apenas como boolean.

Foi feito um ajuste para que a mesma funcione como função de forma semelhante a propriedade disabled, conforme indica a documentação.

Fixes #1836

Page Default

1836


PR Checklist [Revisor]

  • Padrão de Commit (Coeso, de acordo com o que está sendo realizado)
  • Código (Boas práticas, nome de variavéis/métodos, etc.)
  • 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?
A propriedade visible do PoPageAction não estava funcionando como função, apenas como boolean.

Qual o novo comportamento?
Foi feito um ajuste para que a mesma funcione como função de forma semelhante a propriedade disabled, conforme indica a documentação.

Simulação
Para simulação pode ser usado o APP.

@wsteixeira wsteixeira force-pushed the page-default/1836 branch 3 times, most recently from 0a222e9 to 91e54eb Compare October 31, 2023 15:59
@alinelariguet alinelariguet added the hacktoberfest-accepted Aceito para a hacktoberfest label Oct 31, 2023
@likid
Copy link

likid commented Oct 16, 2024

1 ano, esse PR ainda pendente e não sei porque ainda não foi aceito. Atualmente existe um bug tanto no page-default quanto no list-view onde ao passar uma função para definir se a ação será visível ao invés de receber o item por parâmetro, ele recebe a própria função especificada na configuração do componente, tornando inútil este recurso de utilizar uma função.

Hoje no page-default para tornar uma ação visível ou não é necessário fazer uma gambiarra redefinindo o array de ações conforme a condição.

@pedrodominguesp @jnrpalma @alinelariguet

@alinelariguet
Copy link
Member

Priorizada para revisão.

@anabye
Copy link
Contributor

anabye commented Oct 17, 2024

Boa tarde, Wanderley.

Obrigada pela contribuição e paciência.
Sua solução funciona bem quando as actions estão visíveis na tela, mas não quando deveriam iniciar dentro de um dropdown. Quando a action é inicializada com visible: false, ela não fica visível mesmo após ser alterada para true, dependendo do caso, afeta a visão também das que iniciaram como true e posteriormente deveriam aparecer no dropdow.

DTHFUI-10070_cr_bug_dropdow
DTHFUI-10070_cr_bug_dropdow_02

Peço que faça os ajustes necessários para que possamos prosseguir com a validação.

Ficamos no aguardo.

A propriedade `visible` do `PoPageAction` não estava funcionando como
função, apenas como `boolean`.

Foi feito um ajuste para que a mesma funcione como função de forma
semelhante a propriedade `disabled`, conforme indica a documentação

Fixes po-ui#1836
@wsteixeira
Copy link
Contributor Author

Olá @anabye,

PR atualizada conforme solicitado!

@anabye
Copy link
Contributor

anabye commented Oct 21, 2024

Boa tarde Wanderley,

Agradeço o retorno.

O listbox está apresentando os valores corretamente, porém identifiquei um comportamento inesperado que também pode ser observado ao construir o portal. Os dados no listbox não ficam mais visíveis quando rolo o po-page-default, mesmo o botão aparentando estar aberto.

Antes:
DTHFUI-10070_antes

Depois:
DTHFUI-10070_depois

Assim que esse ajuste for realizado e conseguirmos revisar até sexta-feira, faremos a publicação da correção na próxima segunda-feira (28.10).

Aguardo no aguardo.

@wsteixeira
Copy link
Contributor Author

wsteixeira commented Oct 24, 2024

Olá @anabye,

Eu não entendi, se o ajuste deste comportamento será feito pela equipe do PO-UI ou se você está aguardando que eu faça este ajuste!

Pelo teste que eu realizei na versão atual do portal este comportamento já existe independente de minha contribuição!

Atenciosamente,

@alinelariguet
Copy link
Member

Boa tarde! @wsteixeira muito obrigada por mais uma contribuição, pedimos desculpas pelo tempo em espera, estamos trabalhando para melhorar este processo. Ocorreu sim uma mudança no comportamento, mas por conta também do tempo que demoramos para análise vamos absorver aqui do nosso lado. Vamos liberar essa contribuição na próxima release.
Peço que pedidos de priorização sejam feitos através de issues de apoio ou comunicação via ferramenta interna para melhor visibilidade e priorização.
Obrigada mais uma vez.

@anabye anabye merged commit b503b19 into po-ui:master Oct 25, 2024
7 of 8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Page Default: O atributo visible da propriedade actions não está funciona como função
6 participants