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

Error messages text not dispalying on formal_white theme

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 1.9.5
    • Fix Version/s: 1.9.6
    • Component/s: Themes
    • Labels:
      None
    • Affected Branches:
      MOODLE_19_STABLE
    • Fixed Branches:
      MOODLE_19_STABLE

      Description

      Error message text do not display on formal_white theme. You just get a error message box with no text in it.
      See screen shots for comparison of same errors on same system between standard and formal_white themes.
      Debugging was turned on.

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

              Hide
              amirelion Amir Elion added a comment -

              I think this is due to unneeded comment sign on css and can be resolved by editing the fw_color.css

              from:
              .errorbox

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

              to

              .errorbox

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

              or something similar.
              Worked on my install.

              Show
              amirelion Amir Elion added a comment - I think this is due to unneeded comment sign on css and can be resolved by editing the fw_color.css from: .errorbox { /* color:#ffffff; */ border-color:#990000; background-color:#FEF9F6; } to .errorbox { color:#000000; border-color:#990000; background-color:#FEF9F6; } or something similar. Worked on my install.
              Hide
              andreabix Andrea Bicciolo added a comment -

              It is fixed and committed in MOODLE_19_STABLE. Assigned QA to Patrick, if all is ok we can merge in weekly.

              Thanks for the report!

              Show
              andreabix Andrea Bicciolo added a comment - It is fixed and committed in MOODLE_19_STABLE. Assigned QA to Patrick, if all is ok we can merge in weekly. Thanks for the report!
              Hide
              aborrow Anthony Borrow added a comment -

              Andrea - Thanks for checking in the fix. The commit is exactly what I had suggested in the patch on MDL-19934. It would still be good if Patrick could give his seal of approval. Peace - Anthony

              Show
              aborrow Anthony Borrow added a comment - Andrea - Thanks for checking in the fix. The commit is exactly what I had suggested in the patch on MDL-19934 . It would still be good if Patrick could give his seal of approval. Peace - Anthony

                People

                • Votes:
                  1 Vote for this issue
                  Watchers:
                  2 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved:
                    Fix Release Date:
                    21/Oct/09