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
Match Questions in Assessments Tab: Add Overview of class responses by question #28345
Conversation
apps/src/templates/sectionAssessments/MatchAssessmentsOverviewContainer.jsx
Show resolved
Hide resolved
apps/src/templates/sectionAssessments/MatchAssessmentsOverviewTable.jsx
Outdated
Show resolved
Hide resolved
apps/src/templates/sectionAssessments/MatchAssessmentsOverviewTable.jsx
Outdated
Show resolved
Hide resolved
apps/src/templates/sectionAssessments/MatchAssessmentsOverviewTable.story.jsx
Show resolved
Hide resolved
@@ -104,7 +106,8 @@ class SectionAssessments extends Component { | |||
|
|||
state = { | |||
freeResponseDetailDialogOpen: false, | |||
multipleChoiceDetailDialogOpen: false | |||
multipleChoiceDetailDialogOpen: false, | |||
matchDetailDialogOpen: false |
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.
Is there ever a scenario where these two dialogs are trying to be open at the same time?
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.
I don't think so. these open when you click on the see more for the question
apps/test/unit/templates/sectionAssessments/sectionAssessmentsReduxTest.js
Show resolved
Hide resolved
apps/test/unit/templates/sectionAssessments/sectionAssessmentsReduxTest.js
Outdated
Show resolved
Hide resolved
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 a few small clean-up suggestions. so much great work here; way to go, Dani!
…on-assessment-tab-part-3
…on-assessment-tab-part-3
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.
great work Dani!
import { | ||
matchQuestionWith2Pairs, | ||
matchQuestionWith4Pairs | ||
} from './assessmentsTestHelpers'; |
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 is a good example a test fixture, which we are trying to move toward using more often. thanks for adding this!
@@ -0,0 +1,107 @@ | |||
/* eslint-disable react/no-danger */ |
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 doesn't look needed. remove from here, plus wherever you copied it from?
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.
Will do! Going to fix on the part 4 branch and ship this PR
@@ -29,7 +29,7 @@ function calculateOpacity(answered) { | |||
return (answered + 10) / 100; | |||
} | |||
|
|||
class MultipleChoiceAnswerCell extends Component { | |||
class PercentAnsweredCell extends Component { |
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.
good name!
This is part 3 of Match Questions in Assessment Tab.
Part 1: #28343
Part 2: #28344
This adds a section with tables for each match question that breaks down how the class answered that question.
If you click on the "...see full question" you can view the whole question and the answers in a dialog window.
Hovering over a column header will show you the full text in a tooltip.