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

Include progress bar to form #85

Merged
merged 1 commit into from
May 15, 2024
Merged

Conversation

JuliaCasamitjana
Copy link
Collaborator

@JuliaCasamitjana JuliaCasamitjana commented May 10, 2024

Description

To give users a visual reference of the remaining questions we now show a progress bar to the top of the component.

Screenshot 2024-05-10 at 13 17 19

Decisions / Choices I made

  • In contrast with the progress label (1 of 3), which indicates the question the user is currently seeing, the progress bar indicates the questions the user has already resolved. Therefore the bar will update accordingly after the user submits an answer.
  • It uses the progress tag
  • 15px of bar are always shown to make the progress bar visible from the first moment.

Checklist

  • All checks pass successfully
  • All related commits are squashed together
  • Appropriate labels are added to this pull request

@JuliaCasamitjana JuliaCasamitjana added the enhancement New feature or request label May 10, 2024
@JuliaCasamitjana JuliaCasamitjana self-assigned this May 10, 2024
To give users a visual reference of the remaining questions we
now show a progress bar to the top of the component.

Closes #67
@JuliaCasamitjana JuliaCasamitjana force-pushed the jc/include_progress_bar branch from d3d56fe to abc053b Compare May 14, 2024 14:08
@JuliaCasamitjana JuliaCasamitjana merged commit d5b1886 into main May 15, 2024
4 checks passed
@JuliaCasamitjana JuliaCasamitjana deleted the jc/include_progress_bar branch May 15, 2024 12:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants