twenty
twenty copied to clipboard
Tasks editor should be scrollable to show complete content
Bug Description
While typing in tasks editor, if content becomes larger than the available space, it doesn't scroll further. Due to which, the some parts at end of the editor is not visible until user deletes some content above. Also, while user presses "Enter" key, it doesn't automatically scroll to next part due to which the current cursor position is not into view.
https://github.com/twentyhq/twenty/assets/60139930/cfb71e3a-6cbb-4233-b506-b512fca5ea8e
Expected behavior
- It should be scrollable up and down showing complete content in the editor.
- On pressing "Enter", it should scroll to bring the current cursor into view.
Good catch thanks!
I think the issue is with the comment bar. For me it does scroll automatically when I type but not enough, it's still hidden by the command bar's height.
@Bonapara I feel like we should review the comment feature, maybe disable it in the meantime? One thing that would work best would be to have a line-by-line comments? I think that's doable and we could reuse most of the current work (maybe half a day of backend work + two days of frontend to adapt properlu)
We could consider a temporary margin-bottom
to prevent content from hiding under the comments assuming it's indeed the problem.
@FelixMalfait, could you specify your thoughts on line-by-line comments? Do you mean a Notion-like feature to comment on individual sentences or words? I plan to work on improving the comment section soon.
Do you mean a Notion-like feature to comment on individual sentences or words?
Yes that's what I meant. Cool if you had plans to improve it! I'll let you give the final word on what needs to be done on this issue / add relevant tags if good first issue.
Felix is correct. The problem is that the comment section is covering the bottom of the note. To fix this, we need to add a 64px margin-bottom to the content section. This will also enhance the visibility between the content and the comments:
This now seems to be fixed on the main branch!