XMLWordPrintable

    Details

    • Testing Instructions:
      Hide

      In a variety of browsers:

      1. Create a Medium course, and an XS course
      2. Fill in some grade entries, including:
        • From the activity with feedback
        • Overwriting in the gradebook itself
      3. View the gradebook for each course
      4. Confirm:
        1. The tooltips display the correct information for each grade cell
        2. That only one tooltip is displayed at once
      Show
      In a variety of browsers: Create a Medium course, and an XS course Fill in some grade entries, including: From the activity with feedback Overwriting in the gradebook itself View the gradebook for each course Confirm: The tooltips display the correct information for each grade cell That only one tooltip is displayed at once
    • Affected Branches:
      MOODLE_27_STABLE
    • Pull Master Branch:
      MDL-46731-master
    • Sprint:
      FRONTEND Sprint 14

      Description

      This isn't strictly necessary for this task, but having seen the code, we should look at rewriting it. It's currently adding one event handler per grade cell, and then when the tooltip is opened, we add more (which are never removed).
      We also add large numbers of event handlers for other areas.

      Temporarily removing these does seem to improve the performance of the page, but it's difficult to tell subjectively, and it will likely make more of a difference on tablets.

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                dobedobedoh Andrew Nicols
                Reporter:
                dobedobedoh Andrew Nicols
                Peer reviewer:
                Jetha Chan
                Participants:
                Component watchers:
                Jake Dallimore, Jun Pataleta, Ryan Wyllie, Jake Dallimore, Jun Pataleta, Ryan Wyllie
              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: