Skip to content

feat(editor): Enable gapcursor, allows to put cursor between images#5095

Merged
mejo- merged 3 commits into
mainfrom
feat/gapcursor
Dec 6, 2023
Merged

feat(editor): Enable gapcursor, allows to put cursor between images#5095
mejo- merged 3 commits into
mainfrom
feat/gapcursor

Conversation

@mejo-

@mejo- mejo- commented Dec 6, 2023

Copy link
Copy Markdown
Member

📝 Summary

Enable gapcursor extension and remove workaround to add newlines between images when uploading more than one.

🖼️ Screenshots

🏡 After
recording1

🏁 Checklist

  • Code is properly formatted (npm run lint / npm run stylelint / composer run cs:check)
  • Sign-off message is added to all commits

mejo- added 3 commits December 6, 2023 12:44
Signed-off-by: Jonas <jonas@freesources.org>
Signed-off-by: Jonas <jonas@freesources.org>
Fixes: #3427

Signed-off-by: Jonas <jonas@freesources.org>
@mejo- mejo- added this to the Nextcloud 29 milestone Dec 6, 2023
@mejo- mejo- added the ux label Dec 6, 2023

@max-nextcloud max-nextcloud left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

❤️ Thanks for looking into this!

@max-nextcloud

Copy link
Copy Markdown
Collaborator

This might also address some aspects of #3111

@juliusknorr juliusknorr left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚀

(I need those nice test images as well 😁 🦖)

@mejo-

mejo- commented Dec 6, 2023

Copy link
Copy Markdown
Member Author

This might also address some aspects of #3111

Unfortunately not yet. We might be able to configure the mentions node to allow gapcursor before it, but out of the box it's not possible.

@mejo-

mejo- commented Dec 6, 2023

Copy link
Copy Markdown
Member Author

(I need those nice test images as well 😁 🦖)

I love them 🦕 They're from https://prosemirror.net/examples/dino/

@mejo- mejo- merged commit c442ff8 into main Dec 6, 2023
@mejo- mejo- deleted the feat/gapcursor branch December 6, 2023 12:38
@mejo-

mejo- commented Dec 6, 2023

Copy link
Copy Markdown
Member Author

/backport to stable28

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Inserting image into line of text will add line break two chars after

3 participants