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

Donation progress bar UI/UX #54156

Closed
lasjorg opened this issue Mar 20, 2024 · 6 comments · Fixed by #54377
Closed

Donation progress bar UI/UX #54156

lasjorg opened this issue Mar 20, 2024 · 6 comments · Fixed by #54377
Labels
help wanted Open for all. You do not need permission to work on these. platform: learn UI side of the client application that needs familiarity with React, Gatsby etc. scope: UI Threads for addressing UX changes and improvements to user interface

Comments

@lasjorg
Copy link
Contributor

lasjorg commented Mar 20, 2024

Describe the Issue

The progress bar animation gives off a stuck-loading vibe. I'd suggest not animating the progress and just filling it in.

As mentioned in the forum thread this can be labeled better as well. E.g. "Progress toward donation goal" or whatever.

Affected Page

https://www.freecodecamp.org/learn

Steps to Reproduce

  1. Go to '...'
  2. Click on '...'
  3. Scroll down to '...'
  4. See error

Expected behavior

Less potentially confusing UX/UI

Screenshots

progress-bar

System

  • Device: [e.g. iPhone 6, Laptop]
  • OS: [e.g. iOS 14, Windows 10, Ubuntu 20.04]
  • Browser: [e.g. Chrome, Safari]
  • Version: [e.g. 22]

Additional context

Forum: https://forum.freecodecamp.org/t/freecodecamp-wont-load/680791

@lasjorg lasjorg added scope: UI Threads for addressing UX changes and improvements to user interface platform: learn UI side of the client application that needs familiarity with React, Gatsby etc. status: waiting triage This issue needs help from moderators and users to reproduce and confirm its validity and fix. labels Mar 20, 2024
@lasjorg lasjorg changed the title UX for donation progress bar Donation progress bar UI/UX Mar 20, 2024
@imonish8
Copy link

I guess that is progress of how much they have made so far to reach 20,000 monthly supporters. But I would prefer a circle or 360 pie graph to show the progress

That must have confuse supporters and supporter must have just stopped donations too.

Can I make one for it?

image

@lasjorg
Copy link
Contributor Author

lasjorg commented Mar 20, 2024

@imonish8 The issue has not been triaged yet so I would wait.

My issue is not with the use of a progress bar but what an animated progress bar represents and how it is interpreted. Users may associate it with a loading state.

The block would work better with a heading as well.

@bbsmooth
Copy link
Contributor

I agree that a heading would be a good addition. I'm not exactly sure what that heading should read, but it should help explain what the percentage is for, and it should be an h2. The percentage is currently in an h3, but it's not acting as a heading, so it should be in a p element, and we might need to add some visually hidden text after the percenrage to help screen reader users understand what it is for (depending on the heading text).

Also, can we get rid of the hr between the text and the Donate button? You can barely see it, and I'm not sure what purpose it serves, and it's just extra unnecessary noise that screen reader users have to listen to.

@Vijaykv5
Copy link

Vijaykv5 commented Apr 7, 2024

I would love to work on this if the issue has been triaged

@Sembauke Sembauke added help wanted Open for all. You do not need permission to work on these. and removed status: waiting triage This issue needs help from moderators and users to reproduce and confirm its validity and fix. labels Apr 12, 2024
@naomi-lgbt
Copy link
Member

I think the things bbsmooth called out are good, but I think the animation should stay.

It's meant to look like a loading state, because it represents the progress we have made toward our donation goal. I believe with better callouts in the text around that component we may be able to convey that properly?

@lasjorg
Copy link
Contributor Author

lasjorg commented Apr 17, 2024

Just to be clear, this is also based on user feedback. The user thought the page was stuck loading. Not saying that the feedback from a single user is a great metric, but I do understand why someone might think that.

If the progress bar was filling in, having that type of animation would be fine. The fact that it stays at the same progress for as long as it does (the user will likely never see it move in real-time) makes the animation more ambiguous.

As I said, it would work better with a heading. But if we need to "label" a UI element to explain what its purpose is, then there might also be an issue with the UI element semantics.


Personally, I'm also not particularly crazy about the look/style. Maybe just a simple gradient, going from red/orange to teal/green might work.

Simple example:
progress-bar

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Open for all. You do not need permission to work on these. platform: learn UI side of the client application that needs familiarity with React, Gatsby etc. scope: UI Threads for addressing UX changes and improvements to user interface
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants