openstack/horizon
Code Issues Proposed changes

Material theme form errors - fix red text on red bg

Browse Source 
Fixed to match error styling on ng-Launch Instance wizard
Remove 'alert-message' class because it doesn't exist

Change-Id: I936ec521d44b5433c1663afefad2567fd2eef932
Closes-Bug: #1555833
This commit is contained in:
Cindy Lu
2016-03-29 14:13:52 -07:00
committed by Timur Sufiev
parent a379db6aa9
commit b934b16ea0
3 changed files with 5 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
horizon/static/horizon/js/horizon.modals.js
View File

@@ -113,7 +113,7 @@ horizon.modals.init_wizard = function () {
// Add global errors.
$.each(errors, function (index, error) {
$fieldset.find('div.row').prepend(
'<div class="alert alert-message alert-danger">' +
'<div class="alert alert-danger">' +
error + '</div>');
});
$fieldset.find('input, select, textarea').first().focus();
@@ -124,7 +124,7 @@ horizon.modals.init_wizard = function () {
$field.closest('.form-group').addClass('has-error');
$.each(errors, function (index, error) {
$field.after(
'<span class="help-block alert alert-danger">' +
'<span class="help-block">' +
error + '</span>');
});
// Focus the first invalid field.

2
horizon/static/horizon/js/horizon.users.js
View File

@@ -15,7 +15,7 @@ horizon.user = {
check_passwords_match: function() {
var row = $("input#id_confirm_password");
var error_id = "id_confirm_password_error";
var msg = "<span id='" + error_id + "' class='help-block alert alert-danger'>" + gettext("Passwords do not match.") + "</span>";
var msg = "<span id='" + error_id + "' class='help-block'>" + gettext("Passwords do not match.") + "</span>";
var password = $("#id_password").val();
var confirm_password = $("#id_confirm_password").val();

4
horizon/templates/horizon/common/_form_errors.html
View File

@@ -2,12 +2,12 @@
{{ hidden }}
{% endfor %}
{% if form.warnings %}
<div class="alert alert-message alert-warning">
<div class="alert alert-warning">
{{ form.warnings }}
</div>
{% endif %}
{% if form.non_field_errors %}
<div class="alert alert-message alert-danger">
<div class="alert alert-danger">
{{ form.non_field_errors }}
</div>
{% endif %}