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

User tours leave unnecessary aria tags in the page

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 3.2.4, 3.3.1, 3.4
    • Fix Version/s: 3.3.4, 3.4.1
    • Component/s: User Tours
    • Labels:
    • Testing Instructions:
      Hide
      1. Log in as admin
      2. Go to Site administration / Appearance / User tours.
      3. Ensure that editing mode is off (you will see "Add an activity or resource" if it is, you can switch it off by going to the action menu in the header and selecting "Turn editing off")
      4. Edit "Boost - administrator tour" and select "All" for "Role" setting
      5. Click on "Boost - administrator tour" link to access the tour steps
      6. Follow "force the tour to be displayed" link
      7. Go to the site home page so the user tour is executed
      8. Click Next twice, to get to the third step of the tour, about Customisation.
      9. Click End tour.
      10. User your browser's developer tools to inspect your site header (you can search for "#page-header .card-block")
      11. There should not be any "aria-describedby" attribute
      Show
      Log in as admin Go to Site administration / Appearance / User tours. Ensure that editing mode is off (you will see "Add an activity or resource" if it is, you can switch it off by going to the action menu in the header and selecting "Turn editing off") Edit "Boost - administrator tour" and select "All" for "Role" setting Click on "Boost - administrator tour" link to access the tour steps Follow "force the tour to be displayed" link Go to the site home page so the user tour is executed Click Next twice, to get to the third step of the tour, about Customisation. Click End tour. User your browser's developer tools to inspect your site header (you can search for "#page-header .card-block") There should not be any "aria-describedby" attribute
    • Affected Branches:
      MOODLE_32_STABLE, MOODLE_33_STABLE, MOODLE_34_STABLE
    • Fixed Branches:
      MOODLE_33_STABLE, MOODLE_34_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      MDL-60079-master
    • Pull Master Diff URL:

      Description

      Steps to reproduce:

      1. Go to qa.moodle.net, and log in as manager.
      2. Go to Site administration / Appearance / User tours.
      3. Edit the properties on the Boost - administrator tour, to show to all roles.
      4. Go to the site home page.
      5. Click Next twice, to get to the third step of the tour, about Customisation.
      6. Click End tour.
      7. User your browser's developer tools to inspect the 'Moodle QA Testing Site' header.

      Actual result: one of the surrounding divs still has aria-describedby="tour-step-tool_usertours_6_1_1479366244-2-body" even though that thing no longer exists in the page.

      There is also one left on div.drawer-toggle > button, from the second step.

      Expected result: these aria attributes should be removen when you leave that step of the tour.

        Attachments

          Issue Links

          Testing discovered

          Bug - A problem which impairs or prevents Moodle from functioning correctly. MDL-61234 User tours race condition

          • Minor - Minor loss of function where workaround is possible
          • Closed

            Activity

              People

              Assignee:
              thinhpham83 Thinh Pham Tien
              Reporter:
              timhunt Tim Hunt
              Peer reviewer:
              Tim Hunt
              Integrator:
              David Monllaó
              Tester:
              Simey Lameze
              Participants:
              Component watchers:
              Andrew Nicols, Dongsheng Cai, Huong Nguyen, Jun Pataleta, Michael Hawkins, Shamim Rezaie, Simey Lameze
              Votes:
              0 Vote for this issue
              Watchers:
              6 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:
                Fix Release Date:
                15/Jan/18