Issue Details (XML | Word | Printable)

Key: MDL-11506
Type: Bug Bug
Status: Resolved Resolved
Resolution: Fixed
Priority: Minor Minor
Assignee: Francois Marier
Reporter: Chris Bandy
Votes: 0
Watchers: 1
Operations

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

Table headings do not respect align property: print_table regression

Created: 29/Sep/07 12:20 PM   Updated: 07/Dec/07 01:07 PM
Component/s: Accessibility
Affects Version/s: 1.7, 1.7.1, 1.7.2, 1.8, 1.8.1, 1.8.2
Fix Version/s: 1.8.4, 1.9, 2.0

File Attachments: 1. Text File weblib.patch (0.9 kB)

Issue Links:
Relates
 

Participants: Chris Bandy, Francois Marier and Martin Dougiamas
Security Level: None
Resolved date: 07/Dec/07
Affected Branches: MOODLE_17_STABLE, MOODLE_18_STABLE
Fixed Branches: MOODLE_18_STABLE, MOODLE_19_STABLE, MOODLE_20_STABLE


 Description  « Hide
The styles erroneously removed at line 3455 of weblib.php v1.776 (line 4153 of v1.958) successfully validate.

The typo "sytle" may have caused the mix up.



 All   Comments   Change History   Version Control      Sort Order: Ascending order - Click to sort in descending order
Chris Bandy added a comment - 29/Sep/07 12:23 PM
Forgot to mention that it validates as XHTML 1.0 Strict

Martin Dougiamas added a comment - 01/Oct/07 01:49 PM
I have a fix, just waiting for CVS to work again

Chris Bandy added a comment - 18/Oct/07 04:41 AM
Bump. Still present on line 4258 of weblib.php v1.978

Chris Bandy added a comment - 14/Nov/07 05:22 AM
Bump. Still present on line 4291 of weblib.php v1.984

Francois Marier added a comment - 07/Dec/07 08:27 AM
Trivial patch which removes the typo and uncomment the original line.

I have tested it and it validates fine.


Martin Dougiamas added a comment - 07/Dec/07 12:48 PM
Cool, thanks!