Trigger setAttributes on all radio inputs

[frankieroberto]

This enables the conditional reveals to work where two different macros are used with the same name attribute (eg to enable grouping of radio inputs).

One possible fix for #1079.

[kr8n3r]

@frankieroberto /src/components/radios/radios.js:49:49: Block must not be padded by blank lines.

[frankieroberto]

@igloosi fixed, thanks.

[NickColley]

Hey Frankie,

Thanks for the suggestion.

I think we need to fix this in a way that doesn't look for inputs on the document, as that's created bugs in the past.

We're going to see what @colinrotherham has found out from his research and consider a fix based on that.

Are you OK for me to close this based on that?

[frankieroberto]

@NickColley can you explain what the issues are? Maybe there's a way to avoid them?

I did consider whether the event could be triggered on only the radio inputs which shared the same name attribute, but that was a bigger refactor, and triggering it on all radio buttons didn't seem to have any side-effects.

[NickColley]

You can see this bug: #1156

Let me know if you have any ideas. 👍

[NickColley]

I've spoken to @frankieroberto and since this is not blocking him immediately anymore we're going to close this and action the corresponding issue when we have time #1079.

Thanks @frankieroberto for helping us figure this out.