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

mySubmission: smoother ui (fixes #5091) #5095

Closed
wants to merge 1 commit into from

Conversation

Avinash-Codes
Copy link
Member

fixes #5091

@Avinash-Codes Avinash-Codes linked an issue Jan 22, 2025 that may be closed by this pull request
Copy link
Member

@pavi38 pavi38 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey Avinash, the mysubmission tab no longer shows the survey submission.

@Okuro3499
Copy link
Collaborator

Hey Avinash, the mysubmission tab no longer shows the survey submission.

@pavi38 the pr is not yet merged #5084

@pavi38
Copy link
Member

pavi38 commented Jan 22, 2025

@Okuro3499 Wouldn't this create a merge conflict then?

Comment on lines -64 to -83
showHideRadioButton()
}

private fun showHideRadioButton() {
if (type != "survey") {
fragmentMySubmissionBinding.rbExam.isChecked = true
fragmentMySubmissionBinding.rbExam.setOnCheckedChangeListener(this)
fragmentMySubmissionBinding.rbSurvey.setOnCheckedChangeListener(this)
} else {
fragmentMySubmissionBinding.rbSurvey.visibility = View.GONE
fragmentMySubmissionBinding.rbExam.visibility = View.GONE
}
}

override fun onCheckedChanged(compoundButton: CompoundButton, b: Boolean) {
type = if (fragmentMySubmissionBinding.rbSurvey.isChecked) {
"survey_submission"
} else {
"exam"
}
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

mysubmission fragment is reused by mySurvey removing this would cause anotther bug

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

mysurvey title
4 participants