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.

Sign up for GitHub

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

Jump to bottom

Optimize task management page #702

Merged
merged 2 commits into from
Oct 25, 2024
Merged

Optimize task management page #702

merged 2 commits into from
Oct 25, 2024

Conversation

Enet4
Copy link
Collaborator

@Enet4 Enet4 commented Oct 24, 2024

Summary

  • Restrain task status view to showing the 200 most recent tasks, so that the page can still open fine on long-running Dicoogle instances without saturating the browser.
  • Show how many tasks are presented and the number of tasks not shown in the header above the task list.
  • Update task status style so that each task is more compact.

See screenshot.

Screenshot 2024-10-24 095303

Enet4 added 2 commits October 24, 2024 09:54
@Enet4
Restrain task status view to showing the 200 most recent tasks
40b4374 
- prevents webapp saturation from a very large number of tasks
- show count in header panel above the task status list
@Enet4
Update task status style
640501f 
- reduce spacing between properties
- present completed tasks information in the same row
tiberio-baptista
tiberio-baptista approved these changes Oct 25, 2024
View reviewed changes
Copy link
Collaborator

@tiberio-baptista tiberio-baptista left a comment

Choose a reason for hiding this comment

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

Tested this by running locally and spamming index tasks until the UI displayed 200 tasks.

The new visual is better and less bloated, and I've found with 200 tasks on-screen. Logs show no issues as well. LGTM!

@Enet4 Enet4 merged commit 43f67a4 into dev Oct 25, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tiberio-baptista tiberio-baptista tiberio-baptista approved these changes

@bastiao bastiao Awaiting requested review from bastiao

Assignees
No one assigned
Labels
enhancement webapp
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Enet4 @tiberio-baptista