Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Trivial Trivial
    • Resolution: Fixed
    • Affects Version/s: 1.4.3
    • Fix Version/s: None
    • Component/s: Course
    • Labels:
      None
    • Environment:
      All
    • Affected Branches:
      MOODLE_14_STABLE

      Description

      The table in Weekly wiew has two bugs:

      • one </td> that should be removed (this bug causes difficulties defining margin)
      • table should IMHO be same style that courses list on homepage (cellpadding and spacing similar to make consistent look)

      Attached a patch:

      tapio:/var# diff moodle/course/format/weeks/format.php www/course/format/weeks/format.php

      51c51

      < echo <table border=0 cellpadding=3 cellspacing=0 width=100%>;

      > echo <table width=\100%\ border=\0\ cellspacing=\5\ cellpadding=\5\>;

      64c64

      < echo </td><td width=*\>;

      > echo <td width=*\>;

      Please apply,

      Tapio K.

        Gliffy Diagrams

          Activity

          Hide
          Martin Dougiamas added a comment -

          From Jon Papaioannou (pj at moodle.org) Wednesday, 12 January 2005, 06:44 PM:

          Fixed the </td> bug by making the suggested correction.

          I didn't change the table attributes to maintain consistency with the other course formats.

          Show
          Martin Dougiamas added a comment - From Jon Papaioannou (pj at moodle.org) Wednesday, 12 January 2005, 06:44 PM: Fixed the </td> bug by making the suggested correction. I didn't change the table attributes to maintain consistency with the other course formats.
          Hide
          Michael Blake added a comment -

          Temporary transition to reassign bug to "pj".

          Show
          Michael Blake added a comment - Temporary transition to reassign bug to "pj".
          Hide
          Michael Blake added a comment -

          Re-closing bugs after re-assigning to "pj".

          Show
          Michael Blake added a comment - Re-closing bugs after re-assigning to "pj".

            People

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

              Dates

              • Created:
                Updated:
                Resolved: