Skip to content

feat: component ProgressBar - add optional input property 'percentage_start'#1204

Merged
acstll merged 4 commits intotelekom:mainfrom
michael-koeller:feat/progress-animation-start
Sep 27, 2022
Merged

feat: component ProgressBar - add optional input property 'percentage_start'#1204
acstll merged 4 commits intotelekom:mainfrom
michael-koeller:feat/progress-animation-start

Conversation

@michael-koeller
Copy link
Copy Markdown
Contributor

Feature Request:

Property 'percentage_start' at component <scale-progress-bar>

Context

We are working on a Telekom-internal project. We just startet an new UI application for the TROPOS team, using this really great component library.

Use Case

We have a web page displaying a multi-step, partly-manual/partly-automated processing status. Our UI will keep polliing this page to reflect any server-side status changes. In this scenario it's quite annoying that the progress bar would alway restart it's animation from scratch.

Instead, we would rather set the previous percentage and thus only have nice animations on actual status changes.

Copy link
Copy Markdown
Collaborator

@acstll acstll left a comment

Choose a reason for hiding this comment

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

Thanks a lot @michael-koeller — sorry for the delay in the reply!

Comment thread packages/components/src/components/progress-bar/progress-bar.tsx Outdated
Copy link
Copy Markdown
Collaborator

@acstll acstll left a comment

Choose a reason for hiding this comment

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

I think you also need to update the snapshots (e.g. yarn test --spec --updateSnapshot in packages/components) — thanks again!

Comment thread packages/components/src/components/progress-bar/progress-bar.tsx Outdated
@acstll acstll merged commit 573baf3 into telekom:main Sep 27, 2022
@michael-koeller michael-koeller deleted the feat/progress-animation-start branch September 27, 2022 09:48
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.

2 participants