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
Assessments: add summary text component and story #22415
Conversation
return ( | ||
<div> | ||
<div> | ||
{`multiple choice questions overview ${numSubmissions}`} |
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.
nit: uppercase m
(and add to i18n)
@caleybrock Updated PR |
return ( | ||
<div> | ||
<div> | ||
{i18n.summaryAssessmentsOverview()} {`${numStudentSubmissions} / ${totalNumStudents}`} |
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.
you can do this without double {}:
{i18n.summaryAssessmentsOverview() + '${numStudentSubmissions} / ${totalNumStudents}'}
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.
@caleybrock the string concatenation did not display the text correctly. I reverted back to the string interpolation I had before. Thoughts..
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.
@caleybrock Dave helped out with refactoring code. All set.
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.
Just one style change, otherwise looks good!
This PR is to add a "SummaryText" component and story.
The summary text component displays the results of how many students submit an assessment. This component will be displayed above the overview assessment table.