search

Fastspot / bigtree-form-builder

Form Builder Extension for BigTree 4.2+
4 stars 13 forks source link

Lotsa Stuff #14

Open mimoduo opened 5 years ago

mimoduo commented 5 years ago

Tim!

I made a bunch of markup consistency updates and paired down the css to make it easier to style.

I tested by live injecting the stylesheet onto a sample Monticello form with all the fields set to required and added additional copied fields with the respective different options. So the changes may need some extra love afterwards.

Sincerely, Bryan Stoner

timbuckingham commented 5 years ago

I think I'm down with all these changes but we can't move them into the mainline Form Builder extension because they're a huge backwards compatibility break with any existing site that counts on the old markup / CSS styling being the same.

I think we might want to merge this into something like Form Builder 2 (like we did for Events when we broke backwards compatibility). Maybe we can add in some fancy feature like conditional fields into that as well if we have time.

mimoduo commented 5 years ago

Sounds good to me!

There are a few other areas I'd like to touch up on like required field support. Conditional fields sound awesome!