-
Notifications
You must be signed in to change notification settings - Fork 0
Description
Component
ProgressBar
Is this a regression?
None
Describe the bug
The ProgressBar component is currently implemented with a restrictive error prop name that only supports a binary failure state. This limits the component to only error and ignores the need for warning and rest states.
Limiting the state prop to only error mistakes a process failing for the only nuance worth communicating. In the context of a progress bar, users need to know if a process is succeeding with caveats (Warning) or is completely safe (Rest).
This issue aims to refactor and change the error prop to status to match the flexibility of the design system's HelperText.
To Reproduce
- Create an instance of the
ProgressBarcomponent. - Try to configure the progress bar to show a
Warninghelper textto indicate a stable but slow connection or a non-critical issue. - Observe that the current prop name (e.g., error) does not accept "warning" as a value.
Expected behavior
The prop controlling the state should be renamed/refactored to status accepting rest | error | warning.
warning: Renders the progress bar withwarningHelper textto indicate the process is continuing with caveats.error: Renders the progress bar in Red to indicate failure.rest: Renders the standard color to indicate the process is safe.
Screenshots
No response
Package version
v3.0.0
Desktop
. OS:
. Browser & version:
. Additional Information:
Smartphone
. Device:
. OS:
. Browser & version:
. Additional Information:
Exception or Error
Additional context
No response