Skip to content

[Bug]: Refactor ProgressBar to replace binary error prop with variant (rest | warning | error) #359

@AregSoft

Description

@AregSoft

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

  1. Create an instance of the ProgressBar component.
  2. Try to configure the progress bar to show a Warning helper text to indicate a stable but slow connection or a non-critical issue.
  3. 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 with warning Helper text to 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

Metadata

Metadata

Assignees

Labels

phase / in-progressThis issue is in the development processtype / bugSomething isn't working

Type

No type

Projects

No projects

Relationships

None yet

Development

No branches or pull requests

Issue actions