Details

    • Rank:
      44639

      Description

      Issue
      Explicit labels - The select input (drop down menu) for the "Add a block" section has no label associated to it.
      Title attributes - The select input (drop down menu) for the "Add a block" section has no title attribute associated to it. Given the organization of the interface, using a title attribute with a value of "Add a block" would be helpful to non-sighted users.

      Standard Level
      WCAG 2 1.1.1 (A) http://www.w3.org/WAI/WCAG20/quickref/#qr-text-equiv-all

      Impact
      Serious

      Example Link
      http://demo.moodle.net/user/profile.php?id=0&sesskey=ZdAicfllpV&bui_moveid=293

      Test Steps

      1. Login as student
      2. Click on student's name
      3. Click the customize this page button
      4. Tab to the add a block drop down with a screen reader.

        Activity

        Hide
        Michael de Raadt added a comment -

        I can confirm that. It looks like we missed that one.

        Show
        Michael de Raadt added a comment - I can confirm that. It looks like we missed that one.
        Hide
        Jason Fowler added a comment -

        I've added the label, but used CSS to reposition it off the screen so the page layout isn't affected. This should allow screen readers to identify the field (via the for attribute) while preventing changes to the page for sighted users.

        Show
        Jason Fowler added a comment - I've added the label, but used CSS to reposition it off the screen so the page layout isn't affected. This should allow screen readers to identify the field (via the for attribute) while preventing changes to the page for sighted users.
        Hide
        Jason Fowler added a comment -

        need to use accesshide in the class attribute - it's a nice way to do this.

        Show
        Jason Fowler added a comment - need to use accesshide in the class attribute - it's a nice way to do this.
        Hide
        Jason Fowler added a comment -

        Implemented .accesshide now - works as it did before, only in less code, and it's nicer. So glad Fred mentioned this before I put it up for integration.

        Show
        Jason Fowler added a comment - Implemented .accesshide now - works as it did before, only in less code, and it's nicer. So glad Fred mentioned this before I put it up for integration.
        Hide
        Frédéric Massart added a comment -

        Looks good to me Jason. Pushing for integration. Cheers!

        Show
        Frédéric Massart added a comment - Looks good to me Jason. Pushing for integration. Cheers!
        Hide
        Dan Poltawski added a comment -

        Integrated to master, 24, 23 and 22 thanks Jason.

        Show
        Dan Poltawski added a comment - Integrated to master, 24, 23 and 22 thanks Jason.
        Hide
        Andrew Davis added a comment -

        Works as described. Passed.

        Show
        Andrew Davis added a comment - Works as described. Passed.
        Hide
        Eloy Lafuente (stronk7) added a comment -

        Changes are now upstream, thanks for your collaboration!

        If you are going to have any celebration next days, enjoy with your gang, if not, too!

        Ciao

        Show
        Eloy Lafuente (stronk7) added a comment - Changes are now upstream, thanks for your collaboration! If you are going to have any celebration next days, enjoy with your gang, if not, too! Ciao

          People

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

            Dates

            • Created:
              Updated:
              Resolved: