Pending

User tests: Successful: Unsuccessful:

avatar nternetinspired
nternetinspired
7 Sep 2012

is a semantic element, not a presentational one. For stylistic purposes simply set font-weight in css instead. It's use in this context is incorrect.

See: http://html5doctor.com/i-b-em-strong-element/ for details.

Semantically, it would be preferable to wrap the module header in a

element too of course.

I shall submit an additional PR with suggested improvements to the module markup.

Thanks.

avatar nternetinspired nternetinspired - open - 7 Sep 2012
avatar nternetinspired
nternetinspired - comment - 7 Sep 2012

I appear to have spectacularly ballsed up this PR. I'll close this one and resubmit.

avatar nternetinspired nternetinspired - close - 7 Sep 2012

Add a Comment

Login with GitHub to post a comment