Skip to content
This repository has been archived by the owner on Jan 17, 2025. It is now read-only.

feat(process-monitoring): add a button to reset the use case #78

Merged
merged 2 commits into from
Mar 24, 2023

Conversation

tbouffard
Copy link
Member

@tbouffard tbouffard commented Mar 23, 2023

This reset the whole "happy path" animation.
Use the same colors as for the close button of the "advertise" section. Make this button use the regular 'btn' style as done for all buttons of the demo.

closes #73

Demo

pr-78_process-monitoring_reset_button.mp4

Button design

Make it less visible, because this button is a convenience. I don't want it to draw the user's attention.

1st implementation new implementation
image image

@tbouffard tbouffard added depends on another PR ⚠️ Pull request depending on another one. The depending must be merged first enhancement New feature or request labels Mar 23, 2023
@github-actions
Copy link

github-actions bot commented Mar 23, 2023

♻️ PR Preview 3f52f8e has been successfully destroyed since this PR has been closed.

🤖 By surge-preview

@tbouffard tbouffard requested a review from assynour March 23, 2023 16:50
@tbouffard tbouffard changed the title refactor(process-monitoring): put the whole code in a class feat(process-monitoring): add a button to reset the use case Mar 24, 2023
Copy link
Contributor

@assynour assynour left a comment

Choose a reason for hiding this comment

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

Liked it!

This reset the whole "happy path" animation.
Use the same colors as for the close button of the "advertise" section. Make this button use the
regular 'btn' style as done for all buttons of the demo.
@tbouffard tbouffard force-pushed the feat/73-reset_process_monitoring branch from 6c5bee7 to 3f52f8e Compare March 24, 2023 08:20
@tbouffard tbouffard removed the depends on another PR ⚠️ Pull request depending on another one. The depending must be merged first label Mar 24, 2023
@tbouffard tbouffard marked this pull request as ready for review March 24, 2023 08:21
@tbouffard tbouffard merged commit 9d185ea into main Mar 24, 2023
@tbouffard tbouffard deleted the feat/73-reset_process_monitoring branch March 24, 2023 08:23
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FEAT] process-monitoring: let restart the animation
2 participants