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
    • Rank:
      18358

      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: