Hi Massimo,
I sent you this last week :
I hope the patch is still up to date...

With this patch, radio widget can use 'cols' attribute ; As previously
done for
checkboxes.

Grettings,
PO.


diff -r dcd13ab59569 gluon/sqlhtml.py
--- a/gluon/sqlhtml.py  Tue Jul 20 02:43:46 2010 -0500
+++ b/gluon/sqlhtml.py  Tue Jul 20 10:00:06 2010 +0200
@@ -232,10 +232,25 @@
             else:
                 raise SyntaxError, 'widget cannot determine options
of %s' \
                     % field
-        opts = [TR(INPUT(_type='radio', _name=field.name,
+
+        options = [(k, v) for k, v in options if str(v)]
+        opts = []
+        cols = attributes.get('cols',1)
+        totals = len(options)
+        mods = totals%cols
+        rows = totals/cols
+        if mods:
+            rows += 1
+
+        for r_index in range(rows):
+            tds = []
+            for k, v in options[r_index*cols:(r_index+1)*cols]:
+                tds.append(TD(INPUT(_type='radio', _name=field.name,
                          requires=attr.get('requires',None),
                          hideerror=True, _value=k,
-                         value=value), v) for (k, v) in options if
str(v)]
+                         value=value), v))
+            opts.append(TR(tds))
+
         if opts:
             opts[-1][0][0]['hideerror'] = False
         return TABLE(*opts, **attr)
@@ -1199,3 +1214,4 @@
         components.append(TBODY(*tbody))

 form_factory = SQLFORM.factory # for backward compatibility,
deprecated
+




On 27 juil, 11:15, mdipierro <[email protected]> wrote:
> I thought it was included unless I got confused with a different
> patch. Please ask him to resend.
>
> On Jul 27, 3:21 am, Mathieu Clabaut <[email protected]> wrote:
>
> > Hi Massimo,
>
> >  A colleague of mine tell me he sent you a patch to include a horizontal
> > radio buttons in gluon (similar to the existing horizontal checkboxes).
> >  Has the patch a chance to make its way to the trunk ?
>
> > Best regards,
>
> > -Mathieu
>
> > On Tue, Jul 27, 2010 at 10:15, Alastair Medford
> > <[email protected]>wrote:
>
> > > I was wanting to have horizontal radio buttons in my forms, so I set
> > > about making my own widget. The widget works well, except for one odd
> > > behavior. When I load the form, the first radio button is selected as
> > > desired. But after the form submits, page refreshes etc, the last
> > > radio buttons in the list are selected. IE, rather than defaulting to
> > > the first element selected as it did the first time, it defaults to
> > > the last. As this form will be submitted many times in a row, and the
> > > first option is the most often used, this proves to be a bit of a
> > > pain. I've tried this in multiple browsers to the same effect, so I'm
> > > pretty sure it's happening due to how the code is written. But I'm not
> > > really sure where this behavior is written. Anyone have any ideas?
>
> > > The code for my modified widget is below:
>
> > > class RadioWidget_Horizontal(OptionsWidget):
>
> > >   �...@staticmethod
> > >    def widget(field, value, **attributes):
> > >        """
> > >        generates an unordered list of radio buttons, which has
> > >        a css class radio_list so it can be styled inline.
>
> > >        see also: :meth:`FormWidget.widget`
> > >        """
>
> > >        attr = OptionsWidget._attributes(field, {}, **attributes)
> > >        attr['_class'] = attr['_class'] + " radio_list"
>
> > >        if hasattr(field.requires, 'options'):
> > >            options = field.requires.options()
> > >        else:
> > >            raise SyntaxError, 'widget cannot determine options of %s'
> > > % field
> > >        opts = [LI(INPUT(_type='radio', _name=field.name,
> > >                         requires=attr.get('requires',None),
> > >                         hideerror=True, _value=k,
> > >                         value=value), v) for (k, v) in options if
> > > str(v)]
> > >        if opts:
> > >            opts[-1][0]['hideerror'] = False
> > >            opts[0][0]['_checked'] = "checked"
> > >        return UL(*opts, **attr)
>
>

Reply via email to