Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Progress indicator confused for radio bottons (interactive to progress) #8104

Closed
meirachefitz opened this issue Mar 16, 2021 · 9 comments · Fixed by #10291
Closed

Progress indicator confused for radio bottons (interactive to progress) #8104

meirachefitz opened this issue Mar 16, 2021 · 9 comments · Fixed by #10291
Assignees

Comments

@meirachefitz
Copy link

Environment

Operating system
Any
Browser
Any

Within a service in IBM Cloud Platform

Detailed description

What version of the Carbon Design System are you using?
latest
What did you expect to happen?
Users expected that if they click on an "empty circle", it would move them to the next step of the process.
What happened instead?
Only after they noticed the "radio button" was not clickable did the users see the bottom right button to proceed to the next step.

What WCAG 2.1 checkpoint does the issue violate?
I could not map this to a specific item in WCAG 2.1

Steps to reproduce the issue

Dependent upon access to the service.

Please create a reduced test case in CodeSandbox

Additional information

Screen Shot 2021-03-16 at 2 40 40 PM
This is where we are seeing the issue

@tw15egan
Copy link
Collaborator

cc @carbon-design-system/design

@laurenmrice
Copy link
Member

We plan to update the icons in this component for this reason. It will look something like this but we will post final design specs once we come to a conclusion with our icon indicator guidance that we are working on right now. 👍

Artboard

@tw15egan tw15egan self-assigned this Mar 18, 2021
@tw15egan
Copy link
Collaborator

tw15egan commented Jul 6, 2021

@laurenmrice has there been any more work on this?

@laurenmrice
Copy link
Member

@tw15egan Not yet. Once we get some dedicated design time for this we will update the spec. 👍

@tymekg
Copy link

tymekg commented Jul 27, 2021

Hi, not sure if it's worth a separate issue, so commenting here:
@laurenmrice The blue bar on top looks confusing. It does not reinforce the message about progress but adds ambiguity instead. This occurs both in your new design pasted above and in the current implementation (we are using v10.31):
image
I think the bar is stretched to far, it makes impression that it's almost done with the Running step, while actually the step Pending is still in progress.

@thyhmdo
Copy link
Member

thyhmdo commented Nov 19, 2021

We plan to update the icons in this component for this reason. It will look something like this but we will post final design specs once we come to a conclusion with our icon indicator guidance that we are working on right now. 👍

Artboard

This is correct according to our guidance. Except that we have blue for the Not started. And seems like we don't have that icon in either our Icon page or Sketch. We also don't have a specific icon for disabled, but the description says that it is used for upstarted tasks or disabled processes, I think it makes sense.

image

@thyhmdo
Copy link
Member

thyhmdo commented Dec 13, 2021

@abbeyhrt the final spec (updated again, we decided to go back to dark gray for Not started)
progress-indicator-spec

@abbeyhrt
Copy link
Contributor

abbeyhrt commented Dec 14, 2021

@thyhmdo thank you for the final spec! @laurenmrice or Thy, do you all know when the "Not started" icon will be available?

@laurenmrice
Copy link
Member

@abbeyhrt The "Not started" icon is available. The icon name is just circle-dash https://carbon-elements.netlify.app/icons/examples/preview/#CircleDash16

@kodiakhq kodiakhq bot closed this as completed in #10291 Dec 15, 2021
Repository owner moved this from ⏱ Backlog to ✅ Done in Design System Dec 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

6 participants