Skip to content

Fix: added time left to complete download in library - #1344

Open
i-am-space wants to merge 1 commit into
kiwix:mainfrom
i-am-space:clean-fix-issue-1328
Open

Fix: added time left to complete download in library#1344
i-am-space wants to merge 1 commit into
kiwix:mainfrom
i-am-space:clean-fix-issue-1328

Conversation

@i-am-space

@i-am-space i-am-space commented Feb 12, 2025

Copy link
Copy Markdown

Fixes #1328

Closed an old pull request that fixed two problems by accident.
Added a feature to count down time till download completion.
image

@kelson42

Copy link
Copy Markdown
Collaborator

@i-am-space Thank you for your PR, we will review it.

@kelson42
kelson42 force-pushed the clean-fix-issue-1328 branch 2 times, most recently from 62502be to deb6d61 Compare December 8, 2025 08:56

@kelson42 kelson42 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@i-am-space Sorry again for the very late review of this PR. The feature seems to me to work as expected, but I see two things to be fixed:

  • Adding at the bottom the time left makes the whole block not been centered vertically properly. See
Image
  • We have to display something like "10 s.", This should have a hint which says "10 s. left" where the "left" should be translatable. Don't forget the qqq language for the explanation.

@i-am-space Be careful, I have rebased your PR to get the latest version of our codebase with your commit on the top of it.

@kelson42
kelson42 force-pushed the clean-fix-issue-1328 branch from deb6d61 to 9b37054 Compare December 9, 2025 06:49
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.

Library should indicate download time remaining

2 participants