Details

    • Type: Sub-task Sub-task
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 1.9.5
    • Fix Version/s: 2.0
    • Component/s: Accessibility
    • Labels:
      None
    • Affected Branches:
      MOODLE_19_STABLE
    • Fixed Branches:
      MOODLE_20_STABLE
    • Rank:
      837

      Description

      Courses may allow or deny access to guest (i.e., non-enrolled) users. In the "Available courses" list, this is indicated by the presence or absence of the "guest" icon. If guest access is allowed this icon is appended to the course title. The icon is also linked to to course home page, and its ALT attribute is, "This course allows guests to enter."

      Why it matters

      The primary problem is that the icon is also a link. This means that in one of the most common screen reader navigation patterns, reading through a list of all links on the page, the text, "This course allows guest access" appears once for every course. These links are useless in this context because the link text doesn't tell the user where the link leads.

      The second problem is that an important course property (guest access not allowed) is indicated only by absence of this icon. If all courses are so configured no indication is provided.

      Possible solution

      The icon should not be a link. It might be very successful as plain text (e.g., "Open to guests" or "Guest access allowed"). If an icon is desirable and understandable to sighted users, the could be replaced by an icon using CSS.

        Activity

        Hide
        Martin Dougiamas added a comment -

        I think the icon is desirable but you're right, it doesn't need to be a link.

        Rosie, can you remove the link from the icon? (and the enrolment icon beside it too).

        Show
        Martin Dougiamas added a comment - I think the icon is desirable but you're right, it doesn't need to be a link. Rosie, can you remove the link from the icon? (and the enrolment icon beside it too).
        Hide
        Rossiani Wijaya added a comment -

        created patch to remove link from the guess and enrollment icon.

        Show
        Rossiani Wijaya added a comment - created patch to remove link from the guess and enrollment icon.
        Hide
        Ray Lawrence added a comment -

        Ideally "Guest" would be linked to the lang string for standard Guest role so that this updates automatically.

        Show
        Ray Lawrence added a comment - Ideally "Guest" would be linked to the lang string for standard Guest role so that this updates automatically.
        Hide
        Rossiani Wijaya added a comment -

        Added Olli, Nick, Chetz and Leena as watchers.

        Your comments / feedback on this issue would be much appreciated.

        Thanks
        Rosie

        Show
        Rossiani Wijaya added a comment - Added Olli, Nick, Chetz and Leena as watchers. Your comments / feedback on this issue would be much appreciated. Thanks Rosie
        Hide
        Helen Foster added a comment -

        +1 for removing link. The string "This course allows guest users to enter" seems fine to me.

        Show
        Helen Foster added a comment - +1 for removing link. The string "This course allows guest users to enter" seems fine to me.
        Hide
        Olli Savolainen added a comment -

        +1 for removing link, as it appears as a separate element (looking at http://qa.moodle.net/ and http://demo.moodle.net/ front pages). It being a link implies now that the link target has something to do with guest access specifically, which it does not.

        The icon is very ambiguous so simple text would be preferable to avoid users requiring to learn the meaning of an icon that is supposedly relatively secondary to any user's goals.

        I agree with the report that indication of this state would be important regardless of whether guests are allowed or not, for the reason reported above. If the icon is kept: As a first thought, a simple black or grey "banned" circle over the current the icon should do - since this is not an error situation, do not use red. (There is a slight risk of cultural ambiguity, though)

        http://media.photobucket.com/image/forbidden/darenet/forbidden.png

        Show
        Olli Savolainen added a comment - +1 for removing link, as it appears as a separate element (looking at http://qa.moodle.net/ and http://demo.moodle.net/ front pages). It being a link implies now that the link target has something to do with guest access specifically, which it does not. The icon is very ambiguous so simple text would be preferable to avoid users requiring to learn the meaning of an icon that is supposedly relatively secondary to any user's goals. I agree with the report that indication of this state would be important regardless of whether guests are allowed or not, for the reason reported above. If the icon is kept: As a first thought, a simple black or grey "banned" circle over the current the icon should do - since this is not an error situation, do not use red. (There is a slight risk of cultural ambiguity, though) http://media.photobucket.com/image/forbidden/darenet/forbidden.png
        Hide
        Rossiani Wijaya added a comment -

        Petr is working on guess access (MDL-21782). I will post more info regarding this issue.

        Rosie

        Show
        Rossiani Wijaya added a comment - Petr is working on guess access ( MDL-21782 ). I will post more info regarding this issue. Rosie
        Hide
        Rossiani Wijaya added a comment -

        This issue is fixed through MDL-22795.

        Thanks
        Rosie

        Resolve

        Show
        Rossiani Wijaya added a comment - This issue is fixed through MDL-22795 . Thanks Rosie Resolve

          People

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

            Dates

            • Created:
              Updated:
              Resolved: