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

          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