ic-ui-kit icon indicating copy to clipboard operation
ic-ui-kit copied to clipboard

Adding error variant to Ic-Stepper

Open GCHQ-Developer-837 opened this issue 10 months ago โ€ข 4 comments

Summary

Adding error and warning options for Ic-Stepper

๐Ÿ’ฌ Description

Currently Ic-Stepper only has blue for current and green for success. Adding options for error and warning to reflect the state of that page.

๐Ÿ’ฐ Use value

It will help give it more functionality and allow for users to clearly see if there are warnings or errors on certain pages. Increases the usability of steppers and users can clearly see which pages have issues. Users don't have to click through the pages to see issues, they can simply tell from the stepper that a certain page has issues.

๐Ÿ“ Acceptance Criteria

If a page has required fields for example and they haven't been filled but other boxes on that page perhaps have been filled, the stepper should indicate theres an error on that page. If a page has a reason to showing a yellow alert, then in the stepper that page icon should indicate warning.

โœ Designs

If there's a Figma design file (or other mock-up), include it here.

##โ€ฏ๐Ÿงพ Guidance If there's written guidance or documentation, include a link to it here.

Additional info

Tell us anything else useful to help us understand your suggestion.

GCHQ-Developer-837 avatar Mar 05 '25 14:03 GCHQ-Developer-837

  • Renamed Atoms/Step property to State for consistency
  • Added new states to affected atoms for warning and error, keeping style and colours consistent with elsewhere in design system

mi6-577 avatar Mar 24 '25 08:03 mi6-577

For reference, steppers with states exist in other design systems;

  • https://carbondesignsystem.com/components/progress-indicator/usage/
  • https://www.lightningdesignsystem.com/2e1ef8501/p/64d280-progress-indicator
  • https://cloudscape.design/components/steps/?tabId=playground

Stepper warning text isn't WCAG against page header colour, would need to change status/warning/default to change from Yellow/900 to Yellow/1000

Static chip contrast also doesn't meet WCAG against header background as a UI component ...could change to arch/500

mi6-577 avatar Apr 02 '25 07:04 mi6-577

  • Remove warning state, stepper is not intended to have partially completed steps
  • Error state is fine, but guidance would need to ensure page displays full error detail (eg against a form field missing mandatory info)

mi6-577 avatar Apr 02 '25 12:04 mi6-577

Guidance has been updated. This is ready to dev (code and guidance)

mi6-577 avatar Sep 12 '25 11:09 mi6-577