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

UI/UX behaviour Improvements on Report Builder audiences

    XMLWordPrintable

Details

    • Improvement
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • 4.0
    • 4.0
    • Report builder
    • MOODLE_400_STABLE
    • MOODLE_400_STABLE
    • Hide
      1. Login as Administrator
      2. Navigate to Reports > Report builder > Custom reports in site administration
      3. Press "New report" button
      4. Set any report name.
      5. Set Report source to "Users"
      6. Ensure "Include default setup" is checked
      7. Press Save
      8. Press "Audience" tab
      9. Press "All users" in the left sidebar to add the audience
      10. Press "Reload" button (or F5 key) in your browser
      11. Confirm that a warning is shown
      12. Cancel the warning
      13. Press "Save changes"
      14. Confirm that "Audience saved" toast notification is shown
      15. Press "Reload" button (or F5 key) in your browser
      16. Confirm no warning is shown and page reloads
      Show
      Login as Administrator Navigate to  Reports > Report builder > Custom reports  in site administration Press "New report" button Set any report name. Set Report source to "Users" Ensure "Include default setup" is checked Press Save Press "Audience" tab Press "All users" in the left sidebar to add the audience Press "Reload" button (or F5 key) in your browser Confirm that a warning is shown Cancel the warning Press "Save changes" Confirm that "Audience saved" toast notification is shown Press "Reload" button (or F5 key) in your browser Confirm no warning is shown and page reloads

    Description

      During the review of MDL-72588 Jun added these comments on the review that we need to address:

      1. Getting a toast notification when I add an audience category but not when I actually save it seems a bit strange.
      2. Probably good to add a form change checker to prevent the user from accidentally navigating away from the audiences setup page, especially when they haven't saved the changes yet for the audience category that they're setting up for the report.

      Also:

      1. Remove animation when expanding/collapsing audience sidebar (so it's consistent with the sidebar/conditions bar in editor tab).

       

      Attachments

        1. MDL-72952_Step 11.png
          MDL-72952_Step 11.png
          88 kB
        2. MDL-72952_Step 14.png
          MDL-72952_Step 14.png
          76 kB
        3. MDL-72952_Step 16.png
          MDL-72952_Step 16.png
          71 kB
        4. screenshot-1.png
          screenshot-1.png
          20 kB

        Issue Links

          Activity

            People

              mikelmartíncorrales Mikel Martín Corrales
              davidmatamoros David Matamoros
              Paul Holden Paul Holden
              Shamim Rezaie Shamim Rezaie
              Angelia Dela Cruz Angelia Dela Cruz
              Carlos Castillo, David Matamoros, Mikel Martín Corrales, Paul Holden
              Votes:
              0 Vote for this issue
              Watchers:
              7 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:
                19/Apr/22

                Time Tracking

                  Estimated:
                  Original Estimate - 0 minutes
                  0m
                  Remaining:
                  Remaining Estimate - 0 minutes
                  0m
                  Logged:
                  Time Spent - 5 hours, 25 minutes
                  5h 25m