Moodle

"Add Blocks" block doesn't use standard formatting

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Minor Minor
  • Resolution: Fixed
  • Affects Version/s: 1.8.5, 1.9.1
  • Fix Version/s: 1.8.6, 1.9.2
  • Component/s: Accessibility, Blocks, Themes
  • Labels:
    None
  • Environment:
    Moodle 1.9.1 (Build: 20080515), PHP 5.2.0, MySQL 5.0.27, RHEL 4
  • Affected Branches:
    MOODLE_18_STABLE, MOODLE_19_STABLE
  • Fixed Branches:
    MOODLE_18_STABLE, MOODLE_19_STABLE

Description

This is a minor issue, but it created problems in our custom theme.

Most blocks in Moodle use this XHTML code:

<div class="header"><div class="title"><h2>Upcoming Events</h2>

However, the "Blocks" block,which appears when you turn editing on, and which is used to add blocks to a course, does not follow this scheme. Instead, it looks like this:

<div class="header">Blocks</div>

This causes problems with themes which are expecting the convention used with other blocks (the H2 tag in the other block was apparently added for accessibility reasons). In order to bring this block into line with the others, I suggest the attached patch, which adds the appropriate div and H2 tags to the header div.

Activity

Hide
Eloy Lafuente (stronk7) added a comment -

Assigning to Dongsheng... targets are 18_STABLE, 19_STABLE and HEAD.

Thanks for the report!

Show
Eloy Lafuente (stronk7) added a comment - Assigning to Dongsheng... targets are 18_STABLE, 19_STABLE and HEAD. Thanks for the report!
Hide
Dongsheng Cai added a comment -

Fixed for 1.8, 1.9 and head, please review

Show
Dongsheng Cai added a comment - Fixed for 1.8, 1.9 and head, please review
Hide
Kenneth Newquist added a comment -

I tested it on Moodle 1.9.1+ (Build: 20080702) and it worked fine.

Show
Kenneth Newquist added a comment - I tested it on Moodle 1.9.1+ (Build: 20080702) and it worked fine.
Hide
Petr Škoda (skodak) added a comment -

closing, thanks

Show
Petr Škoda (skodak) added a comment - closing, thanks

Dates

  • Created:
    Updated:
    Resolved: