mattmcleod

More small changes, this time to the CSS. 🙂

The basic gist is that you don’t want to imply functionality that isn’t there, so if the page is loaded with Javascript off then you don’t for example want the cursor set to “pointer” or for that matter to have the arrow.

The simplest way I can see to achieve this is to change the .collapseomatic class so that it reads thus:

html.js .collapseomatic {
        background-image: url(images/arrow-down.png);
        background-repeat: no-repeat;
        cursor: pointer;
}

.collapseomatic {
        padding: 0 0 10px 16px;
}

Though this does introduce a dependency on Modernizr so, hrm, not sure.

As a general rule you don’t want to set tabindex to anything other than 0 or -1 unless there’s a very good reason. Setting it to 0 will automatically add it to the natural tab focus order which is usually what you want — tab focus order should follow visual order.