Skip to content
Snippets Groups Projects
Commit df8c52cf authored by Lachlan Ennis's avatar Lachlan Ennis
Browse files

Issue #2748635 by aburke626, amit.drupal, reevo, elachlan: Code Standards Cleanup

parent 388ed8e7
No related branches found
No related tags found
No related merge requests found
......@@ -174,7 +174,7 @@
$links_to_process = $(links).not(links_with_images);
}
// Add data-extlink attribute.
$links_to_process.attr('data-extlink','');
$links_to_process.attr('data-extlink', '');
var i;
var length = $links_to_process.length;
for (i = 0; i < length; i++) {
......@@ -183,10 +183,12 @@
if (Drupal.settings.extlink.extUseFontAwesome) {
if (class_name === Drupal.settings.extlink.mailtoClass) {
$link[icon_placement]('<span class="fa-' + class_name + ' extlink"><i class="fa fa-envelope-o" title="' + Drupal.settings.extlink.mailtoLabel + '"></i></span>');
}else{
}
else {
$link[icon_placement]('<span class="fa-' + class_name + ' extlink"><i class="fa fa-external-link" title="' + Drupal.settings.extlink.extLabel + '"></i></span>');
}
}else{
}
else {
if (class_name === Drupal.settings.extlink.mailtoClass) {
$link[icon_placement]('<span class="' + class_name + '" aria-label="' + Drupal.settings.extlink.mailtoLabel + '"></span>');
}
......
......@@ -36,28 +36,34 @@ function extlink_menu() {
/**
* Implements hook_page_build().
*/
function extlink_page_build() {
function extlink_page_build(&$page) {
$path = drupal_get_path('module', 'extlink');
drupal_add_js($path . '/extlink.js', array('every_page' => TRUE));
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),
'extIconPlacement' => variable_get('extlink_icon_placement', 'append'),
'extSubdomains' => variable_get('extlink_subdomains', 1),
'extExclude' => variable_get('extlink_exclude', ''),
'extInclude' => variable_get('extlink_include', ''),
'extCssExclude' => variable_get('extlink_css_exclude', ''),
'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'),
'mailtoLabel' => check_plain(variable_get('extlink_mailto_label', t('(link sends e-mail)'))),
'extUseFontAwesome' => variable_get('extlink_use_font_awesome', FALSE),
),
), 'setting');
$page['#attached']['js'][] = [
'type' => 'file',
'data' => $path . '/extlink.js',
];
$page['#attached']['js'][] = [
'type' => 'setting',
'data' => [
'extlink' => [
'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),
'extIconPlacement' => variable_get('extlink_icon_placement', 'append'),
'extSubdomains' => variable_get('extlink_subdomains', 1),
'extExclude' => variable_get('extlink_exclude', ''),
'extInclude' => variable_get('extlink_include', ''),
'extCssExclude' => variable_get('extlink_css_exclude', ''),
'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'),
'mailtoLabel' => check_plain(variable_get('extlink_mailto_label', t('(link sends e-mail)'))),
'extUseFontAwesome' => variable_get('extlink_use_font_awesome', FALSE),
],
],
];
}
/**
......
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