Details

    • Type: Sub-task Sub-task
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.7, 1.9
    • Fix Version/s: 1.8.4, 1.9, 2.0
    • Component/s: Accessibility
    • Labels:
      None
    • Affected Branches:
      MOODLE_17_STABLE, MOODLE_19_STABLE
    • Fixed Branches:
      MOODLE_18_STABLE, MOODLE_19_STABLE, MOODLE_20_STABLE
    • Rank:
      34205

      Description

      Clearly identify the distinction of each hypertext link with meaningful text if read out of context, or associate with alterative links that have the same explanatory characteristics, and provide mechanisms that allow repeated reading of sequences of links common to several pages to be avoided.

      WCAG 1.0 References: 13.1, 13.6
      Sec. 508 References: 1194.22 (o)

        Issue Links

          Activity

          Hide
          Martin Dougiamas added a comment -

          Skip links have been added to the blocks, but one link to "skip to content" would be most useful.

          Show
          Martin Dougiamas added a comment - Skip links have been added to the blocks, but one link to "skip to content" would be most useful.
          Hide
          Martin Dougiamas added a comment -

          this can go in pagelib

          Show
          Martin Dougiamas added a comment - this can go in pagelib
          Hide
          Martin Dougiamas added a comment -

          Skip links have been added to course pages and admin settings pages (using accesshide to hide them from normal browsers)

          Show
          Martin Dougiamas added a comment - Skip links have been added to course pages and admin settings pages (using accesshide to hide them from normal browsers)
          Hide
          Nick Freear added a comment - - edited

          We need more discussion of these accessibility fixes - this helps fix MDL-4994, however:
          1) Better if the 'pagelib' link was the first text on page, just after <body> opening tag.
          2) Its broken in Internet Explorer, see recent fix for skip block links, MDL-11676.
          3) Its inaccessible to most keyboard-only users (except screen reader users), MDL-11695.

          I'll try to quickly do a fix, for 1.9!

          Show
          Nick Freear added a comment - - edited We need more discussion of these accessibility fixes - this helps fix MDL-4994 , however: 1) Better if the 'pagelib' link was the first text on page, just after <body> opening tag. 2) Its broken in Internet Explorer, see recent fix for skip block links, MDL-11676 . 3) Its inaccessible to most keyboard-only users (except screen reader users), MDL-11695 . I'll try to quickly do a fix, for 1.9!
          Hide
          Nick Freear added a comment -

          This patch fixes the 3 issues identified above. It affects these 4 files,

          • all, course/format/(NAME)/format.php - 1 line.
          • lib/pagelib.php -2 lines, print_header
          • lib/weblib.php +5 lines, print_header, including regular expression (could be 'split')
          • theme/standard/styles_layout.css

          Also cleans up, lang/en_utf8/moodle.php, theme/standard/styles_color.css, theme/formal_white/fw_layout.css (unused styles).

          Show
          Nick Freear added a comment - This patch fixes the 3 issues identified above. It affects these 4 files, all, course/format/(NAME)/format.php - 1 line. lib/pagelib.php -2 lines, print_header lib/weblib.php +5 lines, print_header, including regular expression (could be 'split') theme/standard/styles_layout.css Also cleans up, lang/en_utf8/moodle.php, theme/standard/styles_color.css, theme/formal_white/fw_layout.css (unused styles).
          Hide
          Nick Freear added a comment -

          Martin, I'm happy to commit this, if you give the nod.

          Note, I'd like to revisit this post-1.9 as this only adds the skip link to course pages, not the home page, modules etc. (Which is why MDL-4994 hasn't been fixed!)

          Many thanks, Nick.

          Show
          Nick Freear added a comment - Martin, I'm happy to commit this, if you give the nod. Note, I'd like to revisit this post-1.9 as this only adds the skip link to course pages, not the home page, modules etc. (Which is why MDL-4994 hasn't been fixed!) Many thanks, Nick.
          Hide
          Nick Freear added a comment -

          I'm currently working on this (assigned to me, elevated priority). Having problems with CVS, but hope to commit tomorrow BST - HEAD and 1.9 branches.

          Show
          Nick Freear added a comment - I'm currently working on this (assigned to me, elevated priority). Having problems with CVS, but hope to commit tomorrow BST - HEAD and 1.9 branches.
          Hide
          Nick Freear added a comment -

          I've committed fixes in 2 stages to 1.9 branch and HEAD, tagged _MERGED.

          • Part 1 is based on the patch (..course-2.diff) above, but introduces the 'skip_main_destination' function to weblib.php - the skip link is still printed in pagelib.php and this fixes all course formats.
          • Part 2 is attached (..course-32.diff) - it replaces the pagelib skip link with one in print_header in weblib, and adds skip links to /index.php and course/index.php.

          Tested on,

          • 'weekscss' and 'topics' course formats;
          • 'standardwhite' and 'cornflower' themes;
          • Win/IE 6, Mac/Firefox 2, Safari (tab key not for links, not functional) (Opera, tab key not for links, use 'a' key).
          Show
          Nick Freear added a comment - I've committed fixes in 2 stages to 1.9 branch and HEAD, tagged _MERGED. Part 1 is based on the patch (..course-2.diff) above, but introduces the 'skip_main_destination' function to weblib.php - the skip link is still printed in pagelib.php and this fixes all course formats. Part 2 is attached (..course-32.diff) - it replaces the pagelib skip link with one in print_header in weblib, and adds skip links to /index.php and course/index.php. Tested on, 'weekscss' and 'topics' course formats; 'standardwhite' and 'cornflower' themes; Win/IE 6, Mac/Firefox 2, Safari (tab key not for links, not functional) (Opera, tab key not for links, use 'a' key).
          Hide
          Nick Freear added a comment -

          (Note, 'skip_main_destination' function should perhaps be 'print_skip_main_destination' - post 1.9?)

          Fixed.

          Show
          Nick Freear added a comment - (Note, 'skip_main_destination' function should perhaps be 'print_skip_main_destination' - post 1.9?) Fixed.
          Hide
          Nick Freear added a comment -

          Reopening - need to use "skip_main_destination" function in Ann Adamcik's "topicscss" course format, see MDL-9306.

          Show
          Nick Freear added a comment - Reopening - need to use "skip_main_destination" function in Ann Adamcik's "topicscss" course format, see MDL-9306 .
          Hide
          Nick Freear added a comment -

          (Link - Ann Adamcik's new "topicscss" course format.)

          Show
          Nick Freear added a comment - (Link - Ann Adamcik's new "topicscss" course format.)
          Hide
          Nick Freear added a comment -

          Fix to "topicscss" in HEAD.

          Show
          Nick Freear added a comment - Fix to "topicscss" in HEAD.
          Hide
          Nick Freear added a comment -

          (Non-specific link to MDL-9306, try again!)

          Show
          Nick Freear added a comment - (Non-specific link to MDL-9306 , try again!)
          Hide
          Nick Freear added a comment -

          (Adding link to meta-bug.)

          Show
          Nick Freear added a comment - (Adding link to meta-bug.)

            People

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

              Dates

              • Created:
                Updated:
                Resolved: