-
Notifications
You must be signed in to change notification settings - Fork 19
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
Small UI changes to Instruction Page Component #138
Conversation
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #138 +/- ##
==========================================
+ Coverage 69.99% 73.87% +3.88%
==========================================
Files 59 59
Lines 7632 7626 -6
Branches 448 801 +353
==========================================
+ Hits 5342 5634 +292
+ Misses 2272 1970 -302
- Partials 18 22 +4
Flags with carried forward coverage won't be shown. Click here to find out more.
☔ View full report in Codecov by Sentry. |
@@ -331,7 +331,7 @@ describe("Player for OMR quizzes", () => { | |||
cy.get('[data-test="questionSetInstruction-1"').should("exist"); | |||
cy.get('[data-test="questionSetInstruction-1"').should( | |||
"have.text", | |||
"Question Set 1 Description\nYou may attempt only up to 3 questions in this section." | |||
"Question Set 1 DescriptionYou may attempt only up to 3 questions in this section." |
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.
Shouldn't there be a :
or something between Description
and You
?
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 removed \n
and used <br>
in instruction component.
have.text
was not capturing the <br>
between "Description" and subsequent text
Why merged without PR? :( |
No description provided.