[Bese-devel] Patch for select fields

[Attila Lendvai]

On 6/19/06, Marijn Haverbeke <marijnh at gmail.com> wrote:
>
> Attached is a patch that makes some rather big changes to the selectfields in components/form.lisp, a small one to checkbox fields, and adds
> hidden fields. Because it seems there is little animo for overhauling the
> make-new-callback system, and the ucw_ajax branch uses hidden fields for
> callbacks that are allways needed, I decided to just hack stuff with hidden
> fields. Later, these hacks can be combined with the callback mechanisms in
> ucw_ajax.
>

i tried to apply this patch to ajax, but there were more bitrot issues then
i can handle now. but imho this patch were a useful addition. could you take
a look at it again? i can help with the ajax features on irc if we arrange a
'meeting'...

some quick comments:
 - there's get-parameter for the backends
 - i'd use iter stuff instead of do
 - record the independent changes in different records when easy/possible

- Setting the value of mapping select fields now work like you'd expect it -
> you give it a value of the kind that you would get if you call value,
> instead of giving it a text label.
>

i dropped an onchange auto ajax submit feature in the form example due to
(the lack of) this...

I added a function next-anon-field-name that generates unique field names,
> because the hidden-input trick requires some kind of name for the element
> even when none is specified. Radio buttons also use this function now - they
> used random strings before and a comment indicated that the person who put
> that there didn't think this was a very good approach either.
>

we can work out these, but always leave a TODO comment (as you did)

-- 
- attila

"- The truth is that I've been too considerate, and so became
unintentionally cruel...
- I understand.
- No, you don't understand! We don't speak the same language!"

Ingmar Bergman - Smultronstället (Wild Strawberries)
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mailman.common-lisp.net/pipermail/bese-devel/attachments/20060710/34bc19dd/attachment.html>