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

Part warnings should be visible, but dismissable #511

Closed
christianp opened this issue Oct 19, 2017 · 2 comments

Comments

Projects
None yet
2 participants
@christianp
Copy link
Member

commented Oct 19, 2017

Some students don't think to mouse over the warning symbol, and it's not very accessible.
Instead, have the warning visible first, with a "dismiss" button. Clicking the button makes the warning go away but marks the part as dirty, so it has to be resubmitted.

@chrismgraham

This comment has been minimized.

Copy link
Contributor

commented Mar 1, 2018

Our recent engineering test has further emphasised that students don't read these warnings. I wonder if it's possible as an alternative/addition to the above, to bring up the "Can not submit answer - check for errors" warning, as you would get entering a fraction into a number entry part?

@christianp

This comment has been minimized.

Copy link
Member Author

commented Mar 1, 2018

The cases from the engineering test shouldn't have prevented submission - it's reasonable to use the wrong variable names, so allow submission.
Issue #345 would fix it - if we could only allow single-letter variable names, then we could prevent submission if the student uses a multi-letter variable name.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.