-
Notifications
You must be signed in to change notification settings - Fork 33
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
[Editor]: Change title input to handle multiline #1040
Conversation
Affected libs:
|
📷 Screenshots are here! |
7d53456
to
1ca084b
Compare
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.
Thank you @cmoinier for adding this multiline feature!
Could you have another look at this? I think it is not yet working as expected. The icon for editing the title is no longer working.
Thanks!
libs/feature/editor/src/lib/components/record-form/form-field/form-field.component.ts
Show resolved
Hide resolved
789581f
to
b073f6a
Compare
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.
There are some little things to clean up before merging.
I'll test it again tomorrow. Thank you @cmoinier !
libs/feature/editor/src/lib/components/record-form/form-field/form-field.component.ts
Outdated
Show resolved
Hide resolved
libs/feature/editor/src/lib/components/record-form/form-field/form-field.component.html
Outdated
Show resolved
Hide resolved
libs/feature/editor/src/lib/components/record-form/form-field/form-field.component.ts
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.
Thank you for the cleanup @cmoinier !
The PR looks good to go 🚀
Sorry for the delay!
Description
This PR changes the title input of the editor to a textarea that can handle multilines, for consultation and editing.
Screenshots
Quality Assurance Checklist
breaking change
labelbackport <release branch>
label