Skip to content

Conversation

@tpokorra
Copy link

@tpokorra tpokorra commented Aug 19, 2025

This PR adds a button to the issues list, and when you click on it, the currently displayed issues will be downloaded as an Excel file.

fixes #29906

Screenshot after adding the button "Export to Excel":

grafik

Alternatively, perhaps it is better to add the button to the Actions?

Screenshot 2025-11-18 154725

@GiteaBot
Copy link
Collaborator

@tpokorra I noticed you've updated the locales for non-English languages. These will be overwritten during the sync from our translation tool Crowdin. If you'd like to contribute your translations, please visit https://crowdin.com/project/gitea. Please revert the changes done on these files. 🍵

@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Aug 19, 2025
@github-actions github-actions bot added modifies/translation modifies/go Pull requests that update Go code modifies/templates This PR modifies the template files modifies/dependencies labels Aug 19, 2025
@tpokorra tpokorra force-pushed the TP-20250818-export_issues_to_excel branch 2 times, most recently from d38901f to 1d87056 Compare August 19, 2025 14:57
@tpokorra tpokorra changed the title add button to export issues to Excel; related to #29906 add button to export issues to Excel Aug 19, 2025
@silverwind
Copy link
Member

silverwind commented Aug 19, 2025

The button is far too prominent and there should only ever be one primary button on a page indicating the most likely action the user will take.

I recommend making a right-aligned "..." button on the right side of "Sort" and putting it in there. Check repo frontpage on how to implement this "..." menu using tippy.js.

@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 21, 2025
@hiifong hiifong self-requested a review August 21, 2025 01:17
@GiteaBot GiteaBot added lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. and removed lgtm/need 1 This PR needs approval from one additional maintainer to be merged. labels Aug 21, 2025
@tpokorra
Copy link
Author

tpokorra commented Oct 7, 2025

I recommend making a right-aligned "..." button on the right side of "Sort" and putting it in there. Check repo frontpage on how to implement this "..." menu using tippy.js.

Thanks for the hint @silverwind

I guess you are refering to initClonePanelButton in https://github.com/go-gitea/gitea/blob/main/web_src/js/features/repo-common.ts#L133 which uses Tippy.
and https://github.com/go-gitea/gitea/blob/main/templates/repo/clone_panel.tmpl#L6

I tried to find the place where I should add that new button.
This is where the sort dropdowns are defined: https://github.com/go-gitea/gitea/blob/main/templates/repo/issue/filter_list.tmpl#L121
this is included here: https://github.com/go-gitea/gitea/blob/main/templates/repo/issue/filters.tmpl#L22

I will have to continue trying to figure this out in the coming days.

@tpokorra
Copy link
Author

I finally got the tippy button to work. I still have a small css issue, but I will fix that.

One question: Now I wonder, if the "Export" button should actually be in a ... button right of the filters, or if it should be displayed with the other actions (close, delete, etc), once one or multiple issues are selected?

@tpokorra tpokorra force-pushed the TP-20250818-export_issues_to_excel branch from 2018f3d to 7212c2d Compare November 18, 2025 12:34
@tpokorra
Copy link
Author

The tippy.js dropdown now works beside the filter.

I also implemented the button with the actions, but I don't filter for the selected issues yet. If I would export only the selected issues, I can only export the currently displayed issues. That is probably not what we want.

So I tend to drop the export button from the Actions, and just keep the tippy dropdown beside the filter.

What do you think, @silverwind ?

@tpokorra tpokorra force-pushed the TP-20250818-export_issues_to_excel branch from 3d9b665 to 93dad9f Compare November 25, 2025 13:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. modifies/dependencies modifies/frontend modifies/go Pull requests that update Go code modifies/templates This PR modifies the template files modifies/translation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Print/Export List of Issues

4 participants