Skip to content

feat(offline_download): add task progress list#411

Open
zwzzrl wants to merge 13 commits intoOpenListTeam:mainfrom
zwzzrl:main
Open

feat(offline_download): add task progress list#411
zwzzrl wants to merge 13 commits intoOpenListTeam:mainfrom
zwzzrl:main

Conversation

@zwzzrl
Copy link

@zwzzrl zwzzrl commented Mar 10, 2026

Description / 描述

为工具栏的离线下载页面添加了任务列表,使得添加完离线下载任务后可以看到任务状态。

Motivation and Context / 背景

方便添加完离线下载任务后看到任务状态,不用打开后台。
屏幕截图 2026-03-10 223936
屏幕截图 2026-03-10 224009

Closes #410

How Has This Been Tested? / 测试

添加离线下载任务列表

Checklist / 检查清单

  • I have read the CONTRIBUTING document.
    我已阅读 CONTRIBUTING 文档。
  • I have formatted my code with go fmt or prettier.
    我已使用 go fmtprettier 格式化提交的代码。
  • I have added appropriate labels to this PR (or mentioned needed labels in the description if lacking permissions).
    我已为此 PR 添加了适当的标签(如无权限或需要的标签不存在,请在描述中说明,管理员将后续处理)。
  • I have requested review from relevant code authors using the "Request review" feature when applicable.
    我已在适当情况下使用"Request review"功能请求相关代码作者进行审查。
  • I have updated the repository accordingly (If it’s needed).
    我已相应更新了相关仓库(若适用)。

zwzzrl and others added 6 commits March 10, 2026 22:21
@xrgzs xrgzs changed the title feat(offline_download):添加离线下载任务列表功能 feat(offline_download): add task list Mar 11, 2026
@xrgzs xrgzs changed the title feat(offline_download): add task list feat(offline_download): add task progress list Mar 11, 2026
@xrgzs xrgzs requested a review from Copilot March 11, 2026 02:08
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds an in-modal task progress list to the Home toolbar “Offline Download” flow so users can see ongoing task status immediately after submitting tasks (addresses #410).

Changes:

  • Extend the offline download modal to optionally show a periodically refreshed task list.
  • Add a small tasks fetch/store module and a TaskProgress UI component to render task state/progress.
  • Extend ModalInput with onSubmitWithValue to allow submit handlers to clear/reset the input value after success.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 9 comments.

Show a summary per file
File Description
src/pages/home/toolbar/OfflineDownload.tsx Shows task list in the modal and polls backend for task updates
src/pages/home/offlinedownload/task.ts Implements task fetching + merged list state (old/new endpoints)
src/pages/home/offlinedownload/TaskProgress.tsx Renders individual task progress cards with parsed metadata/links
src/components/ModalInput.tsx Adds onSubmitWithValue callback to expose setValue to submit handlers
src/lang/en/tasks.json Adds i18n strings for the offline download task list UI

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

Copy link
Member

@xrgzs xrgzs left a comment

Choose a reason for hiding this comment

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

需要清除重复代码、将文件移动到正确位置

@zwzzrl zwzzrl requested a review from xrgzs March 11, 2026 07:37
xrgzs added 2 commits March 11, 2026 16:36
Signed-off-by: MadDogOwner <xiaoran@xrgzs.top>
Signed-off-by: MadDogOwner <xiaoran@xrgzs.top>
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated 5 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

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.

[Feature]添加离线下载任务列表功能

3 participants