-
Notifications
You must be signed in to change notification settings - Fork 521
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
Fixes 1432: Fix prototype exploration answer #1469
Conversation
@@ -705,7 +705,7 @@ class StateFragmentTest { | |||
onView(withId(R.id.submit_answer_button)).perform(click()) | |||
onView(withId(R.id.continue_navigation_button)).perform(click()) | |||
|
|||
// Seventh state: Drag Drop Sort. Correct answer: Move 1st item to 3rd position. | |||
// Seventh state: Drag Drop Sort. Correct answer: Move 1st item to 4rd position. |
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.
4th
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.
Thanks for the quick fix @aggarwalpulkit596! LGTM minus some nits.
@@ -849,7 +849,7 @@ | |||
}, | |||
"content": { | |||
"content_id": "content", | |||
"html": "<p>Sort the following in ascending order, putting equal items in the same position.</p>" | |||
"html": "<p>Sort the following in descending order, putting equal items in the same position.</p>" |
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 think that the first drag & drop state also says 'putting equal items in the same position' but grouping isn't enabled for it. Can you please fix the text accordingly?
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.
done
@@ -705,7 +705,7 @@ class StateFragmentTest { | |||
onView(withId(R.id.submit_answer_button)).perform(click()) | |||
onView(withId(R.id.continue_navigation_button)).perform(click()) | |||
|
|||
// Seventh state: Drag Drop Sort. Correct answer: Move 1st item to 3rd position. | |||
// Seventh state: Drag Drop Sort. Correct answer: Move 1st item to 4rd position. |
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 added another action here (e.g. you seem to be moving two things here); please clarify in the comment.
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.
done
Explanation
Fixes #1432
Checklist