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

In formal_white, environmenttable does not highlight matched, unmatched and warnings on requirements

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.0.4, 2.1.1, 2.2
    • Fix Version/s: 2.0.5, 2.1.2
    • Component/s: Themes
    • Labels:
    • Testing Instructions:
      Hide

      go to Site administration -> Server -> Environment
      and look at the status column.
      "Ok" should be written in black with green background,
      "Check" should be written in black with yellow background,
      "Error" should be written in black with red background.

      Show
      go to Site administration -> Server -> Environment and look at the status column. "Ok" should be written in black with green background, "Check" should be written in black with yellow background, "Error" should be written in black with red background.
    • Affected Branches:
      MOODLE_20_STABLE, MOODLE_21_STABLE, MOODLE_22_STABLE
    • Fixed Branches:
      MOODLE_20_STABLE, MOODLE_21_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      MDL-28587_master

      Gliffy Diagrams

        Attachments

          Activity

          Hide
          samhemelryk Sam Hemelryk added a comment -

          Thanks Daniele, this has been integrated now

          Show
          samhemelryk Sam Hemelryk added a comment - Thanks Daniele, this has been integrated now
          Hide
          andyjdavis Andrew Davis added a comment -

          I only have a mixture of "ok"s and "checks" but presumably "error"s work to.

          I presume the master diff URL is meant to be https://github.com/kordan/moodle/compare/master...MDL-28587_master

          Presumably Sam was able to figure it out.

          Show
          andyjdavis Andrew Davis added a comment - I only have a mixture of "ok"s and "checks" but presumably "error"s work to. I presume the master diff URL is meant to be https://github.com/kordan/moodle/compare/master...MDL-28587_master Presumably Sam was able to figure it out.
          Hide
          stronk7 Eloy Lafuente (stronk7) added a comment -

          Sent upstream and closing, many thanks!

          Show
          stronk7 Eloy Lafuente (stronk7) added a comment - Sent upstream and closing, many thanks!
          Hide
          daniss Daniele Cordella added a comment -

          @Andrew: yes, Andrew! Sure. The master diff URL is the one you presumed. It was a silly typo I made from the beach. Thanks once again.

          Show
          daniss Daniele Cordella added a comment - @Andrew: yes, Andrew! Sure. The master diff URL is the one you presumed. It was a silly typo I made from the beach. Thanks once again.

            People

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

              Dates

              • Created:
                Updated:
                Resolved:
                Fix Release Date:
                10/Oct/11