Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-2350

Two small bugs in Weekly wiew

    Details

    • Type: Bug
    • Status: Closed
    • Priority: 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

          Attachments

            Activity

            Hide
            dougiamas 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
            dougiamas 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
            mblake Michael Blake added a comment -

            Temporary transition to reassign bug to "pj".

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

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

            Show
            mblake 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: