oppia-android icon indicating copy to clipboard operation
oppia-android copied to clipboard

[Feature Request]: Disable the submit button until an answer option is selected or an answer is typed

Open seanlip opened this issue 2 years ago • 18 comments

Is your feature request related to a problem? Please describe.

We've received some learner feedback that:

  • The student did not understand that she is expected to tap on an answer option before she can proceed.
  • The student did not understand that she is expected to input an answer in the input field. She was confused about not seeing the Continue button like in the previous cards.

Describe the solution you'd like

We plan to disable the Submit button until an answer/response is selected and provide "help text" to instruct the learner on what to do. Once the learner modifies the answer field, the Submit button should become active again.

Please see the Mock up and the Figma file created by @Himali07, as well as this design thread comment for all the error messages.

This issue can be collaborated on by multiple contributors, by updating one interaction at a time. Here is the list of interactions:

  • [ ] Number with units (e.g. 4 km)

  • [x] Fraction input @masclot in https://github.com/oppia/oppia-android/pull/5224

  • [x] Ratio input @theMr17 in https://github.com/oppia/oppia-android/pull/5263

  • [x] Checkbox input @theMr17 https://github.com/oppia/oppia-android/pull/5319

  • [x] Multiple choice input @theMr17 https://github.com/oppia/oppia-android/pull/5319

  • [x] Math equation (e.g. 2x + 3y = 5) @Vishwajith-Shettigar https://github.com/oppia/oppia-android/pull/5317

  • [x] Algebraic expression (e.g. 2x + 3y) @Vishwajith-Shettigar https://github.com/oppia/oppia-android/pull/5317

  • [x] Number input @theMr17 https://github.com/oppia/oppia-android/pull/5310

  • [x] Numeric expression (e.g. 5 + 2) @Vishwajith-Shettigar https://github.com/oppia/oppia-android/pull/5317

  • [x] Text input @Vishwajith-Shettigar https://github.com/oppia/oppia-android/pull/5311

  • [x] Click on image @Vishwajith-Shettigar https://github.com/oppia/oppia-android/pull/5316

  • [x] Drag-and-drop sort @theMr17 https://github.com/oppia/oppia-android/pull/5323

seanlip avatar Jun 20 '23 14:06 seanlip

can I work on this issue?

adityabarodiya avatar Jul 08 '23 14:07 adityabarodiya

can I work on this issue?

Hi @adityabarodiya, thanks for your interest in contributing at oppia-android. This issue requires a bit of context about interactions in the app, and I would suggest you start with a good first issue to farmiliarize. Could you please look at this: #3930

adhiamboperes avatar Jul 08 '23 19:07 adhiamboperes

I think we need clarity from the product team here. The current the app already keeps the submit button disabled until an answer is selected. Should we be enabling it and then showing the requested errors for each interaction type if the submit button is clicked without an answer being entered?

BenHenning avatar Aug 29 '23 17:08 BenHenning

@seanlip could you please provide context?

BenHenning avatar Aug 29 '23 17:08 BenHenning

Yup, the plan is to enable the button and show the requested errors for each interaction type if the submit button is clicked without an answer being entered. However, I'm going to adjust the impact of this to "low" since it looks like the button is already disabled until an answer is selected (so this is not as big an issue as it seems to have been before).

I think in the longer run, it would be good for this to match the behaviour on Web: https://github.com/oppia/oppia/pull/18379. The reason for this is that it is helpful to show a message to instruct the learner on what to do, but we can't do this when clicking on a disabled button for accessibility reasons (see https://github.com/oppia/design-team/issues/71#issuecomment-1570552877). So this seemed like the best option.

seanlip avatar Sep 03 '23 14:09 seanlip

I created a draft PR #5197 for a single interaction: fraction interaction. It contains a demo with the UI changes. Please review and let me know if the direction is good; I will then finalize the PR with tests. Once this PR is approved, I can also work on other interaction view models in separate PRs.

masclot avatar Oct 17 '23 14:10 masclot

Continuing with Number input.

theMr17 avatar Jan 17 '24 10:01 theMr17

I would like to work on Text input.

Vishwajith-Shettigar avatar Jan 17 '24 19:01 Vishwajith-Shettigar

Continuing with Selection Input which includes both Checkbox input and Multiple choice input.

theMr17 avatar Jan 18 '24 07:01 theMr17

Working on Click on image.

Vishwajith-Shettigar avatar Jan 20 '24 03:01 Vishwajith-Shettigar

Working on MathExpressionInteractionsView, which includes Math equation, Algebraic expression, Numeric expression.

Vishwajith-Shettigar avatar Jan 21 '24 15:01 Vishwajith-Shettigar

Continuing with Drag-and-drop sort.

theMr17 avatar Jan 23 '24 04:01 theMr17

Going with Number with units

Vishwajith-Shettigar avatar Jan 23 '24 05:01 Vishwajith-Shettigar

Can anybody help me find the topic where Number with units is discussed in the app?

Vishwajith-Shettigar avatar Jan 29 '24 14:01 Vishwajith-Shettigar

Can anybody help me find the topic where Number with units is discussed in the app?

Hi @Vishwajith-Shettigar, I did a bit of searching in the codebase. I believe NumberWithUnitsInteractionView is not yet implemented in the app as per issue https://github.com/oppia/oppia-android/issues/30.

theMr17 avatar Jan 29 '24 17:01 theMr17

Hi @Vishwajith-Shettigar, I did a bit of searching in the codebase. I believe NumberWithUnitsInteractionView is not yet implemented in the app as per issue #30.

Great finding, thank you.

Vishwajith-Shettigar avatar Jan 30 '24 03:01 Vishwajith-Shettigar

@seanlip, FYI for product--this issue has been addressed with the exception of number with units, which we currently don't have in the app.

adhiamboperes avatar Mar 18 '24 06:03 adhiamboperes

This has been fully implemented except Number with units (e.g. 4 km) which we currently do not have in the beta assets.

Moving to backlog.

adhiamboperes avatar Apr 18 '24 07:04 adhiamboperes

Marking as complete because we don't have number with units yet.

adhiamboperes avatar Jul 01 '24 21:07 adhiamboperes