Hi:
Maybe this is not in the right category but I don't know where to report it. This are some changes that seems right for us, but I don't know if they fit to all users.
We've change the way that student grades can be viewed.
We make two different functions (one for student view and another for theacher view).
We also change the style to make it more like drupal default theme.
| Comment | File | Size | Author |
|---|---|---|---|
| gradebook_style.patch | 618 bytes | manolopm | |
| gradebook_view.patch | 9.36 KB | manolopm |
Comments
Comment #1
MGN commentedNot a bug, more of a feature request. Thanks for the patches. I'll have a look and get back to you...
Comment #2
MGN commentedThanks for your work on this. I agree that we should have separate themable functions for the student and gradebook pages. This way, the views can be customized on a case by case basis using theme overrides. I think I'll be able to incorporate much of your gradebook_view.patch to provide this feature. Once that is accomplished, you can copy and modify anyway you like to suit your theme, as described in http://drupal.org/node/341628.
Style sheets can already be overridden - see http://drupal.org/node/263967 - so there is no need to change the default gradebook.css file unless one of the new theme functions introduces new classes. So there is no need for the style patch.
I'll update the status of this once the changes have been committed. Thanks again.
Comment #3
MGN commented@manolopm, I have implemented your suggestions in the most recent commit. I have gone a little farther and also unraveled the export code so that it has its own themable function as well. There have been other changes to take note of. See the warning on the Gradebook project home page (as well as the CHANGELOG file and/or the CVS messages) regarding the need to update to og_gradebook 6.x.2.x, as well as changes to the gradebook directory structure.