Radio input fields in Forms do not include the "form-control" CSS class anymore

Ce fil est résolu. Voici une description du problème et la solution proposée.

Problem: Radio input fields in Forms do not include the form-control CSS class anymore.

Solution: It's expected, though we realize it may cause some problems for existing sites that used this class for style selectors. From our lead developer:
"CRED used to have a setting about loading Bootstrap markup for forms, which added a form-control CSS class to some form elements. As we added fully native Bootstrap compatibility, that option was deprecated and removed. This also meant removing the extra class when it was turned on.
Consequently, some sites that had this option turned on will not output the CSS class in question. To resolve possible styling issues, the solution is to simply add a class=”form-control” attribute to the fields that used to get this class from CRED.
Unfortunately, this might require manual editing of some of the existing forms, but we honestly think this is the best path forward."

Relevant Documentation:
https://toolset.com/documentation/user-guides/styling-cred-forms/#toolset-forms-markup-overview

This support ticket is created Il y a 6 années et 5 mois. There's a good chance that you are reading advice that it now obsolete.

This is the technical support forum for Toolset - a suite of plugins for developing WordPress sites without writing PHP.

Everyone can read this forum, but only Toolset clients can post in it. Toolset support works 6 days per week, 19 hours per day.

Aucun de nos assistants n'est disponible aujourd'hui sur le forum Jeu d'outils. Veuillez créer un ticket, et nous nous le traiterons dès notre prochaine connexion. Merci de votre compréhension.

Ce sujet est une subdivision de https://toolset.com/fr/forums/topic/unable-to-display-custom-field-value-from-parent-post/

Sun	Mon	Tue	Wed	Thu	Fri	Sat
8:00 – 12:00	8:00 – 12:00	8:00 – 12:00	8:00 – 12:00	8:00 – 12:00	-	-
13:00 – 17:00	13:00 – 17:00	13:00 – 17:00	13:00 – 17:00	13:00 – 17:00	-	-

Supporter timezone: America/New_York (GMT-04:00)

Ce sujet contient 2 réponses, a 2 voix.

Dernière mise à jour par julieP Il y a 6 années et 5 mois.

Assisté par: Christian Cox.

Auteur

Publications

juin 11, 2018 at 2:18 pm #911849

julieP

Hi Christian

I really appreciate the offer to look at the install. I've since taken that install offline and am working on a fresh re-build in its place (it's a dev site). I'm not importing anything I've previously captured using module manager or WordPress importer. I've set up a basic form and a view with a post edit link displaying the parent post title and this is working as expected now.

That's the good news.

I've just settled down to creating more forms and I've noticed that whereas ALL cred fields used to contain class="form-control", now they don't. It seems to be the radio fields that don't. Is this expected?

juin 11, 2018 at 4:42 pm #911948

Christian Cox

It seems to be the radio fields that don't. Is this expected?
Yes, it's expected, though we realize it may cause some problems for existing sites that used this class for style selectors. From our lead developer:
"CRED used to have a setting about loading Bootstrap markup for forms, which added a form-control CSS class to some form elements. As we added fully native Bootstrap compatibility, that option was deprecated and removed. This also meant removing the extra class when it was turned on.
Consequently, some sites that had this option turned on will not output the CSS class in question. To resolve possible styling issues, the solution is to simply add a class=”form-control” attribute to the fields that used to get this class from CRED.
Unfortunately, this might require manual editing of some of the existing forms, but we honestly think this is the best path forward."

Also, please check the notes here about which types of fields accept the class attribute:
https://toolset.com/documentation/user-guides/styling-cred-forms/#toolset-forms-markup-overview

juin 12, 2018 at 5:28 am #912167

julieP

Hi Christian

Thank you for this and thanks also to the developer for providing a clear explanation.