Affects Version/s: 2.5.3
- Turn off JS in your browser
- Enabled editing in a course
- Move an item
- Check the source code of the move-target (the dotted box)
- The link should now have the title "Move 'sdaf' to this location"
Fixed Branches:MOODLE_26_STABLE, MOODLE_27_STABLE
Pull from Repository:
Pull Master Branch:
Pull Master Diff URL:
Sprint:FRONTEND Sprint 13
When you are moving items (using the non-AJAX version), the move targets are written out using code like the following:
As you can see, there is a title attribute on the LI tag which does not make a lot of sense. It should be on the link tag. (Additionally, there is an unnecessary title attribute on the img tag.)
This was caused by
The previous code was:
This code is correct, with a title attribute on the A tag containing the full text, and an alt attribute (no title) on the IMG tag.
For accessibility reasons, none of these titles are suitable anyway. All links should have different texts so while the longer text includes the name of the item being moved, that doesn't actually help as it's still the same for all of them. An accessible solution would use text like:
'Move item to Week 4 before "Study resources"'
In other words, I didn't describe the item being moved (because they probably know, and it's the same for all of them) but I included the section name of the target location and a relative before/after for the position within that section.
I am not sure whether or not it is worth doing anything about this because I know Moodle HQ is planning to stop supporting the non-JS graceful degradation features. I was thinking about fixing the regression and putting it back to have the right attribute on the right tag, but since it's inaccessible even after doing that (and always was), I don't think I am going to bother...
So probably just filing this bug and leave it for Moodle staff to decide what to do, sorry!
The offending code where it puts the title on the li is course/renderer.php in course_section_cm_list, currently line 1097/1106, btw.