Details

    • Type: Sub-task
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.4
    • Fix Version/s: 2.5
    • Component/s: Administration
    • Labels:
    • Testing Instructions:
      Hide

      I'm not sure if this needs testing as it is just renaming a css class which is predefined for the purpose it is meant for. So the tables in this change will be consistent with other tables in Moodle.

      Show
      I'm not sure if this needs testing as it is just renaming a css class which is predefined for the purpose it is meant for. So the tables in this change will be consistent with other tables in Moodle.
    • Affected Branches:
      MOODLE_24_STABLE
    • Fixed Branches:
      MOODLE_25_STABLE
    • Pull Master Branch:
      wip-MDL-37591_master

      Description

      See META issue:

      There are many locations where the generalbox class is applied to tables in the core Moodle:

        Gliffy Diagrams

          Activity

          Hide
          nebgor Aparup Banerjee added a comment -

          thanks, integrated into master.

          Show
          nebgor Aparup Banerjee added a comment - thanks, integrated into master.
          Hide
          nebgor Aparup Banerjee added a comment -

          i just tested this by looking at the effects at the page in admin/mnet , faint table borders now seen, all ok!

          Show
          nebgor Aparup Banerjee added a comment - i just tested this by looking at the effects at the page in admin/mnet , faint table borders now seen, all ok!
          Hide
          stronk7 Eloy Lafuente (stronk7) added a comment -

          Surely you will be happy to know that your code is now part of Moodle upstream. Thanks, thanks!

          Closing as fixed, ciao

          Show
          stronk7 Eloy Lafuente (stronk7) added a comment - Surely you will be happy to know that your code is now part of Moodle upstream. Thanks, thanks! Closing as fixed, ciao

            People

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

              Dates

              • Created:
                Updated:
                Resolved:
                Fix Release Date:
                14/May/13