search

inboundnow / retired-cta

Create Calls to Action for your WordPress site. Monitor and improve conversion rates, run A/B split tests, customize your own CTA templates and more
http://www.inboundnow.com/cta/
1 stars 3 forks source link

Added support for label|value entries for dropdown, checkbox, and radio ... #86

Closed burn0050 closed 9 years ago

burn0050 commented 9 years ago

...for forms. Added help text for fields.

atwellpub commented 9 years ago

Hey awesome. So far looks good. When I get back home I'll pull the merge request and test it out. There's also been a request to preselect an option. Would love to hear your comments on this.

atwellpub commented 9 years ago

I pulled it in and went over everything and it all looks good and strong man. Good job on your first commit.

burn0050 commented 9 years ago

Thanks!

I've been thinking about the pre-select. It seems there are 2. 1 is to overload what I did, adding a 3rd option via pipe that says which is the default selection. The other is to add another field which is for the pre-select.

I don't think those are the best options. I think it might be better to add a new way to add options.

Thanks,

Bill Burns

On Feb 21, 2015, at 7:21 PM, Hudson Atwell notifications@github.com wrote:

I pulled it in and went over everything and it all looks good and strong man. Good job on your first commit.

— Reply to this email directly or view it on GitHub.