Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Required fields #36

Open
paveltyavin opened this issue Oct 7, 2013 · 8 comments
Open

Required fields #36

paveltyavin opened this issue Oct 7, 2013 · 8 comments

Comments

@paveltyavin
Copy link

How required are being rendered? Is there any difference between requiered and not required?

@meteozond
Copy link

As I've seen it doesn't make any difference during rendering.

@troygrosfield
Copy link

Would be nice to add support for this feature.

@meteozond
Copy link

I think it's a bit problematic, because there is no any default approach to required field highlighting in bootstrap.

@meteozond
Copy link

As I can see in 3.1 required_css_class was added.
Now you can add your own css for required fields and define it's name on your form like this:

class Form:
    required_css_class = 'requiredField'

So, I think this issue can be closed.

@tzangms
Copy link
Owner

tzangms commented Dec 10, 2013

@meteozond Does it works for you?

I should update the document for this feature

@meteozond
Copy link

@tzangms Yes, It does.

@meteozond
Copy link

More than that, may be it would be useful, to use default class name (something like "bootstrap_form_required"). And give ability to redefine it in form.

@stasdavydov
Copy link

Is there a reason why not to add "required" attribute for mandatory form fields?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

6 participants
@tzangms @paveltyavin @meteozond @stasdavydov @troygrosfield and others