Details

    • Type: Sub-task
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 1.9.4
    • Fix Version/s: 1.9.5
    • Component/s: Gradebook, Usability
    • Labels:
      None
    • Affected Branches:
      MOODLE_19_STABLE
    • Fixed Branches:
      MOODLE_19_STABLE

      Description

      Add a horizontal scrollbar to scroll grades while students stay fixed (as used in the LSU simple grader report).

      Note: This feature is not possible in IE6.

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

              Hide
              tsala Helen Foster added a comment -

              Comment from Nicolas copied from MDL-17807:

              This is implemented in HEAD. Thanks for your feedback, please keep it coming.

              Show
              tsala Helen Foster added a comment - Comment from Nicolas copied from MDL-17807 : This is implemented in HEAD. Thanks for your feedback, please keep it coming.
              Hide
              carolinemoore Caroline Moore added a comment -

              This would be a truly fantastic addition to the Moodle gradebook. I'd also love to see the column headings (assignment names) stay fixed as one scrolls down the page; this would be very helpful for classes with more than 10 students. I've had this request from many of my instructors.

              Show
              carolinemoore Caroline Moore added a comment - This would be a truly fantastic addition to the Moodle gradebook. I'd also love to see the column headings (assignment names) stay fixed as one scrolls down the page; this would be very helpful for classes with more than 10 students. I've had this request from many of my instructors.
              Hide
              elenaivanova Elena Ivanova added a comment -

              Transferring my comments from MDL-17807, since this is not-LSU specific.

              Horizontal scrollbar that is not part of the browser window makes it good only in the case of small amount of students per page (max 5)
              Unless we can make the horizontal stroller to appear in browser, I suggest to improve the functionality and allow user to choose whether to have student names stuck or not, plus possibility to repeat both rows with assignment names and columns with student names.

              We can do it similar to 1.8 Preferences:

              • Reprint Headers: select # (meaning how often headers will be repeated)
              • Reprint Headers with Categories names: Yes/No (depending on gradebook structure, instructor may want to see just names of the activities or corresponding categories as well)
              • Reprint Student names : select #
              • Stick Student names on the left?: Yes/No
              Show
              elenaivanova Elena Ivanova added a comment - Transferring my comments from MDL-17807 , since this is not-LSU specific. Horizontal scrollbar that is not part of the browser window makes it good only in the case of small amount of students per page (max 5) Unless we can make the horizontal stroller to appear in browser, I suggest to improve the functionality and allow user to choose whether to have student names stuck or not, plus possibility to repeat both rows with assignment names and columns with student names. We can do it similar to 1.8 Preferences: Reprint Headers: select # (meaning how often headers will be repeated) Reprint Headers with Categories names: Yes/No (depending on gradebook structure, instructor may want to see just names of the activities or corresponding categories as well) Reprint Student names : select # Stick Student names on the left?: Yes/No
              Hide
              tsala Helen Foster added a comment -

              Just to note a problem Martin mentioned when using Firefox on Vista: The horizontal scrollbar obscures the overall average row.

              Show
              tsala Helen Foster added a comment - Just to note a problem Martin mentioned when using Firefox on Vista: The horizontal scrollbar obscures the overall average row.
              Hide
              dougiamas Martin Dougiamas added a comment -

              Also, this display with the fixed column should be turned OFF when USER->screenreader is on (ie go back to the old method for this case).

              The reason is that the separate tables break accessibility.

              Show
              dougiamas Martin Dougiamas added a comment - Also, this display with the fixed column should be turned OFF when USER->screenreader is on (ie go back to the old method for this case). The reason is that the separate tables break accessibility.
              Hide
              nicolasconnault Nicolas Connault added a comment -

              Added a grader report preference for using the new fixed student column, and made it always off if $USER->screenreader is set. You can see this on the head test server now.

              Show
              nicolasconnault Nicolas Connault added a comment - Added a grader report preference for using the new fixed student column, and made it always off if $USER->screenreader is set. You can see this on the head test server now.
              Hide
              tsala Helen Foster added a comment -

              Hmm, I just tried setting screenreader to Yes in my profile, but the horizontal scrollbar remained.

              Show
              tsala Helen Foster added a comment - Hmm, I just tried setting screenreader to Yes in my profile, but the horizontal scrollbar remained.
              Hide
              aborrow Anthony Borrow added a comment -

              I wonder if the following strings may be confusing to English speakers:

              $string['configfixedstudents'] = 'Fixes the students column in the grader report, allowing the grades to scroll horizontally.';
              $string['fixedstudents'] = 'Fixed students column';

              I think someone may read 'Fixed students column' and think that this one is fixing something that was broken rather than providing a static student column. I would probably stick with the singular hear because you are referring to a single column called students, so perhaps 'Static student column' might be better. The same is true of the config description. I think we are trying to say that it: "Allows grades to scroll horizontally by adding a static student column".

              Peace - Anthony

              Show
              aborrow Anthony Borrow added a comment - I wonder if the following strings may be confusing to English speakers: $string ['configfixedstudents'] = 'Fixes the students column in the grader report, allowing the grades to scroll horizontally.'; $string ['fixedstudents'] = 'Fixed students column'; I think someone may read 'Fixed students column' and think that this one is fixing something that was broken rather than providing a static student column. I would probably stick with the singular hear because you are referring to a single column called students, so perhaps 'Static student column' might be better. The same is true of the config description. I think we are trying to say that it: "Allows grades to scroll horizontally by adding a static student column". Peace - Anthony
              Hide
              tsala Helen Foster added a comment -

              Anthony, thanks for your comments.

              I'd rather not have a horizontal scrollbar grader report preference at all, as I think that over 20 settings is too much, and should be reduced.

              How about only having an admin setting for determining whether the horizontal scrollbar is used site-wide or not (in addition to the screenreader on/off setting)?

              Show
              tsala Helen Foster added a comment - Anthony, thanks for your comments. I'd rather not have a horizontal scrollbar grader report preference at all, as I think that over 20 settings is too much, and should be reduced. How about only having an admin setting for determining whether the horizontal scrollbar is used site-wide or not (in addition to the screenreader on/off setting)?
              Hide
              nicolasconnault Nicolas Connault added a comment -

              Thanks for the ideas Anthony, I have changed the strings accordingly. I've removed the grader report setting but left it in admin, and tested the screenreader setting.

              Show
              nicolasconnault Nicolas Connault added a comment - Thanks for the ideas Anthony, I have changed the strings accordingly. I've removed the grader report setting but left it in admin, and tested the screenreader setting.
              Hide
              elenaivanova Elena Ivanova added a comment -

              I suggest adding an option for the admins to either allow or prevent instructors from setting up the static column. Add "Force" checkbox, for example.
              (we have over 1000 instructors and I would rather let them decide on their own in this case).
              Thank you.

              Show
              elenaivanova Elena Ivanova added a comment - I suggest adding an option for the admins to either allow or prevent instructors from setting up the static column. Add "Force" checkbox, for example. (we have over 1000 instructors and I would rather let them decide on their own in this case). Thank you.
              Hide
              nicolasconnault Nicolas Connault added a comment -

              Gradebook patch applied, resolving this issue.

              Show
              nicolasconnault Nicolas Connault added a comment - Gradebook patch applied, resolving this issue.
              Hide
              tsala Helen Foster added a comment -

              Thanks Nicolas, the improvement is documented in http://docs.moodle.org/en/Gradebook_report_settings and in http://docs.moodle.org/en/Grader_report

              Removing 2.0 fix version as instructions state 'Do not include the current HEAD version unless it's the only one.'

              Show
              tsala Helen Foster added a comment - Thanks Nicolas, the improvement is documented in http://docs.moodle.org/en/Gradebook_report_settings and in http://docs.moodle.org/en/Grader_report Removing 2.0 fix version as instructions state 'Do not include the current HEAD version unless it's the only one.'
              Hide
              mattgibson Matt Gibson added a comment -

              I love this new feature, but our IT guys are dragging their heels on IE upgrades leaving us with a mixture of IE6 and IE7, which means that some people are getting confusing screens. Could a browser check be added so that if HTTP_USER_AGENT is IE6, this feature is disabled for the session?

              Show
              mattgibson Matt Gibson added a comment - I love this new feature, but our IT guys are dragging their heels on IE upgrades leaving us with a mixture of IE6 and IE7, which means that some people are getting confusing screens. Could a browser check be added so that if HTTP_USER_AGENT is IE6, this feature is disabled for the session?
              Hide
              nicolasconnault Nicolas Connault added a comment -

              Yes Matt, this will most likely be fixed in 1.9.6. IE6 users seem to enjoy waiting for updates, so I'm sure they won't mind.

              Show
              nicolasconnault Nicolas Connault added a comment - Yes Matt, this will most likely be fixed in 1.9.6. IE6 users seem to enjoy waiting for updates, so I'm sure they won't mind.
              Hide
              samhemelryk Sam Hemelryk added a comment -

              V. Nice thanks

              Show
              samhemelryk Sam Hemelryk added a comment - V. Nice thanks
              Hide
              tsala Helen Foster added a comment -

              Matt and Nicolas, thanks for your comments. Please watch / vote for / comment on MDL-19596 - Disable gradebook horizontal scrollbar for IE6 users.

              Show
              tsala Helen Foster added a comment - Matt and Nicolas, thanks for your comments. Please watch / vote for / comment on MDL-19596 - Disable gradebook horizontal scrollbar for IE6 users.
              Hide
              nicolasconnault Nicolas Connault added a comment -

              There's a bug in IE7: a vertical scrollbar appears on the grader report. If scrolled, the rows come out of alignment with the student names.

              Please put any other bug report regarding this feature here.

              Show
              nicolasconnault Nicolas Connault added a comment - There's a bug in IE7: a vertical scrollbar appears on the grader report. If scrolled, the rows come out of alignment with the student names. Please put any other bug report regarding this feature here.
              Hide
              nicolasconnault Nicolas Connault added a comment -

              This should be resolved now, please comment here if the solution doesn't work for you.

              Show
              nicolasconnault Nicolas Connault added a comment - This should be resolved now, please comment here if the solution doesn't work for you.
              Hide
              dburdick Debra Burdick added a comment -

              This is still a problem for IE8 on Moodle 1.9.5+ (Build: 20090722).
              Can not get the rows to align no matter what I do.
              This is preventing us from upgrading and our timeline is getting really tight.
              I hope someone is able to fix this. I use Firefox but many of our faculty & course designers are prevented from downloading and using it and are forced to use IE.
              HELP. I will also attach a screen shot. Thanks.

              Show
              dburdick Debra Burdick added a comment - This is still a problem for IE8 on Moodle 1.9.5+ (Build: 20090722). Can not get the rows to align no matter what I do. This is preventing us from upgrading and our timeline is getting really tight. I hope someone is able to fix this. I use Firefox but many of our faculty & course designers are prevented from downloading and using it and are forced to use IE. HELP. I will also attach a screen shot. Thanks.
              Hide
              dburdick Debra Burdick added a comment -

              This problem continues in Moodle 1.9.5+ (Build: 20090722)

              Show
              dburdick Debra Burdick added a comment - This problem continues in Moodle 1.9.5+ (Build: 20090722)
              Hide
              dburdick Debra Burdick added a comment -

              Well apparently this is just a problem with our custom theme does not appear to be a problem with standard moodle themes. So I appologize for the post.

              Show
              dburdick Debra Burdick added a comment - Well apparently this is just a problem with our custom theme does not appear to be a problem with standard moodle themes. So I appologize for the post.
              Hide
              dburdick Debra Burdick added a comment -

              Well apparently this problem occurs in a lot of themes. Remote Learner is having the same problem on their training server. I am currently completing my MCCC so have a sandbox on their server so did a little testing and the same issue arises. The themes in the red box DO NOT appear to have any alignment issues with IE8 all of the other ones do. We are hoping to get this resolved for at least one of the themes. Any other assistance you can provide to remote learner would be most welcome. This has been taking an incredible amount of time to resolve and has really delayed our 1.9.5 upgrade. Most of our faculty is forced to use IE so we need this to work and simply disabling the scrolling features is not an option. Thanks!

              Show
              dburdick Debra Burdick added a comment - Well apparently this problem occurs in a lot of themes. Remote Learner is having the same problem on their training server. I am currently completing my MCCC so have a sandbox on their server so did a little testing and the same issue arises. The themes in the red box DO NOT appear to have any alignment issues with IE8 all of the other ones do. We are hoping to get this resolved for at least one of the themes. Any other assistance you can provide to remote learner would be most welcome. This has been taking an incredible amount of time to resolve and has really delayed our 1.9.5 upgrade. Most of our faculty is forced to use IE so we need this to work and simply disabling the scrolling features is not an option. Thanks!
              Hide
              tsala Helen Foster added a comment -

              Debra, thanks for your comments.

              I'm resolving this issue since the horizontal scrollbar seems to be working fine when using any of the standard themes.

              If you're still having problems with a custom theme, please post in the themes forum http://moodle.org/mod/forum/view.php?id=46 and/or contact the theme creator.

              Show
              tsala Helen Foster added a comment - Debra, thanks for your comments. I'm resolving this issue since the horizontal scrollbar seems to be working fine when using any of the standard themes. If you're still having problems with a custom theme, please post in the themes forum http://moodle.org/mod/forum/view.php?id=46 and/or contact the theme creator.
              Hide
              andyjdavis Andrew Davis added a comment -

              This fix (Added a bit of JS to resize the left header of the fixed student column. This should help to keep the rows below aligned) is causing some display problems in IE 7 and 8.

              Show
              andyjdavis Andrew Davis added a comment - This fix (Added a bit of JS to resize the left header of the fixed student column. This should help to keep the rows below aligned) is causing some display problems in IE 7 and 8.
              Hide
              andyjdavis Andrew Davis added a comment -

              Closing. The alignment bugs were dealt with as part of another issue.

              Show
              andyjdavis Andrew Davis added a comment - Closing. The alignment bugs were dealt with as part of another issue.

                People

                • Votes:
                  15 Vote for this issue
                  Watchers:
                  4 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved:
                    Fix Release Date:
                    13/May/09