Skip to content

Use insertAttachmentPreview instead of insertAttachment on pasting or…#8916

Open
hiobjan wants to merge 2 commits into
nextcloud:mainfrom
Dataport:issue/8745-drag-drop-preview
Open

Use insertAttachmentPreview instead of insertAttachment on pasting or…#8916
hiobjan wants to merge 2 commits into
nextcloud:mainfrom
Dataport:issue/8745-drag-drop-preview

Conversation

@hiobjan

@hiobjan hiobjan commented Jul 27, 2026

Copy link
Copy Markdown

📝 Summary

Problem / Usecase
When adding a file to a text via drag and drop, no preview window is generated. It also isn't possible to optionally enable the preview.

Solution
Use the same behaviour/code as by inserting via smartpicker. I didnt change the behaviour of images, because the attachment-view is better than the attachment-preview. #8774

🖼️ Screenshots

🏚️ Before
image
🏡 After
image

🚧 TODO

🏁 Checklist

  • Code is properly formatted (npm run lint / npm run stylelint / composer run cs:check)
  • Sign-off message is added to all commits
  • [] Tests (unit, integration and/or end-to-end) passing and the changes are covered with tests
  • Documentation (README or documentation) has been updated or is not required

🤖 AI (if applicable)

  • The content of this PR was partly or fully generated using AI tools
  • The AI-generated content was reviewed, comprehended and tested by a human

… droping non-image-files

Signed-off-by: Janina Hiob <user-y5tppt@dev.dataport.de>
Assisted-by: GPT-5.3-Codex
Signed-off-by: Janina Hiob <user-y5tppt@dev.dataport.de>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Paste PDF files with preview by default + add option to toggle mode

1 participant