[Fusionforge-general] [Fusionforge-commits] r9195 - in branches/Branch_5_0/gforge/www: forum/include news news/admin pm tracker

Roland Mas lolando at debian.org
Sun Mar 21 21:41:12 CET 2010


Roland Mas, 2010-03-21 21:26:23 +0100 :

> Author: lolando
> Date: 2010-03-21 21:26:23 +0100 (Sun, 21 Mar 2010)
> New Revision: 9195
[...]
> Log:
> Fixed notepad invocations to refer to the correct form [#100]

[...]

> -		<strong><?php echo _('Message:'); ?></strong><?php echo notepad_button('document.forms[1].body') ?><?php echo utils_requiredField(); ?><br />
> +		<strong><?php echo _('Message:'); ?></strong><?php echo notepad_button('document.forms[2].body') ?><?php echo utils_requiredField(); ?><br />

As noticed by Alain, this fix is rather fragile, and it depends on the
theme not adding a new form in the header (or removing the Quick Jump
form, for instance).  There must be a better way to identify forms and
fields, but I'm clueless with Javascript so I'll leave it for someone
who knows what they're doing.  It shouldn't be hard™ :-)

Roland.
-- 
Roland Mas

One... two... one, two, many, lots!
  -- Lias, in Soul music (Terry Pratchett)




More information about the Fusionforge-general mailing list