Issue Details (XML | Word | Printable)

Key: MDL-17570
Type: Bug Bug
Status: Closed Closed
Resolution: Fixed
Priority: Trivial Trivial
Assignee: Dongsheng Cai
Reporter: Liam Morland
Votes: 0
Watchers: 3
Operations

Add/Edit UI Mockup to this issue
If you were logged in you would be able to see more operations.
Moodle

jumpmenu should use CSS instead of align="center"

Created: 10/Dec/08 04:11 AM   Updated: 31/Dec/08 12:51 AM
Return to search
Component/s: General
Affects Version/s: 1.9.3
Fix Version/s: 1.9.4

Participants: Dongsheng Cai, Eloy Lafuente (stronk7), Liam Morland and Petr Skoda
Security Level: None
QA Assignee: Petr Skoda
Difficulty: Easy
Resolved date: 10/Dec/08
Affected Branches: MOODLE_19_STABLE
Fixed Branches: MOODLE_19_STABLE


 Description  « Hide
On the course homepage, the following code is used to center the jump menu:

<div align="center" class="jumpmenu">

This is invalid XHTML. To fix, replace the above code with:

<div style="text-align: center;" class="jumpmenu">

Or better yet, remove the align attribute and add the equivalent CSS declaration to the jumpmenu class.

The fix need to be made on line 329 of: /course/format/lams/format.php

 All   Comments   Change History   Version Control      Sort Order: Ascending order - Click to sort in descending order
Eloy Lafuente (stronk7) added a comment - 10/Dec/08 04:15 AM
Assignnig to Dongsheng... can you plz review all course format to see how this is being used? And do it in the correct way for formats using the incorrect one?

TIA, ciao


Dongsheng Cai added a comment - 10/Dec/08 04:25 PM
Fixed many align properties in div.

Petr Skoda added a comment - 31/Dec/08 12:51 AM
reviewed, thanks, closing