Details

    • Type: Sub-task Sub-task
    • Status: Closed
    • Priority: Minor 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
    • Rank:
      36413

      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.

        Issue Links

          Activity

          Hide
          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
          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
          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
          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
          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
          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
          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
          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
          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
          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
          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
          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
          Helen Foster added a comment -

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

          Show
          Helen Foster added a comment - Hmm, I just tried setting screenreader to Yes in my profile, but the horizontal scrollbar remained.
          Hide
          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
          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
          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
          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
          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
          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
          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
          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
          Nicolas Connault added a comment -

          Gradebook patch applied, resolving this issue.

          Show
          Nicolas Connault added a comment - Gradebook patch applied, resolving this issue.
          Hide
          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
          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
          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
          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
          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
          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
          Sam Hemelryk added a comment -

          V. Nice thanks

          Show
          Sam Hemelryk added a comment - V. Nice thanks
          Hide
          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
          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
          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
          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
          Nicolas Connault added a comment -

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

          Show
          Nicolas Connault added a comment - This should be resolved now, please comment here if the solution doesn't work for you.
          Hide
          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
          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
          Debra Burdick added a comment -

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

          Show
          Debra Burdick added a comment - This problem continues in Moodle 1.9.5+ (Build: 20090722)
          Hide
          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
          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
          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
          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
          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
          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
          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
          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
          Andrew Davis added a comment -

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

          Show
          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: