Moodle
  1. Moodle
  2. MDL-12830

Missing errorbox css in formal white theme

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 1.8.4, 1.9
    • Fix Version/s: 1.8.5, 1.9.1
    • Component/s: Themes
    • Labels:
      None
    • Affected Branches:
      MOODLE_18_STABLE, MOODLE_19_STABLE
    • Fixed Branches:
      MOODLE_18_STABLE, MOODLE_19_STABLE
    • Rank:
      30783

      Description

      Moodle standard theme has reddish background-color css declaration in styles_color.css
      .errorbox {
      color:#ffffff;
      border-color:#660000;
      background-color:#990000;
      }

      .errorboxcontent {
      background-color:#FFBBBB;
      }
      Current moodle standard distribution themes: chameleon, cornflower, metal, oceanblue and orangewhite have their own background colors for errorbox.
      However, popular formal_white is missing .errorbox and .errorboxcontent colors.

      Suggested fix for moodle\theme\formal_white\fw_color.css
      add
      .errorbox {
      color:#ffffff;
      border-color:#660000;
      background-color:#990000;
      }

      .errorboxcontent {
      }
      or something to that effect...
      Joseph

        Activity

        Hide
        Andrea Bicciolo added a comment -

        Fixed in 1.7, 1.8, 1.9 and HEAD

        Show
        Andrea Bicciolo added a comment - Fixed in 1.7, 1.8, 1.9 and HEAD
        Hide
        Andrea Bicciolo added a comment -

        Closing now. Thanks for the report.

        Show
        Andrea Bicciolo added a comment - Closing now. Thanks for the report.
        Hide
        Joseph Rézeau added a comment -

        Andrea,
        I do not understand why the red color is commented out for the error box. This is precisely what I remarked was missing in error messages.
        .errorbox

        { /* color:#ffffff; */ border-color:#990000; background-color:#FEF9F6; }

        should be:
        .errorbox

        { color:#ffffff; border-color:#990000; background-color:#FEF9F6; }

        Joseph

        Show
        Joseph Rézeau added a comment - Andrea, I do not understand why the red color is commented out for the error box. This is precisely what I remarked was missing in error messages. .errorbox { /* color:#ffffff; */ border-color:#990000; background-color:#FEF9F6; } should be: .errorbox { color:#ffffff; border-color:#990000; background-color:#FEF9F6; } Joseph
        Hide
        Andrea Bicciolo added a comment -

        Joseph, simply put a white text would be poorly contrasted over the current backgorund. That's all. Let me know If I'm missing something.

        Show
        Andrea Bicciolo added a comment - Joseph, simply put a white text would be poorly contrasted over the current backgorund. That's all. Let me know If I'm missing something.
        Hide
        Joseph Rézeau added a comment -

        Oops, you are right! What I want in fact is RED text over white background, inside a red box (similar in fact to the following ERROR or MISSING REQUIRED ERROR formattings in moodle 1.8 and 1.9 mforms:

        form.mform span.error, form.mform span.required

        { color: red; }

        form.mform div.error,form.mform fieldset.error

        { border: 1px solid red; padding: 5px; color: inherit; }
        Show
        Joseph Rézeau added a comment - Oops, you are right! What I want in fact is RED text over white background, inside a red box (similar in fact to the following ERROR or MISSING REQUIRED ERROR formattings in moodle 1.8 and 1.9 mforms: form.mform span.error, form.mform span.required { color: red; } form.mform div.error,form.mform fieldset.error { border: 1px solid red; padding: 5px; color: inherit; }

          People

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

            Dates

            • Created:
              Updated:
              Resolved: