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

Is there a format for required fields? #18

Closed
nurikabe opened this issue Dec 19, 2011 · 5 comments
Closed

Is there a format for required fields? #18

nurikabe opened this issue Dec 19, 2011 · 5 comments
Assignees

Comments

@nurikabe
Copy link

Looks like the Twitter Bootstrap guys are against required field error styles:

twbs/bootstrap#207

Does Mopa add this by any chance?

@ghost ghost assigned phiamo Dec 19, 2011
@phiamo
Copy link
Owner

phiamo commented Dec 19, 2011

hmm not sure what this exactly means.
we have a required marking by using asterisk on the labels.
You could easily use the attr => array("class" => "error") field to add a red outline for your required fields.
Sure it might be nice to have something like $builder->setAttribute('required', 'error_style') or something to automagically add the error class to required fields ...

@nurikabe
Copy link
Author

I only ask because a couple of months ago Bootstrap (or Mopa.. I'm not sure which but I think the former) used to automagically backlight problem fields in red after submission.

Being able to set a default error style for Mopa in config that gets applied to all error fields would be handy.

@phiamo
Copy link
Owner

phiamo commented Dec 19, 2011

error fields should indeet be highlighted,
i will check that ... give me some time

@phiamo
Copy link
Owner

phiamo commented Feb 3, 2012

As far as i see atm for 2.0 this should work as expected again!

@phiamo phiamo closed this as completed Feb 3, 2012
@nurikabe
Copy link
Author

nurikabe commented Feb 3, 2012

Time to upgrade I guess. Thanks!

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

2 participants