Skip to content
Snippets Groups Projects
Commit ba6b8ec4 authored by Nate Lampton's avatar Nate Lampton
Browse files

Issue #2255295: Improve compatibility with screen readers.

parent 45a22e14
No related branches found
No related tags found
No related merge requests found
......@@ -3,12 +3,14 @@ span.ext {
width: 10px;
height: 10px;
padding-right: 12px;
text-decoration: none;
}
span.mailto {
background: url(extlink_s.png) -20px center no-repeat;
width: 10px;
height: 10px;
padding-right: 12px;
text-decoration: none;
}
/* Hide the extra spans when printing. */
......
......@@ -123,13 +123,7 @@ Drupal.extlink.applyClassAndSpan = function (links, class_name) {
$links_to_process = $(links);
}
else {
if (parseFloat($().jquery) < 1.2) {
$links_to_process = $(links).not('[img]');
}
else {
var links_with_images = $(links).find('img').parents('a');
$links_to_process = $(links).not(links_with_images);
}
$links_to_process = $(links).not('[img]');
}
$links_to_process.addClass(class_name);
var i;
......@@ -138,9 +132,10 @@ Drupal.extlink.applyClassAndSpan = function (links, class_name) {
var $link = $($links_to_process[i]);
if ($link.css('display') == 'inline' || $link.css('display') == 'inline-block') {
if (class_name == Drupal.settings.extlink.mailtoClass) {
$link.after('<span class=' + class_name + '><div class="element-invisible">' + Drupal.t('Email links icon') + '</div></span>');
}else {
$link.after('<span class=' + class_name + '><div class="element-invisible">' + Drupal.t('External Links icon') + '</div></span>');
$link.append('<span class="' + class_name + '"><span class="element-invisible"> ' + Drupal.settings.extlink.mailtoLabel + '</span></span>');
}
else {
$link.append('<span class="' + class_name + '"><span class="element-invisible"> ' + Drupal.settings.extlink.extLabel + '</span></span>');
}
}
}
......
......@@ -24,6 +24,7 @@ function extlink_page_build() {
drupal_add_js(array('extlink' => array(
'extTarget' => variable_get('extlink_target', 0),
'extClass' => variable_get('extlink_class', 'ext'),
'extLabel' => check_plain(variable_get('extlink_label', t('(link is external)'))),
'extImgClass' => variable_get('extlink_img_class', 0),
'extSubdomains' => variable_get('extlink_subdomains', 1),
'extExclude' => variable_get('extlink_exclude', ''),
......@@ -32,8 +33,9 @@ function extlink_page_build() {
'extCssExplicit' => variable_get('extlink_css_explicit', ''),
'extAlert' => variable_get('extlink_alert', 0),
'extAlertText' => variable_get('extlink_alert_text', 'This link will take you to an external web site. We are not responsible for their content.'),
'mailtoClass' => variable_get('extlink_mailto_class', 'mailto'))), 'setting'
);
'mailtoClass' => variable_get('extlink_mailto_class', 'mailto'),
'mailtoLabel' => check_plain(variable_get('extlink_mailto_label', t('(link sends e-mail)'))),
)), 'setting');
}
function extlink_admin_settings() {
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment