Moodle
  1. Moodle
  2. MDL-32743

New resource javascript icons can't be accessed with keyboard

    Details

    • Testing Instructions:
      Hide
      • open course page
      • turn editing on
      • tab to a command area
        • ensure that all icons are selectable
      • indent right an activity for the first time
        • ensure that the indent left icon is also tabbable

      Please note that the drag/drop icon cannot be tabbed to (it doesn't support keyboard drag/drop)

      Show
      open course page turn editing on tab to a command area ensure that all icons are selectable indent right an activity for the first time ensure that the indent left icon is also tabbable Please note that the drag/drop icon cannot be tabbed to (it doesn't support keyboard drag/drop)
    • Difficulty:
      Easy
    • Affected Branches:
      MOODLE_23_STABLE
    • Fixed Branches:
      MOODLE_23_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      MDL-32743-master-2

      Description

      It's no longer possible to tab to each of the javascript icons (e.g. left, right, delete, show/hide).

      This is caused by a removal of the href attribute on the anchor.

        Gliffy Diagrams

          Activity

          Hide
          Ruslan Kabalin added a comment -

          +1

          Show
          Ruslan Kabalin added a comment - +1
          Hide
          Ruslan Kabalin added a comment -

          Although it is used everywhere, using hash in href is not good from semantics point, preserving the original link would look better (even though it would be overriden by js action).

          Show
          Ruslan Kabalin added a comment - Although it is used everywhere, using hash in href is not good from semantics point, preserving the original link would look better (even though it would be overriden by js action).
          Hide
          Andrew Nicols added a comment -

          Thanks for that thought Ruslan.

          I've made the suggested change and add e.preventDefault() to all events.

          Show
          Andrew Nicols added a comment - Thanks for that thought Ruslan. I've made the suggested change and add e.preventDefault() to all events.
          Hide
          Ruslan Kabalin added a comment -

          Looks good, thanks Andrew.

          Show
          Ruslan Kabalin added a comment - Looks good, thanks Andrew.
          Hide
          Sam Hemelryk added a comment -

          Thanks guys, code looks good and this has been integrated now

          Show
          Sam Hemelryk added a comment - Thanks guys, code looks good and this has been integrated now
          Hide
          Frédéric Massart added a comment -

          Test successful

          Show
          Frédéric Massart added a comment - Test successful
          Hide
          Eloy Lafuente (stronk7) added a comment -

          This is now part of Moodle and a few millions people around the globe will be using it soon. Isn't that awesome?

          Many, many thanks and don't forget http://youtu.be/4N7dPaP5Z8U

          Closing, ciao

          Show
          Eloy Lafuente (stronk7) added a comment - This is now part of Moodle and a few millions people around the globe will be using it soon. Isn't that awesome? Many, many thanks and don't forget http://youtu.be/4N7dPaP5Z8U Closing, ciao

            People

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

              Dates

              • Created:
                Updated:
                Resolved: