Skip to content
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

make filename accessible in dropzone #26487

Closed
wants to merge 1 commit into from

Conversation

earl-warren
Copy link
Contributor

  • In Forgejo you can upload files by drag n dropping them in selective 'zones', these are handled by
    dropzone. While waiting for it to finish uploading, you can see the file's size and name.
  • The name was conviently placed under the progress bar, so as an file is uploading you couldn't see the filename anymore.
  • This patch fixes that by simply moving the filename a few pixels down.
  • Resolves https://codeberg.org/Codeberg/Community/issues/1104

Refs: https://codeberg.org/forgejo/forgejo/pulls/1242

(cherry picked from commit 58d500aafdd2dd5756427376763132cdcc5b608c)

- In Forgejo you can upload files by drag n dropping them in selective
'zones', these are handled by
[dropzone](https://github.com/dropzone/dropzone). While waiting for it
to finish uploading, you can see the file's size and name.
- The name was conviently placed under the progress bar, so as an file
is uploading you couldn't see the filename anymore.
- This patch fixes that by simply moving the filename a few pixels down.
- Resolves https://codeberg.org/Codeberg/Community/issues/1104

Refs: https://codeberg.org/forgejo/forgejo/pulls/1242

(cherry picked from commit 58d500aafdd2dd5756427376763132cdcc5b608c)
@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Aug 14, 2023
@pull-request-size pull-request-size bot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Aug 14, 2023
@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 Aug 14, 2023
@lunny lunny added the topic/ui Change the appearance of the Gitea UI label Aug 14, 2023
@wxiaoguang
Copy link
Contributor

wxiaoguang commented Aug 14, 2023

This change is not ideal.

For UI alignment, you might need to put the progress bar below the filename, like this. Otherwise there is too much space between the "size" and "filename"

image

And from the HTML layout perspective, the dropzone also prefers to put the progress bar after details (size+filename):

image

And the top: 50% on the progress bar causes another problem: the UI flicks when the uploading is completing.

@wxiaoguang
Copy link
Contributor

-> Move dropzone progress bar to bottom to show filename when uploading #26492

@go-gitea go-gitea locked as resolved and limited conversation to collaborators Nov 13, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
lgtm/need 1 This PR needs approval from one additional maintainer to be merged. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. topic/ui Change the appearance of the Gitea UI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants