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

Materialize (additional fields) radio buttons are broken #337

Closed
itsdeluxe opened this issue Mar 31, 2015 · 3 comments
Closed

Materialize (additional fields) radio buttons are broken #337

itsdeluxe opened this issue Mar 31, 2015 · 3 comments

Comments

@itsdeluxe
Copy link

Looks like adding radio fields in the Materialize theme is broken due to a missing 'for' attribute on the label tag. In other words, you can't select the radio buttons. Here is an example:

AccountsTemplates.addField({
_id: "account",
type: "radio",
displayName: "Choose account type",
select: [
{
text: "Type 1",
value: "t1",
},
{
text: "Type 2",
value: "t2",
},
],
});

This is the html generated:

Type 1
Type 2

Notice there is no 'for' attribute on the label tag. I believe it should look like this:

Type 1
Type 2

Please let me know if I am missing something and thank you for this awesome package!

@splendido
Copy link
Member

Hi @itsdeluxe,
did you solve this?

@garysxn
Copy link

garysxn commented Aug 11, 2015

Hi @itsdeluxe
I have same problem, unable to select radio buttons because 'for' attribute on label tag is missing.
are you resolve this problem?

my code:
{
_id: 'gender',
type: 'radio',
placeholder:'Gender',
displayName: 'Gender',
select:[{
text: "Male",
value: "male"
},
{
text: "Female",
value: "female"
}]
}

@splendido
Copy link
Member

@garysxn this should be solved right now...
See this line which was fixed by meteor-useraccounts/materialize@bfc239e
The same issue was reported also on the core repo: #421

Are you sure you're running on the latest version?

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

3 participants