Perhaps we can have "Sam Student" as text and next to it "View profile" as a link. That would give us consistency between these links and still inform the user of what account they are logged in as.

Thanks for reporting this Jason,

I am not sure if anything is wrong in having two links (with different names), pointing to same url.

As per standard: The purpose of each link can be determined from the link text alone or from the link text together with its programmatically determined link context.

which seems to be fine here. Also, looking at G19, G53, it seems current implementation is fine, as enclosing text or text on link is sufficiently explaining the link.

Please suggest if we are missing something here.

It is true that these links appear in slightly different contexts. The links in the header and footer indicate the name of the user logged in where the link in the navigation appears below the text "My profile".

We could make the change I suggested above, but do we really need to.

What do you think, Jason?

I believe the issue is that the first link actually does not convey context. The reason for this is that Sam Student is ambiguous especially when seen in light of the same url with a different name. View profile sets the context of where I am going when i click the link Sam Student does not unless I look at the exact url to determine where I am going. Even then I only know I am going to user/view.php I can assume that is viewing the user but I can't be certain.

I think Michael's suggestion would be a fine solution. We have implemented different link names based on what the user will be doing and not linking the user's name to anything other than an expandable menu with the correct aria tags.

I would point to the attached links Techniques item 8. THis describes the programmatically determine link context using the previous information around the link, which Sam student does not provide in my opinion. Another option would be just to add a title that says view profile as per item 7 in the link.

Thanks for the feedback Jason,

I have added title for user profile link.

[Y] Syntax
[Y] Output
[Y] Whitespace
[Y] Language
[-] Databases
[Y] Testing
[-] Security
[-] Documentation
[Y] Git
[Y] Sanity check

The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week.

TIA and ciao

Hi Raj,

Please could you clarify the testing instructions on this issue - they do not make clear that we are talking about the loginas feature, and not just the normal login page.

Thanks Dan,

done.

Integrated to master, 24 and 23 - thanks Raj.

(Btw, to give my two cents - I would probably have converted those urls to moodle_urls, rather than continue to have the ugly &amp and escaped quote marks etc, but there you go )

I'll test this now.

This solution does not do what is intended. When tabbing to the user link, NVDA reads "Admin User link" and completely ignores the title attribute.

Sorry (I followed the testing instructions)!

So - interesting note for accessibility testing - NVDA only reads the title attribute for links in Firefox - not chrome or IE.

From testing various screen readers in different browsers it seems that reading the title attribute of a link that has content is rarely supported - in fact NVDA with firefox was the only supported combination.

It would be good to know what happens with chrome - but we also tested the screen reader on iOS and that did not read the title attribute.

Reading through the linked document in this issue it says:

Because of the extensive user agent limitations in supporting access to the title attribute, authors should use caution in applying this technique. For this reason, it is preferred that the author use technique C7: Using CSS to hide a portion of the link text (CSS) or H30: Providing link text that describes the purpose of a link for anchor elements.

We can modify the testing instructions to pass it because it is consistent to how we have been addressing this in other issues - but IMO it is flawed and does not help 99% of people.

Will raise a new issue for this as it applies more broadly than just this one link.

This issue can pass as it works as expected (but only in NVDA/firefox).

Passing but will raise new issue.

Your awesome contributions are now part of Moodle, your fav LMS out there.

Closing this as fixed.

Many thanks for all the hard work, ciao