-
-
Notifications
You must be signed in to change notification settings - Fork 398
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 linebreaks in WYSWYG editor #6996
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The PR diff size of 11974 lines exceeds the maximum allowed for the inline comments feature.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @lahdeero ! Everything seems ok, but first of all you have to rebase your branch due to some conflicts.
- Fixed bug where backspace deleted text twice - Fixed bug where backspace didnt delete two invisible characters at once - Added test
53cc810
to
7e9ad54
Compare
@ivan-mr Done. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! thanks @lahdeero .
🎩 What? Why?
Common content management need is that user can make single line breaks into text. Currently in Decidim it's impossible to make line breaks, paragraph spacing comes after each press of the enter. For example, admin wants to add contact details into contact information page like this:
This is more practical to do using line breaks / br tags and this module (pull request) adds line break / br tag feature to quill (editor). Works with shift+enter or clicking the icon in the toolbar.
📌 Related Issues
#2000
slab/quill#1187
slab/quill#252
Testing
Describe the best way to test or validate your PR.
📋 Checklist
🚨 Please review the guidelines for contributing to this repository.
docs/
.📷 Screenshots
Please add screenshots of the changes you're proposing