[FEATURE] BackgroundTask: replace legacy inline progress bar - #11850
Draft
thibsy wants to merge 2 commits into
Draft
[FEATURE] BackgroundTask: replace legacy inline progress bar#11850thibsy wants to merge 2 commits into
thibsy wants to merge 2 commits into
Conversation
thibsy
force-pushed
the
feature/12/bt-progress-bar
branch
from
August 4, 2026 12:23
81e95e5 to
83466ea
Compare
thibsy
marked this pull request as draft
August 4, 2026 12:27
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Hi @thojou,
This was my rather desperate attempt of replacing the inline legacy progress bar with the new UI component (
Progress\Bar). However, I could not manage to replace this yet, because funding dried out and some issues came to light.The main issue is that the whole process of how these notification items are updated is not compatible with the progress bar anymore. Until now, the entire notification item was updated async every X seconds, which delivered an updated version of the legacy progress bar. This is not possible anymore, because the new progress bar can only be updated on the client via async requests, and since they are part of the notification item which should be replaced entirely, this would break the progress of the progress bar. I managed to work around this for "normal" tasks, but I did not find a solution for the "user interaction" ones. But this was mostly due to the fact that – for the life of me – I could not manage to implement a working example with a user interaction.
The PR also contains a possible approach for registering update listeners to the new progress bar, which would allow to trigger a replacement request for the "user interaction" tasks once they are done, to fully either replace surroundings of the progress bar. But I could not look into this anymore.
So next steps would be to check if the approach using update listeners is feasible to replace the "update the notification every X seconds" concept.
Since this cannot be merged, feel free to just close this PR.
Kind regards,
@thibsy