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

Book hidden chapter block contains markup in alt and title attributes.

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 2.9.4, 3.0.2
    • Fix Version/s: 2.9.5, 3.0.2
    • Component/s: Accessibility, Book
    • Labels:
    • Testing Instructions:
      Hide
      1. In a course, create a book with three sample chapters.
      2. On the table of contents block, hide chapter two with the eye.
      3. Inspect the markup of the chapter's action list:
        • Confirm no 'span' tags in the 'alt' and 'title' tags of the chapter action list (the 'div class="action-list"').
      4. Make the 3rd chapter as a subchapter.
      5. Hide the subchapter.
      6. Inspect the markup of the subchapter's action list:
        • Confirm no 'span' tags in the 'alt' and 'title' tags of the chapter action list (the 'div class="action-list"').
      Show
      In a course, create a book with three sample chapters. On the table of contents block, hide chapter two with the eye. Inspect the markup of the chapter's action list: Confirm no 'span' tags in the 'alt' and 'title' tags of the chapter action list (the 'div class="action-list"'). Make the 3rd chapter as a subchapter. Hide the subchapter. Inspect the markup of the subchapter's action list: Confirm no 'span' tags in the 'alt' and 'title' tags of the chapter action list (the 'div class="action-list"').
    • Affected Branches:
      MOODLE_29_STABLE, MOODLE_30_STABLE
    • Fixed Branches:
      MOODLE_29_STABLE, MOODLE_30_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      wip_MDL-53029_master
    • Pull Master Diff URL:

      Description

      Ref: https://moodle.org/mod/forum/discuss.php?d=327661#p1317724

      When hiding a chapter in a book, the book table of contents block changes to reflect this. However the alt and title attributes of the link and image tags contain embedded span tags which have speech marks. In other themes (besides Clean) this is breaking things. In Clean it does not make any difference visually, however from an accessibility point of view I dread to think what the screen reader says to the user.

        Attachments

          Activity

            People

            Assignee:
            gb2048 Gareth J Barnard
            Reporter:
            gb2048 Gareth J Barnard
            Peer reviewer:
            Jun Pataleta Jun Pataleta
            Integrator:
            Dan Poltawski Dan Poltawski
            Tester:
            Ryan Wyllie Ryan Wyllie
            Participants:
            Component watchers:
            Andrew Lyons, Dongsheng Cai, Huong Nguyen, Jun Pataleta, Michael Hawkins, Shamim Rezaie, Simey Lameze, David Jones, Adrian Greeve, Jake Dallimore, Mathew May, Mihail Geshoski, Peter Dias, Sujith Haridasan
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:
              Fix Release Date:
              11/Jan/16