Moodle
  1. Moodle
  2. MDL-11506

Table headings do not respect align property: print_table regression

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor 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
            Chris Bandy added a comment -

            Forgot to mention that it validates as XHTML 1.0 Strict

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

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

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

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

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

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

            Show
            Chris Bandy added a comment - Bump. Still present on line 4291 of weblib.php v1.984
            Hide
            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 Marier added a comment - Trivial patch which removes the typo and uncomment the original line. I have tested it and it validates fine.
            Hide
            Martin Dougiamas added a comment -

            Cool, thanks!

            Show
            Martin Dougiamas added a comment - Cool, thanks!

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved: