History | Log In     View a printable version of the current page.  
We are currently focused especially on Moodle 2.0, Moodle 1.9.x bugs and Moodle 1.9.x testing.    Confused? Lost? Please read this introduction to the Tracker.
Issue Details (XML | Word | Printable)

Key: MDL-15501
Type: Bug Bug
Status: Closed Closed
Resolution: Fixed
Priority: Minor Minor
Assignee: Dongsheng Cai
Reporter: Kenneth Newquist
Votes: 0
Watchers: 1
Operations

If you were logged in you would be able to see more operations.
Moodle

"Add Blocks" block doesn't use standard formatting

Created: 02/Jul/08 05:10 AM   Updated: 09/Jul/08 06:37 PM
Component/s: Accessibility, Blocks, Themes
Affects Version/s: 1.9.1, 1.8.5
Fix Version/s: 1.8.6, 1.9.2

File Attachments: 1. Text File blocklib_addblock_patch.txt (0.5 kb)

Environment: Moodle 1.9.1 (Build: 20080515), PHP 5.2.0, MySQL 5.0.27, RHEL 4

Participants: Dongsheng Cai, Eloy Lafuente (stronk7), Kenneth Newquist and Petr Škoda
Security Level: None
QA Assignee: Petr Škoda

Sub-Tasks  All   Open   
 Sub-Task Progress: 

 Description  « Hide
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.



 All   Comments   Change History   Version Control      Sort Order: Ascending order - Click to sort in descending order
Eloy Lafuente (stronk7) - 02/Jul/08 07:39 AM
Assigning to Dongsheng... targets are 18_STABLE, 19_STABLE and HEAD.

Thanks for the report!

Dongsheng Cai - 02/Jul/08 01:03 PM
Fixed for 1.8, 1.9 and head, please review ;)

Kenneth Newquist - 02/Jul/08 09:42 PM
I tested it on Moodle 1.9.1+ (Build: 20080702) and it worked fine.

Petr Škoda - 09/Jul/08 06:37 PM
closing, thanks