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(rich-text-body): realiza sanitização do texto colado #2098

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

Conversation

bpereiraalmeida7
Copy link

@bpereiraalmeida7 bpereiraalmeida7 commented Jun 11, 2024

po-rich-text

#1872


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?
O componente po-rich-text ao receber um texto colado, não sanitiza o css do mesmo, gravando informações desnecessárias na model.

Qual o novo comportamento?

Realiza a sanitização da model, para que receba apenas o valor, deixando a estilização a cargo do usuário, através das ferramentas disponibilizadas no componente.

Simulação

Esta correção pode ser validada utilizando o sample labs no portal.

O componente po-rich-text ao receber um texto colado, não sanitiza o css do mesmo, gravando informações desnecessárias na model.
Realiza a sanitização da model, para que receba apenas o valor, deixando a estilização a cargo do usuário, através das ferramentas disponibilizadas no componente.

Fixes po-ui#1872
@bpereiraalmeida7
Copy link
Author

Falha no job de notificação do Discord

@alinelariguet
Copy link
Member

Olá! @bpereiraalmeida7 , tudo bem?
o componente tem esse comportamento hoje, caso a gente libere a funcionalidade implementada nesta contribuição todos os produtos terão o comportamento alterado por padrão desta nova funcionalidade. Na época do desenvolvimento deste componente já era esperado que o estilo colado seria aplicado. Sugiro a criação de uma propriedade para habilitar ou não este recruso para não quebrarmos o comportamento padrão do componente. Você poderia ajustar esse ponto na sua contribuição?

Att.

@alinelariguet alinelariguet added the status: awaiting author Additional information has been requested from the author label Aug 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: awaiting author Additional information has been requested from the author
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants