Skip to content

enhance(web): show attachment URL and UUID in dropzone preview - #39203

Merged
bircni merged 16 commits into
go-gitea:mainfrom
GrigorTonikyan:enhance-dropzone-attachment-preview
Sep 5, 2026
Merged

enhance(web): show attachment URL and UUID in dropzone preview#39203
bircni merged 16 commits into
go-gitea:mainfrom
GrigorTonikyan:enhance-dropzone-attachment-preview

Conversation

@GrigorTonikyan

@GrigorTonikyan GrigorTonikyan commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

When editing comments with multiple attachments, Dropzone preview cards only display filenames without identifying UUIDs or URLs, making it difficult to match cards to Markdown attachment tags when removing or rearranging files.

This change:

  • Sets data-tooltip-content with the name and UUID on preview cards.
  • Converts attachment filenames and thumbnail images into clickable links opening the attachment in a new tab.

Resolves #38802

Also fix the isImageFile regression caused by #30513 which stopped the preview image from showing.

Show attachment URL and UUID on dropzone preview cards and linkify filenames and thumbnails to make cross-referencing with Markdown text easy.

Resolves go-gitea#38802

Assisted-by: Antigravity:gemini-3.7-flash
@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Sep 1, 2026
@github-actions github-actions Bot added the type/enhancement An improvement of existing functionality label Sep 1, 2026
Comment thread web_src/js/features/dropzone.test.ts Outdated
Comment thread web_src/js/features/dropzone.ts Outdated
Comment thread web_src/js/features/dropzone.ts Outdated
@wxiaoguang
wxiaoguang marked this pull request as draft September 2, 2026 19:03
@GrigorTonikyan
GrigorTonikyan marked this pull request as ready for review September 2, 2026 22:52
@wxiaoguang

wxiaoguang commented Sep 4, 2026

Copy link
Copy Markdown
Contributor
image image image image

@GiteaBot GiteaBot added lgtm/need 1 This PR needs approval from one additional maintainer to be merged. and removed lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. labels Sep 4, 2026
@wxiaoguang wxiaoguang added this to the 28.0.0 milestone Sep 4, 2026
# Conflicts:
#	templates/repo/issue/fields/textarea.tmpl
#	templates/repo/upload.tmpl
@GiteaBot GiteaBot added lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. and removed lgtm/need 1 This PR needs approval from one additional maintainer to be merged. labels Sep 5, 2026
@bircni
bircni enabled auto-merge (squash) September 5, 2026 08:34
@bircni bircni added the reviewed/wait-merge This pull request is part of the merge queue. It will be merged soon. label Sep 5, 2026
@bircni
bircni merged commit bde1af5 into go-gitea:main Sep 5, 2026
23 checks passed
@GiteaBot GiteaBot removed the reviewed/wait-merge This pull request is part of the merge queue. It will be merged soon. label Sep 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. type/enhancement An improvement of existing functionality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Show hash/URL of attachments when editing

4 participants