Moodle

"Users tagged with ..." should be marked up as a list

Details

  • Type: Sub-task Sub-task
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 1.9
  • Fix Version/s: 1.9, 2.0
  • Component/s: Accessibility, Tags
  • Labels:
    None

Description

Several accessibility issues:

I have a patch, tested on Win/Firefox 2.0.0.8 and IE 6, affecting 2 files:

  • tag/lib.php, print_user_list (clue in the name) and print_user_box functions.
  • theme/standard/styles_layout.css - a long overdue fix for "inline-list" class!

(Also see bug MDL-11430)

Issue Links

Activity

Hide
Nick Freear added a comment -

Committed fixes to HEAD, and 1.9 branch tagged _MERGED.

(Luiz Cruz and Petr did a lot of work on tag/lib, see MDL-11048)

Show
Nick Freear added a comment - Committed fixes to HEAD, and 1.9 branch tagged _MERGED. (Luiz Cruz and Petr did a lot of work on tag/lib, see MDL-11048)
Hide
Nicolas Martignoni added a comment -

Verified on 1.9beta2, closing.

Show
Nicolas Martignoni added a comment - Verified on 1.9beta2, closing.
Hide
Martin Dougiamas added a comment -

On moodle.org these are now all being displayed in one column down the page which looks crappy ...

Show
Martin Dougiamas added a comment - On moodle.org these are now all being displayed in one column down the page which looks crappy ...
Hide
Martin Dougiamas added a comment -

Hmmm been playing with this and

.inline-list li {
display: inline;
}

doesn't appear to even work ...

Show
Martin Dougiamas added a comment - Hmmm been playing with this and .inline-list li { display: inline; } doesn't appear to even work ...
Hide
Martin Dougiamas added a comment -

Nick, can you look into this today? The fix appears to be non-trivial.

Show
Martin Dougiamas added a comment - Nick, can you look into this today? The fix appears to be non-trivial.
Hide
Martin Dougiamas added a comment -

Nick, can you at least confirm if you are able to work on this?

http://moodle.org/tag/index.php?id=6

Show
Martin Dougiamas added a comment - Nick, can you at least confirm if you are able to work on this? http://moodle.org/tag/index.php?id=6
Hide
Martin Dougiamas added a comment -

Nick C, can you please look into this ASAP?

Show
Martin Dougiamas added a comment - Nick C, can you please look into this ASAP?
Hide
Martin Dougiamas added a comment -

Sorry for the noise. I'll leave this closed and let Nick C work on it over at MDL-12941 instead.

Show
Martin Dougiamas added a comment - Sorry for the noise. I'll leave this closed and let Nick C work on it over at MDL-12941 instead.
Hide
Nicolas Martignoni added a comment -

More noise: closing again :-D

Show
Nicolas Martignoni added a comment - More noise: closing again :-D

People

Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: