Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-11506

Table headings do not respect align property: print_table regression

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • 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
    • Component/s: Accessibility
    • Labels:
      None
    • Affected Branches:
      MOODLE_17_STABLE, MOODLE_18_STABLE
    • Fixed Branches:
      MOODLE_18_STABLE, MOODLE_19_STABLE, MOODLE_20_STABLE

      Description

      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.

        Gliffy Diagrams

        1. weblib.patch
          0.9 kB
          Francois Marier

          Issue Links

            Activity

            Hide
            cbandy Chris Bandy added a comment -

            Forgot to mention that it validates as XHTML 1.0 Strict

            Show
            cbandy Chris Bandy added a comment - Forgot to mention that it validates as XHTML 1.0 Strict
            Hide
            dougiamas Martin Dougiamas added a comment -

            I have a fix, just waiting for CVS to work again

            Show
            dougiamas Martin Dougiamas added a comment - I have a fix, just waiting for CVS to work again
            Hide
            cbandy Chris Bandy added a comment -

            Bump. Still present on line 4258 of weblib.php v1.978

            Show
            cbandy Chris Bandy added a comment - Bump. Still present on line 4258 of weblib.php v1.978
            Hide
            cbandy Chris Bandy added a comment -

            Bump. Still present on line 4291 of weblib.php v1.984

            Show
            cbandy Chris Bandy added a comment - Bump. Still present on line 4291 of weblib.php v1.984
            Hide
            francois Francois Marier added a comment -

            Trivial patch which removes the typo and uncomment the original line.

            I have tested it and it validates fine.

            Show
            francois Francois Marier added a comment - Trivial patch which removes the typo and uncomment the original line. I have tested it and it validates fine.
            Hide
            dougiamas Martin Dougiamas added a comment -

            Cool, thanks!

            Show
            dougiamas Martin Dougiamas added a comment - Cool, thanks!

              People

              • Votes:
                0 Vote for this issue
                Watchers:
                1 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  11/Jan/08