-
Notifications
You must be signed in to change notification settings - Fork 522
[AITT-1252] Add progress bar to the personalization quiz #68945
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR implements a progress bar for the personalization quiz to provide users with visual feedback on their completion progress. The progress bar displays the current question number, total questions, and percentage complete.
- Adds a new
PersonalizationProgressBarcomponent with question tracking and percentage calculation - Updates layout styling to accommodate the progress bar at the top of the quiz
- Integrates the progress bar into the main quiz container with conditional rendering
Reviewed Changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| PersonalizationProgressBar.tsx | New component implementing progress bar with question count and percentage display |
| PersonalizationCollectorContainer.tsx | Integrates progress bar component and updates layout structure |
| personalization-information.module.scss | Adds styling for progress bar container and updates page layout |
| personalization_information.html.haml | Removes center justification to accommodate new layout |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
apps/src/aiDifferentiation/personalization/PersonalizationProgressBar.tsx
Show resolved
Hide resolved
apps/src/aiDifferentiation/personalization/PersonalizationProgressBar.tsx
Outdated
Show resolved
Hide resolved
| import style from './personalization-information.module.scss'; | ||
|
|
||
| type PersonalizationProgressBarProps = { | ||
| /** Current question number */ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure if these comments are needed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, this is a nice to have, not a need to have. This is intentional documentation for easier support in the future. While this is more straight forward example, I still try to add docs as a rule so that I don't forget to add it where it's less straight forward.
|
LGTM! |
|
@MollyPeredo @kobryan0619 - tapping you in as well |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
|
This looks great. Love how smooth the animation is! |
[AITT-1252] Add progress bar to the personalization quiz
2025-10-20.16.57.09.mov
Links
Testing story
Deployment strategy
Follow-up work
Privacy
Security
Caching
PR Creation Checklist: