[dev] [PATCH] turba/add.php

Marc Jauvin marc at r4l.com
Wed Oct 12 09:25:45 PDT 2005


I tried removing the "notification" line for the focus, but it does not 
seem to work... a way around this would be to use the form name in the 
previous code I submitted:


$notification->push('var i = 0; var form = document.horde_form; while 
(form.elements[i] && (form.elements[i].type == \'hidden\' || 
form.elements[i].disabled)) i++; if (form.elements[i]) 
form.elements[i].focus();', 'javascript');


Marc Jauvin <marc at r4l.com> wrote:

> Jan Schneider <jan at horde.org> wrote:
>
>> Zitat von Marc Jauvin <marc at r4l.com>:
>>
>>> (when clicking on "New Contact", the focus is not set)
>>>
>>> The included patch fixes a focus problem (was using forms[0] instead
>>> of forms[1]) and also simplifies the JS code using a variable to
>>> point to the form.
>>
>> Your patch only works if there is a source selection field in the top right.
>>
>> I removed the code completely instead, the Horde_Form code is already
>> taking care of field focusing.
>
> Oh, well... at least I raised a flag! ;)
>
> --
> Marc Jauvin
> http://register4less.com
> 514-905-6500
>
>
> --
> Horde developers mailing list - Join the hunt: http://horde.org/bounties/
> Frequently Asked Questions: http://horde.org/faq/
> To unsubscribe, mail: dev-unsubscribe at lists.horde.org
>



-- 
Marc Jauvin
http://register4less.com
514-905-6500


More information about the dev mailing list