-
Notifications
You must be signed in to change notification settings - Fork 40
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
New: Added support for _canShowCorrectness #224
base: master
Are you sure you want to change the base?
Conversation
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.
This works as expected thanks. Please see comment raised regarding consistency of selector naming/nesting.
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.
This works as expected thanks
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.
All working and much nicer than toggle buttons 👍
Needs schemas, readme and example.json for |
This comment was marked as resolved.
This comment was marked as resolved.
...change isn't specific to _canShowModelAnswer functionality and will also apply to other question components
part of adaptlearning/adapt_framework#3597
Allows option
_canShowCorrectness
instead of_canShowModelAnswer
, to replace the associated toggle button and display correctness directly on the component.New
requires
ref adaptlearning/adapt-contrib-core#582
ref adaptlearning/adapt-contrib-vanilla#527