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

          Attachments

            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