-
Notifications
You must be signed in to change notification settings - Fork 479
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
Standards Report: Progress box tooltips #32589
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.
One question but overall looks good to me.
lessonNumber={lesson.lessonNumber} | ||
/> | ||
<span> | ||
<ReactTooltip |
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.
Could we move the ReactTooltip inside ProgressBoxForLessonNumber?
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.
Sounds good! I moved the tooltips and added lesson to a new standards shapes file.
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.
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.
Ya makes sense we wouldn't want them there. Good catch!
…ment in the interator will have a unique identifier and storybook will be happy
0dd29ad
to
befdbff
Compare
LP-962 && LP-1128
Spec
This adds tooltips to the progress boxes that will be displayed on the standards view of the progress tab. They include the lesson name, completion status and # of students out of the total # in the section who have completed the lesson.
And links the progress box to the lesson plan on curriculum.code.org
Right now, this is all fake data.