Moodle Community Sites

Middle column headings are H1, better as H2 - poor hierarchy!

Details

  • Type: Bug Bug
  • Status: Resolved Resolved
  • Priority: Minor Minor
  • Resolution: Fixed
  • Component/s: moodle.org
  • Labels:
    None

Description

Eloy, thanks for tackling the other 2 - this is the last!
The following headings in the content (I think) in the middle column would be better as <h2> - this may need a tweak to styling (see below):

Order | Level | Text
5 1 Welcome to Moodle!
6 1 Moodle Community
7 1 Moodle Development
8 1 Moodle Downloads

Example:
<h1 class="frontblock fronthead p1">Welcome to Moodle!</h1>

Becomes:
<h2 class="frontblock fronthead p1">Welcome to Moodle!</h2>

CSS, theme/moodleorange/moodle.css - about line 291:

frontblock { ... }

Becomes:
h1.frontblock, .frontblock { ... }


As with previous bugs, MDLSITE-49 and MDLSITE-293 the purpose of these improvements is twofold:

  • Improve the accessibility of http://moodle.org/
  • Make the site a better demonstration of Moodle, for any institutions evaluating Moodle for accessibility.

Activity

Hide
Eloy Lafuente (stronk7) added a comment -

Done,

I've changed those 4 <h1> to <h2> and chaged css from:

frontblock {

to

frontblock, h2.frontblock {

(leaving .frontblock alone was breaking the nice boxes around the text.

Tested under FF, IE and Safari.

Thanks Nick!

Closing, ciao

Show
Eloy Lafuente (stronk7) added a comment - Done, I've changed those 4 <h1> to <h2> and chaged css from:

frontblock {

to

frontblock, h2.frontblock {

(leaving .frontblock alone was breaking the nice boxes around the text. Tested under FF, IE and Safari. Thanks Nick! Closing, ciao
Hide
Nick Freear added a comment -

Thank you!

Show
Nick Freear added a comment - Thank you!

People

Vote (0)
Watch (1)

Dates

  • Created:
    Updated:
    Resolved: