Would be nice to see an option to edit this text to make it say other things.

Thanks for the great module though :)

Comments

joel07’s picture

Can this in any way be done???

seanr’s picture

Status: Active » Postponed

You can do this currently through the locales module. I may eventually add it to the admin settings page, but it won't happen for a while. That page is already quite cluttered.

mr.j’s picture

Using this as a guide
http://drupal.org/node/78562#comment-250863

I overrode the text like so:

// Insert into template.php:
function custom_links($links, $type='page') {

	if ($links['forward_links'] != '') {
		$links['forward_links']['title'] = 'E-mail';
	}

	return theme('links', $links, array('class' => 'links inline'));
}

// Replace links in node.tpl.php:
if ($node->links) { 
	print '<div class="links">' . custom_links($node->links) . '</div>';
}
jenlampton’s picture

Title: 'Email This Page' text » 'Email This Page' text on settings page / theme_forward_link function
Version: 5.x-1.x-dev » 6.x-1.x-dev
Status: Postponed » Active

I agree that changing this text should be easier. If not via the settings page (which could be made simpler by splitting forward and ePostcard into two modules - each with its own settings page - but I'll post that in another issue queue) then at least by putting the forward link (icon vs no icon, text with note type or not ) into it's own theme function. check out how the print module handles this...

function theme_print_format_link() {
  $print_html_link_class = variable_get('print_html_link_class', PRINT_HTML_LINK_CLASS_DEFAULT);
  $print_html_new_window = variable_get('print_html_new_window', PRINT_HTML_NEW_WINDOW_DEFAULT);
  $print_html_show_link = variable_get('print_html_show_link', PRINT_HTML_SHOW_LINK_DEFAULT);
  $print_html_link_text = variable_get('print_html_link_text', t('Printer-friendly version'));

  $img = drupal_get_path('module', 'print') .'/icons/print_icon.gif';
  $title = t('Display a printer-friendly version of this page.');
  $class = strip_tags($print_html_link_class);
  $new_window = $print_html_new_window;
  $format = _print_format_link_aux($print_html_show_link, $print_html_link_text, $img);

  return array('text' => $format['text'],
               'html' => $format['html'],
               'attributes' => print_fill_attributes($title, $class, $new_window),
              );
}

It'd be cool to get something like this into forward.

seanr’s picture

Status: Active » Fixed

This is now fixed in both versions; releases should be up any minute now.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.