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

Questionnaire: make Summary page display more intuitive #350

Closed

Conversation

xcaro
Copy link

@xcaro xcaro commented Jul 12, 2021

E.g. In this questionnaire, they are 151 questionnaire responses, but 198 question responses. As only complete questionnaire responses are analyzed (the data is exported and cleaned), showing more question responses than questionnaire responses in the summary is confusing.

E.g. here's a view of a recent case:
image

If we could make the view selectable, this would clean up the summary and make a bit more sense.

  • Drop-down: "All responses" [198 in this case]; "Full submissions" [151 in this case]; "Responses not submitted" [47],
  • Consistent language to delineate between question-specific responses and full/complete questionnaire submissions.
  • Correct the 'total' and percentages on the visual graphs depending on the view.

E.g. here's a view after improvement
image

@xcaro
Copy link
Author

xcaro commented Jul 12, 2021

Hi @mchurchward ,

We are from the Open University. Could you please help me to review this PR?

Thanks,

@xcaro xcaro changed the title Improment: make Summary page display more intuitive Questionnaire: make Summary page display more intuitive Jul 12, 2021
@mchurchward mchurchward force-pushed the main branch 3 times, most recently from eef1768 to 2a35b36 Compare October 24, 2022 20:41
@mchurchward mchurchward changed the base branch from main to MOODLE_400_STABLE January 3, 2023 16:08
@toanlamt
Copy link
Contributor

Hi @rezeau
I will take responsibility for this issue. I have created a new pull request #589 and rebased it with the MOODLE_401_STABLE branch. Could you please review it again?

Many thanks

@mchurchward mchurchward changed the base branch from MOODLE_400_STABLE to MOODLE_401_STABLE August 27, 2024 21:19
@mchurchward
Copy link
Contributor

Closing since it has been replaced with #589

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

Successfully merging this pull request may close these issues.

3 participants