Skip to content
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

Submit button needs to be correctly activated/deactived in player #1344

Closed
1 of 2 tasks
rt4914 opened this issue Jun 17, 2020 · 0 comments · Fixed by #1443
Closed
1 of 2 tasks

Submit button needs to be correctly activated/deactived in player #1344

rt4914 opened this issue Jun 17, 2020 · 0 comments · Fixed by #1443
Assignees
Labels
Priority: Essential This work item must be completed for its milestone. Z-ibt Temporary label for Ben to keep track of issues he's triaged.

Comments

@rt4914
Copy link
Contributor

rt4914 commented Jun 17, 2020

Currently in Exploration Player and Question Player the Submit button is active by default but instead it should be correctly controlled as active/deactive.

Submit button should be active under following conditions only:

  1. Atleast some answer should be available for submission.
  2. There is should be no pending error from input interactions.

Other than this Submit button should be deactive in all cases.

Checklist:

  • Fix this in Exploration Player (@rt4914 )
  • Fix this in Question Player (@rt4914 )
@rt4914 rt4914 assigned BenHenning and rt4914 and unassigned BenHenning Jun 17, 2020
@rt4914 rt4914 added this to the Alpha milestone Jun 18, 2020
rt4914 added a commit that referenced this issue Jun 19, 2020
* submit button active/deactive [skip CI]

* ItemSelection button control

* Default button is inactive[skip CI]

* Submit button final changes

Co-authored-by: Rajat Talesra <[email protected]>
@BenHenning BenHenning added Priority: Essential This work item must be completed for its milestone. and removed Workstream: Lowfi UI labels Jun 23, 2020
rt4914 added a commit that referenced this issue Jul 6, 2020
* Submit button active/inactive

* Nit ktlint fix

Co-authored-by: Rajat Talesra <[email protected]>
@BenHenning BenHenning added the Z-ibt Temporary label for Ben to keep track of issues he's triaged. label Sep 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Priority: Essential This work item must be completed for its milestone. Z-ibt Temporary label for Ben to keep track of issues he's triaged.
Development

Successfully merging a pull request may close this issue.

3 participants