Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-30819

Cannot highlight the columns in the grader report

    Details

    • Database:
      MySQL
    • Testing Instructions:
      Hide

      1. Make sure Javascript is enabled in your browser. Also make sure that the site settings grade_report_enableajax and enableajax are turned on.
      2. Go to the grader report http://YOURSITE/grade/report/grader/index.php?id=XX
      3. Try clicking on empty space in the cell that contains the students name. This should highlight the entire row. Clicking it again should unhighlight the row.
      4. Try clicking on empty space in the cell containing an activity name. This should highlight the entire column. Clicking it again should unhighlight the column.

      Show
      1. Make sure Javascript is enabled in your browser. Also make sure that the site settings grade_report_enableajax and enableajax are turned on. 2. Go to the grader report http://YOURSITE/grade/report/grader/index.php?id=XX 3. Try clicking on empty space in the cell that contains the students name. This should highlight the entire row. Clicking it again should unhighlight the row. 4. Try clicking on empty space in the cell containing an activity name. This should highlight the entire column. Clicking it again should unhighlight the column.
    • Affected Branches:
      MOODLE_20_STABLE, MOODLE_21_STABLE
    • Fixed Branches:
      MOODLE_21_STABLE, MOODLE_22_STABLE
    • Pull Master Branch:
      MDL-30819_grader_col_highlight

      Description

      Currently in Moodle 2.0.3 & 2.1.3 only the rows can be highlighted

      Documentation says you should be able to highlight both the column and the row in the grader report: http://docs.moodle.org/22/en/Gradebook#Highlighting_rows_and_columns

      In 1.9 you could highlight both the students' rows and the columns of the assignments and this should work in 2.x also.

      Tested this in FireFox 8.0.1, Internet Explorer 8.0.7600, Chrome 16.0.912.63 all had the same results.

        Gliffy Diagrams

          Attachments

            Activity

            Hide
            salvetore Michael de Raadt added a comment -

            Thanks for reporting this.

            Show
            salvetore Michael de Raadt added a comment - Thanks for reporting this.
            Hide
            andyjdavis Andrew Davis added a comment -

            Added a solution as well as editing the testing instructions a little.

            Show
            andyjdavis Andrew Davis added a comment - Added a solution as well as editing the testing instructions a little.
            Hide
            poltawski Dan Poltawski added a comment -

            Looks good to me

            Show
            poltawski Dan Poltawski added a comment - Looks good to me
            Hide
            andyjdavis Andrew Davis added a comment -

            Added branches. Putting this up for integration.

            Show
            andyjdavis Andrew Davis added a comment - Added branches. Putting this up for integration.
            Hide
            poltawski Dan Poltawski added a comment -

            The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week.

            TIA and ciao

            Show
            poltawski Dan Poltawski added a comment - The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week. TIA and ciao
            Hide
            samhemelryk Sam Hemelryk added a comment -

            Thanks Andrew, this has been integrated now

            Show
            samhemelryk Sam Hemelryk added a comment - Thanks Andrew, this has been integrated now
            Hide
            salvetore Michael de Raadt added a comment -

            Test result: Success

            Tested in 2.1, 2.2 and master.

            Show
            salvetore Michael de Raadt added a comment - Test result: Success Tested in 2.1, 2.2 and master.
            Hide
            poltawski Dan Poltawski added a comment -

            Bonza mate!

            Your changes have made it into the Moodle release! Its time to celebrate! Put a shrimp on the barbie and grab a stubby.

            Hooroo

            Show
            poltawski Dan Poltawski added a comment - Bonza mate! Your changes have made it into the Moodle release! Its time to celebrate! Put a shrimp on the barbie and grab a stubby. Hooroo

              People

              • Votes:
                1 Vote for this issue
                Watchers:
                1 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  14/May/12