search

lsuits / OBSOLETE--DO-NOT-USE--gradebook_moodle

LSU's fork of Moodle for gradebook changes
GNU General Public License v3.0
2 stars 1 forks source link

Keep Highest / Drop Lowest Number Increase #29

Closed BuddyEthridge closed 12 years ago

BuddyEthridge commented 12 years ago

A request came in to increase the maximum possible in the dropdown for both Keep Highest and Drop Lowest. It is currently set at "20".

Is it possible to make this an integer field that allows instructors to manually input their desired number of items to keep/drop? Let's discuss options.

philcali commented 12 years ago

This is interesting. This number was a Moodle default stolen from the item_form.php, but their full view implementation was exactly as you suggested prior to us changing it.

rrusso commented 12 years ago

I wanted to do this anyway. Drop downs are annoying.

philcali commented 12 years ago

Yeah, but I like our consistent approach. I say we either make the maximum admin configurable, or make them all text boxes. I guess at this point, we need to figure which is the most user friendly.

rrusso commented 12 years ago

I think they should all be text boxes including max grade.

philcali commented 12 years ago

Max grade is already text boxes with the exception of graded modules, correct?

rrusso commented 12 years ago

Yes. Inconsistent and I don't like it.

It should be text boxes in modules too.

philcali commented 12 years ago

Fixed in https://github.com/lsuits/gradebook_moodle/commit/e7651dd186766e1bf07a1b0e9c320930e8eb393b