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

feat(PoIcon): anuncia depreciação #2136

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

renanarosario
Copy link
Contributor

Foi adiconado mesagens de depreciação do PoIcon a partir da v20.

Fixes DTHFUI-9008

BREAKING CHANGE: anuncia depreciação

Foi adicionado mensagens onde é chamado a Biblioteca de ícones (/guides/icons), nas documentações.

Antes:

Depois:

  • Deprecated v20.x.x. É possível usar qualquer um dos ícones da Biblioteca de ícones. conforme exemplo abaixo:

  • Utilize os ícones disponíveis na Lib Phosphor, conforme exemplo:

  • <po-component

  • [p-property]="[{ label: 'ICON', icon: 'ph ph-user' }]">

  • 
    

< COMPONENTE >

< NÚMERO DA ISSUE >


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?

Qual o novo comportamento?

Simulação

@renanarosario renanarosario force-pushed the icon/DTHFUI-9008 branch 2 times, most recently from d0322ab to 4d158a3 Compare July 11, 2024 17:33
A Lib Po Icon está depreciada e será removida na v20 do PO UI.

Fixes DTHFUI-9008

BREAKING CHANGE: Depreciação da Lib Po Icon
A Lib Po Icon está depreciada e será removida na v20 do PO UI.

Antes:

É possível usar qualquer um dos ícones da [Lib PO Icon](/guides/icons).

Depois:

Utilize os ícones disponíveis na Lib [Phosphor](/guides/icons), conforme documentação.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants