Radio buttons not working with numeric value

[PeterDraex]

I've read the docs and implemented a form with radio buttons, but when you click on a radio button, the form state is not updated.

Here is a simplified code example.

const entities = [
  { id: 1, label: 'one' },
  { id: 2, label: 'two' },
];

type RadioForm = {
  entityID: number;
};

<For each={entities}>
  {(entity) => (
    <Field name="entityID" type="number">
      {(field, props) => (
        <label>
          <input
            {...props}
            type="radio"
            value={entity.id}
            checked={field.value == entity.id}
          />
          {entity.label}
        </label>
      )}
    </Field>
  )}
</For>
Current value: {getValue(radioForm, 'entityID')}

Manually adding onChange={(e) => setValue(radioForm, 'entityID', parseInt(e.target.value))} to the input fixes the issue, but I'm guessing that's not the correct use, since no other input type requires manually setting the new value.

The full example is available on Stackblitz.

Is this a bug?

[fabian-hiller]

What framework are you using? I am happy to share a working example from our playground with you.

[PeterDraex]

I'm using SolidJS. Right, in the StackBlitz example above, I used the playground as the basis.

The issue is that when I use number for the form field type, it breaks the radio buttons. I am assuming numeric values should be working too?

[fabian-hiller]

I don't think <input type="radio />" supports the .valueAsNumber property. When using number, the library doesn't do any transformation itself, it just reads the value of the HTML element.

[PeterDraex]

I see. Would a PR that implements the conversion be welcomed? I think this is an unnecessary caveat

[fabian-hiller]

No, because I plan to rewrite the library. I will take this into account when rewriting. For now, I recommend to stick with string and transform it afterwards.

[fabian-hiller]

I reopen it so that I don't forget this when rewriting the library.

[brandonpittman]

Glad I found this. I got stuck on this for a while this morning.