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.

Sign up for GitHub

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

Jump to bottom

Unsaved changes feature not working properly on a record with rich text editor #1478

Closed
kelanik8 opened this issue Oct 17, 2023 · 0 comments · Fixed by #1632
Closed

Unsaved changes feature not working properly on a record with rich text editor #1478

kelanik8 opened this issue Oct 17, 2023 · 0 comments · Fixed by #1632
Assignees
@kelanik8
Labels
bug Something isn't working
Milestone
2023.9.3

Comments

@kelanik8
Copy link
Contributor

Issue:
Unsaved changes feature not working properly on a record with rich text editor

How to reproduce:

  • Create a module with a text field that has rich text configurator option enabled
  • Create a record
  • Edit the record and revert the changes
  • Try leaving the page and the unsaved changes alert is triggered

The problem is the rich text editor adds this"<p></p><hr contenteditable=\"false\"><p></p>" when the field has an empty value. It happens in different patters
@kelanik8 kelanik8 added the bug Something isn't working label Oct 17, 2023
@kelanik8 kelanik8 added this to the 2023.9.1 milestone Oct 17, 2023
@kelanik8 kelanik8 self-assigned this Oct 17, 2023
@kelanik8 kelanik8 mentioned this issue Oct 19, 2023
Closed
@Fajfa Fajfa modified the milestones: 2023.9.1, 2023.9.2 Nov 27, 2023
@Fajfa Fajfa modified the milestones: 2023.9.2, 2023.9.3 Dec 13, 2023
@kelanik8 kelanik8 linked a pull request Jan 16, 2024 that will close this issue
Fix unsaved changes alert from triggering when RTE is empty #1632
Merged
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@kelanik8 kelanik8

Labels
bug Something isn't working
Projects
None yet
Milestone
2023.9.3
Development

Successfully merging a pull request may close this issue.

Fix unsaved changes alert from triggering when RTE is empty cortezaproject/corteza
2 participants
@Fajfa @kelanik8