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

Make the "Show Problem Grader" checkbox a button instead. #2395

Merged

Conversation

drgrice1
Copy link
Sponsor Member

The button is below the other usual problem/test submit buttons. When clicked problem graders are shown, and the button becomes the "Hide Problem Grader" button. Obviously when that is clicked the problem graders are hidden.

For now nothing is done with the check boxes for showing the various PG debugging information elements. In thinking about this further, there really isn't a better way of handling those with the current structure. Those are only shown for admin users (by default) so I don't think this is something that will be changed for this release. I think that these should be removed from the problem page entirely, and instead only shown (for those with the permission to see them) on the PG editor page. The PG code for handling these things should be completely revised and cleaned up. It really is a hack for them to work with these check boxes to begin with. On the PG editor page a modal dialog could be made available to view this data.

As to showing correct answers in a list after the problem (or at the bottom inside the probem?), I am not sure that is the best idea either. We can talk about this more in a meeting, but I think it will end up being a rather ugly addition to the page. We might as well go back to the results table. This needs more thought.

Copy link
Contributor

@Alex-Jordan Alex-Jordan left a comment

Choose a reason for hiding this comment

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

This works, and is a nice change.

@drgrice1 drgrice1 force-pushed the problem-grader-checkbox-to-button branch 3 times, most recently from 510a39e to bb454d1 Compare April 14, 2024 10:36
@pstaabp
Copy link
Sponsor Member

pstaabp commented Apr 17, 2024

As to showing correct answers in a list after the problem (or at the bottom inside the probem?), I am not sure that is the best idea either. We can talk about this more in a meeting, but I think it will end up being a rather ugly addition to the page. We might as well go back to the results table. This needs more thought.

First, since this looks like the previous problem grader, I take this as a comment as how to improve this and somewhat unrelated to this PR. It might be nice to integrate into the feedback buttons or perhaps have a separate grader button instead.

@drgrice1
Copy link
Sponsor Member Author

I should not have made that comment here. That is unrelated to this pull request entirely. However, it was part of the discussion that led to this pull request. The comment belongs in openwebwork/pg#1047.

@drgrice1
Copy link
Sponsor Member Author

Note also, that this does not change the problem grader at all. It just changes the mechanism that activates it.

The button is below the other usual problem/test submit buttons.  When
clicked problem graders are shown, and the button becomes the "Hide
Problem Grader" button.  Obviously when that is clicked the problem
graders are hidden.
@pstaabp
Copy link
Sponsor Member

pstaabp commented Apr 21, 2024

Going to go ahead a merge.

@pstaabp pstaabp merged commit e8747c4 into openwebwork:develop Apr 21, 2024
2 checks passed
@drgrice1 drgrice1 deleted the problem-grader-checkbox-to-button branch April 21, 2024 11:09
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.

None yet

3 participants