Moodle
  1. Moodle
  2. MDL-30900

consistency in denoting start of the main content

    Details

    • Rank:
      33908

      Description

      There needs to be consistency across all of the pages within Moodle to denote the main content area. The skip to main content link works, but there needs to be a consistent heading structure so users always know the main content starts, like always putting the same heading level at the start of the main content.

        Activity

        Hide
        Sam Hemelryk added a comment -

        Massively epic change unfortunately.

        We'd need to investigate all pages one by one to look for this as each page has control of the pages main heading.
        We could probably easily get a head start using a simple grep for scripts not setting $PAGE->header. But it wouldn't constitute an accurate portrayal and we would need to check every page one by one still.

        Although certainly this is worth doing, just noting it will be a big job.

        Many thanks
        Sam

        Show
        Sam Hemelryk added a comment - Massively epic change unfortunately. We'd need to investigate all pages one by one to look for this as each page has control of the pages main heading. We could probably easily get a head start using a simple grep for scripts not setting $PAGE->header. But it wouldn't constitute an accurate portrayal and we would need to check every page one by one still. Although certainly this is worth doing, just noting it will be a big job. Many thanks Sam
        Hide
        Sam Hemelryk added a comment -

        Coming back to this.

        Originally I was thinking of the main page heading, at the top of the page by default, not really what you would call the main content area.
        For many pages we don't have a heading at the start of the main content, we use the banner heading to signify the "heading" for the main content on many pages, and when there is no clear decision the about the heading the sites/courses full name is used.

        If the concern is simply that there be that constant header then we can identify all pages that are not setting it and look at them (I have a script for that) but we would still need to review every entry page to make sure that the heading being used is meaningful.
        However I get the impression reading this issue that perhaps we need to either have a heading in the main content of link the current heading (aria-labelledby perhaps) to the main content.
        If thats the case then fine, we still need to review every page, but this time we would be looking to add a second heading where there isn't already one and make sure that the two headings are both meaningful and not duplicated.

        Greg would you mind shedding some light on this, I think either way it is going to be a big job which isn't a problem but before I start I want to know that the time I invest will be spent in the right way.

        Many thanks
        Sam

        Show
        Sam Hemelryk added a comment - Coming back to this. Originally I was thinking of the main page heading, at the top of the page by default, not really what you would call the main content area. For many pages we don't have a heading at the start of the main content, we use the banner heading to signify the "heading" for the main content on many pages, and when there is no clear decision the about the heading the sites/courses full name is used. If the concern is simply that there be that constant header then we can identify all pages that are not setting it and look at them (I have a script for that) but we would still need to review every entry page to make sure that the heading being used is meaningful. However I get the impression reading this issue that perhaps we need to either have a heading in the main content of link the current heading (aria-labelledby perhaps) to the main content. If thats the case then fine, we still need to review every page, but this time we would be looking to add a second heading where there isn't already one and make sure that the two headings are both meaningful and not duplicated. Greg would you mind shedding some light on this, I think either way it is going to be a big job which isn't a problem but before I start I want to know that the time I invest will be spent in the right way. Many thanks Sam
        Hide
        Greg Kraus added a comment -

        It's been a while since I went through this testing. From what I remember there were times that the "main content" seemed to be in slightly different locations on different pages. I'm sorry, I don't have specifics to point out right now. It was something I just generally remembered about my testing overall.

        If you implement the ARIA role="main" throughout the system, that would be great. I wouldn't get rid of your "skip to main content" links though. They still provide good access for keyboard only users.

        I'll be doing some more testing soon and I'll make better notes of where this problem occurs.

        Show
        Greg Kraus added a comment - It's been a while since I went through this testing. From what I remember there were times that the "main content" seemed to be in slightly different locations on different pages. I'm sorry, I don't have specifics to point out right now. It was something I just generally remembered about my testing overall. If you implement the ARIA role="main" throughout the system, that would be great. I wouldn't get rid of your "skip to main content" links though. They still provide good access for keyboard only users. I'll be doing some more testing soon and I'll make better notes of where this problem occurs.
        Hide
        Michael de Raadt added a comment -

        I had some feedback from Chetz at the OU. She agrees with the proposal to use the ARIA role "main".

        Show
        Michael de Raadt added a comment - I had some feedback from Chetz at the OU. She agrees with the proposal to use the ARIA role "main".
        Hide
        Michael de Raadt added a comment -

        I'm shifting this issue into a new Epic issue, where we are collecting together accessibility issues.

        Show
        Michael de Raadt added a comment - I'm shifting this issue into a new Epic issue, where we are collecting together accessibility issues.

          People

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

            Dates

            • Created:
              Updated: