Commit 5d896a2e authored by alexpott's avatar alexpott

Issue #2334067 by mgifford, LewisNyman: error.svg doesn't have sufficient color contrast

parent 42c9be93
<svg xmlns="http://www.w3.org/2000/svg" width="16" height="16" fill="#ea2800"><path d="M8.002 1c-3.868 0-7.002 3.134-7.002 7s3.134 7 7.002 7c3.865 0 7-3.134 7-7s-3.135-7-7-7zm4.025 9.284c.062.063.1.149.1.239 0 .091-.037.177-.1.24l-1.262 1.262c-.064.062-.15.1-.24.1s-.176-.036-.24-.1l-2.283-2.283-2.286 2.283c-.064.062-.15.1-.24.1s-.176-.036-.24-.1l-1.261-1.262c-.063-.062-.1-.148-.1-.24 0-.088.036-.176.1-.238l2.283-2.285-2.283-2.284c-.063-.064-.1-.15-.1-.24s.036-.176.1-.24l1.262-1.262c.063-.063.149-.1.24-.1.089 0 .176.036.24.1l2.285 2.284 2.283-2.284c.064-.063.15-.1.24-.1s.176.036.24.1l1.262 1.262c.062.063.1.149.1.24 0 .089-.037.176-.1.24l-2.283 2.284 2.283 2.284z"/></svg> <svg xmlns="http://www.w3.org/2000/svg" width="16" height="16" fill="#e32700"><path d="M8.002 1c-3.868 0-7.002 3.134-7.002 7s3.134 7 7.002 7c3.865 0 7-3.134 7-7s-3.135-7-7-7zm4.025 9.284c.062.063.1.149.1.239 0 .091-.037.177-.1.24l-1.262 1.262c-.064.062-.15.1-.24.1s-.176-.036-.24-.1l-2.283-2.283-2.286 2.283c-.064.062-.15.1-.24.1s-.176-.036-.24-.1l-1.261-1.262c-.063-.062-.1-.148-.1-.24 0-.088.036-.176.1-.238l2.283-2.285-2.283-2.284c-.063-.064-.1-.15-.1-.24s.036-.176.1-.24l1.262-1.262c.063-.063.149-.1.24-.1.089 0 .176.036.24.1l2.285 2.284 2.283-2.284c.064-.063.15-.1.24-.1s.176.036.24.1l1.262 1.262c.062.063.1.149.1.24 0 .089-.037.176-.1.24l-2.283 2.284 2.283 2.284z"/></svg>
\ No newline at end of file
...@@ -33,5 +33,5 @@ ...@@ -33,5 +33,5 @@
.admin-dblog .dblog-critical .icon, .admin-dblog .dblog-critical .icon,
.admin-dblog .dblog-alert .icon, .admin-dblog .dblog-alert .icon,
.admin-dblog .dblog-emergency .icon { .admin-dblog .dblog-emergency .icon {
background-image: url(../../../misc/icons/ea2800/error.svg); background-image: url(../../../misc/icons/e32700/error.svg);
} }
...@@ -35,7 +35,7 @@ function hook_filter_info_alter(&$info) { ...@@ -35,7 +35,7 @@ function hook_filter_info_alter(&$info) {
*/ */
function hook_filter_secure_image_alter(&$image) { function hook_filter_secure_image_alter(&$image) {
// Turn an invalid image into an error indicator. // Turn an invalid image into an error indicator.
$image->setAttribute('src', base_path() . 'core/misc/icons/ea2800/error.svg'); $image->setAttribute('src', base_path() . 'core/misc/icons/e32700/error.svg');
$image->setAttribute('alt', t('Image removed.')); $image->setAttribute('alt', t('Image removed.'));
$image->setAttribute('title', t('This image has been removed. For security reasons, only images from the local domain are allowed.')); $image->setAttribute('title', t('This image has been removed. For security reasons, only images from the local domain are allowed.'));
......
...@@ -834,7 +834,7 @@ function _filter_html_image_secure_process($text) { ...@@ -834,7 +834,7 @@ function _filter_html_image_secure_process($text) {
*/ */
function filter_filter_secure_image_alter(&$image) { function filter_filter_secure_image_alter(&$image) {
// Turn an invalid image into an error indicator. // Turn an invalid image into an error indicator.
$image->setAttribute('src', base_path() . 'core/misc/icons/ea2800/error.svg'); $image->setAttribute('src', base_path() . 'core/misc/icons/e32700/error.svg');
$image->setAttribute('alt', t('Image removed.')); $image->setAttribute('alt', t('Image removed.'));
$image->setAttribute('title', t('This image has been removed. For security reasons, only images from the local domain are allowed.')); $image->setAttribute('title', t('This image has been removed. For security reasons, only images from the local domain are allowed.'));
$image->setAttribute('height', '16'); $image->setAttribute('height', '16');
......
...@@ -89,7 +89,7 @@ function testImageSource() { ...@@ -89,7 +89,7 @@ function testImageSource() {
$csrf_path = $public_files_path . '/' . implode('/', array_fill(0, substr_count($public_files_path, '/') + 1, '..')); $csrf_path = $public_files_path . '/' . implode('/', array_fill(0, substr_count($public_files_path, '/') + 1, '..'));
$druplicon = 'core/misc/druplicon.png'; $druplicon = 'core/misc/druplicon.png';
$red_x_image = base_path() . 'core/misc/icons/ea2800/error.svg'; $red_x_image = base_path() . 'core/misc/icons/e32700/error.svg';
$alt_text = t('Image removed.'); $alt_text = t('Image removed.');
$title_text = t('This image has been removed. For security reasons, only images from the local domain are allowed.'); $title_text = t('This image has been removed. For security reasons, only images from the local domain are allowed.');
......
...@@ -74,7 +74,7 @@ protected static function buildStatusImageMap() { ...@@ -74,7 +74,7 @@ protected static function buildStatusImageMap() {
); );
$image_fail = array( $image_fail = array(
'#theme' => 'image', '#theme' => 'image',
'#uri' => 'core/misc/icons/ea2800/error.svg', '#uri' => 'core/misc/icons/e32700/error.svg',
'#width' => 18, '#width' => 18,
'#height' => 18, '#height' => 18,
'#alt' => 'Fail', '#alt' => 'Fail',
......
...@@ -93,6 +93,6 @@ abbr.ajax-changed { ...@@ -93,6 +93,6 @@ abbr.ajax-changed {
height: 14px; height: 14px;
width: 14px; width: 14px;
vertical-align: sub; vertical-align: sub;
background: url(../../../../misc/icons/ea2800/error.svg) no-repeat; background: url(../../../../misc/icons/e32700/error.svg) no-repeat;
background-size: contain; background-size: contain;
} }
...@@ -59,7 +59,7 @@ ...@@ -59,7 +59,7 @@
.messages--error { .messages--error {
background-color: #fcf4f2; background-color: #fcf4f2;
color: #a51b00; color: #a51b00;
background-image: url(../../../../misc/icons/ea2800/error.svg); background-image: url(../../../../misc/icons/e32700/error.svg);
border-color: #f9c9bf #f9c9bf #f9c9bf transparent; /* LTR */ border-color: #f9c9bf #f9c9bf #f9c9bf transparent; /* LTR */
box-shadow: -8px 0 0 #e62600; /* LTR */ box-shadow: -8px 0 0 #e62600; /* LTR */
} }
......
...@@ -208,7 +208,7 @@ small .admin-link:after { ...@@ -208,7 +208,7 @@ small .admin-link:after {
display: block; display: block;
} }
.system-status-report__status-icon--error:before { .system-status-report__status-icon--error:before {
background-image: url(../../../misc/icons/ea2800/error.svg); background-image: url(../../../misc/icons/e32700/error.svg);
} }
.system-status-report__status-icon--warning:before { .system-status-report__status-icon--warning:before {
background-image: url(../../../misc/icons/e29700/warning.svg); background-image: url(../../../misc/icons/e29700/warning.svg);
......
...@@ -274,7 +274,7 @@ function template_preprocess_update_project_status(&$variables) { ...@@ -274,7 +274,7 @@ function template_preprocess_update_project_status(&$variables) {
case UPDATE_NOT_SECURE: case UPDATE_NOT_SECURE:
case UPDATE_REVOKED: case UPDATE_REVOKED:
case UPDATE_NOT_SUPPORTED: case UPDATE_NOT_SUPPORTED:
$uri = 'core/misc/icons/ea2800/error.svg'; $uri = 'core/misc/icons/e32700/error.svg';
$text = t('Error'); $text = t('Error');
break; break;
case UPDATE_NOT_CHECKED: case UPDATE_NOT_CHECKED:
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment