Closed (fixed)
Project:
XStandard WYSIWYG Editor
Version:
6.x-1.0-beta3
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Reporter:
Created:
29 Dec 2008 at 15:47 UTC
Updated:
26 Mar 2009 at 01:50 UTC
In the xstandard_process_textarea(); function, an assumption is made that $element['#suffix'] doesn't already exist and so is unconditionally set. It would be helpful if it was changed to something like:
$notice = '<span class="xstandard-id">'. t('Use XStandard to edit @title content by including this id: %id', array('@title' => $element['#title'], '%id' => $element['#id'])) .'</span>';
$element['#suffix'] = (!empty($element['#suffix']) ? $element['#suffix'] : '') . $notice;
Or even just:
$element['#suffix'] .= '<span class="xstandard-id">'. t('Use XStandard to edit @title content by including this id: %id', array('@title' => $element['#title'], '%id' => $element['#id'])) .'</span>';
I have data I'm throwing into #suffix that this module is overwriting.
Comments
Comment #1
henrrrik commentedGood idea. Commited. Apologies for letting this gather dust.