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

      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.

        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: