Moodle
  1. Moodle
  2. MDL-28587

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

    Details

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

        Activity

        Hide
        Sam Hemelryk added a comment -

        Thanks Daniele, this has been integrated now

        Show
        Sam Hemelryk added a comment - Thanks Daniele, this has been integrated now
        Hide
        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
        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
        Eloy Lafuente (stronk7) added a comment -

        Sent upstream and closing, many thanks!

        Show
        Eloy Lafuente (stronk7) added a comment - Sent upstream and closing, many thanks!
        Hide
        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
        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: