required=True in newforms Fields does not result in an change in as_p

[anders conbere]

I noticed this today when writing some forms, and whipped up a quick
patch. It could probably use one or two more eyes to look at it. a) so
it doesn't get lost, b) to come up with some better variable names.

The problem is pretty simple, as_p, as_ul and as_table, don't include
an option to pass css classes into their fields. As a result it
impossibly to change the appearance of required fields. My patch
simple adds a new label into the string that defines each row, creates
a css_classes list that stores each of the classes to be added to the
row delineator, and the concatenates them with spaces and dumps them
in.

~ Anders

[anders conbere]

ugh.. forgot to include a link to the ticket I made with my patch in it.

http://code.djangoproject.com/ticket/4024

[Waylan Limberg]

There's already a ticket on this (3512) with a few patches and
discussion. I would suggest marking your's as a duplicate.

http://code.djangoproject.com/ticket/3512

anders conbere wrote:
> ugh.. forgot to include a link to the ticket I made with my patch in it.
>
> http://code.djangoproject.com/ticket/4024
>
> >
>

--
Waylan Limberg
way...@gmail.com

[anders conbere]

terrific, marked as a duplicate (which is horribly counter intuitive in trac)

~ Anders