Moodle

Error messages text not dispalying on formal_white theme

Details

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

Issue Links

This issue duplicates:
MDL-19934 Theme: Formal_white - errorbox text same as background Minor Closed

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
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
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
Permalink
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
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
Permalink
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
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

Vote (1)
Watch (2)

Dates

  • Created:
    Updated:
    Resolved:
Atlassian JIRA (v4.4.3#663-r165197) | Report a problem
Powered by a free Atlassian JIRA open source license for Moodle. Try JIRA - bug tracking software for your team.