Skip to content

Show hours in queue duration#207

Merged
marcpope merged 3 commits intomarcpope:mainfrom
c0dr1ver:fix/queue-duration-hours
Apr 28, 2026
Merged

Show hours in queue duration#207
marcpope merged 3 commits intomarcpope:mainfrom
c0dr1ver:fix/queue-duration-hours

Conversation

@c0dr1ver
Copy link
Copy Markdown
Contributor

Summary

  • Format queue durations of one hour or longer as Hh Mm Ss in the initial PHP render.
  • Apply the same hour-aware formatting in the /queue auto-refresh JavaScript renderer.

Verification

  • php -l src/Views/queue/index.php
  • Checked sample JS output: 59 => 59s, 60 => 1m 0s, 3800 => 1h 3m 20s

Notes

  • This PR targets main and only changes src/Views/queue/index.php.

Testing

  • Tested on Docker
  • Tested on bare metal
  • Agent changes tested on Linux
  • Agent changes tested on Windows

@c0dr1ver c0dr1ver force-pushed the fix/queue-duration-hours branch from ff8f602 to 5b0bc75 Compare April 26, 2026 21:03
@c0dr1ver
Copy link
Copy Markdown
Contributor Author

Hi @marcpope

this PR includes one fix along with two alternative approaches for redesigning the Queue duration visualization.

Would appreciate if you could take a closer look at both variants - in my view, this approach makes the UI more intuitive, as it immediately highlights the “heaviest” jobs and improves overall readability.

Happy to adjust based on your preference 👍

@c0dr1ver
Copy link
Copy Markdown
Contributor Author

image

@MegaV0lt
Copy link
Copy Markdown
Contributor

The ideas of showing the bars for the time is great.

I am not sure if 'Red' is a good choise, because it is mainly associated with error or fail. Maybe a darker yellow.
Also the badge for the time value should be dropped to habe an consistent look (Same look as in 'Server Healt')

@marcpope
Copy link
Copy Markdown
Owner

I like this idea, but agree on red/ yellow. I'd probably make them all the same color? Only Red if it wasn't completed.

@c0dr1ver
Copy link
Copy Markdown
Contributor Author

@MegaV0lt @marcpope no-nightmare mode is on ?

Untitled

p.s. to be honest I have some questions to the accounter of "Avg duration" too )

@MegaV0lt
Copy link
Copy Markdown
Contributor

Great, but two things:

  • The badge is not consistent (Cyan instead of blue))
  • Time value have an extra background badge (should be dropped for consistency)
Auswahl_643

@marcpope marcpope merged commit 936f1d5 into marcpope:main Apr 28, 2026
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.

3 participants