Details

    • Type: Sub-task Sub-task
    • Status: Closed
    • Priority: Minor 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
          Aparup Banerjee added a comment -

          thanks, integrated into master.

          Show
          Aparup Banerjee added a comment - thanks, integrated into master.
          Hide
          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
          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
          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
          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: