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
    • Rank:
      28506

      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.

      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: