Where is the checked propery for radio?

Question

Where is the checked propery for radio?

Closed this issue 5 years ago · 16 comments

forki commented 5 years ago

I can find a prop.isChecked, but that one is invalid

Answer 1 · 2020-03-08T12:17:14.000Z

It's called checked'. Don't you get intellisense for it?

Answer 2 · 2020-03-08T15:06:42.000Z

I think that's because he was looking for it in prop and not radio, and the property is radio.checked'

Answer 3 · 2020-03-08T15:07:29.000Z

Oh, right. Feliz.MaterialUI doesn't define anything in prop, that's from Feliz.

Answer 4 · 2020-03-08T21:39:57.000Z

From prop , it is called prop.isChecked

Answer 5 · 2020-03-08T23:47:52.000Z

From prop , it is called prop.isChecked

That's a different prop. The MUI Radio component has its own checked prop.

Answer 6 · 2020-03-08T23:52:22.000Z

You lost me.

There is no "checked". could you please try to write a demo. This would also help further generations

Answer 7 · 2020-03-09T08:02:18.000Z

Like this:

It's no different from any other MUI prop. Are you really not seeing it in your IDE?

Answer 8 · 2020-03-09T08:17:18.000Z

Ok did not see the prime. Thanks it works but please consider to put it in the docs. Thanks! Christer van der Meeren <notifications@github.com> schrieb am Mo., 9. März 2020, 09:02:

…

Like this: [image: image] <https://user-images.githubusercontent.com/7766733/76193607-aceef280-61e4-11ea-887f-185a41e64573.png> — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#41>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAAOANAKHO4JMJT3ZOSJMJ3RGSPAXANCNFSM4LDYGG5A> .

Answer 9 · 2020-03-09T09:58:18.000Z

seems something is still wrong

Answer 10 · 2020-03-09T12:03:42.000Z

That's a different issue. How are you using radio.checked'?

Answer 11 · 2020-03-09T12:11:04.000Z

forki commented 5 years ago

Answer 12 · 2020-03-09T12:14:36.000Z

Hmm, weird. The code is literally a simple as this, I can't see anything wrong on Feliz.MaterialUI's end here:

My guess is the problem lies elsewhere, but I don't know where. Which versions of Feliz.MaterialUI and @material-ui/core are you using?

Answer 13 · 2020-03-09T12:20:38.000Z

Feliz.MaterialUI (0.13.1)
"@material-ui/core@^4.9.2"

inside of electron

Answer 14 · 2020-03-09T12:23:50.000Z

Can you try replacing

radio.checked' config.SyncImagesOnlyWithWifi

with

prop.custom("checked", config.SyncImagesOnlyWithWifi)

If that also fails, the error is not in Feliz.MaterialUI.

Answer 15 · 2020-03-09T12:43:23.000Z

ok I'm offically confused now.

As you expected prop.custom("checked", config.SyncImagesOnlyWithWifi) did not make it work, BUT prop.custom("checked", config.SyncImagesOnlyWithWifi = true) did. So some additional check making it super explicitid that this a bool made it work. Is this a problem with some js prop types lib!? //cc @ncave

Answer 16 · 2020-03-09T13:02:36.000Z

Ok I found the reason. For some reason Sqlite is giving me 1 instead of a true. It never really is a problem because of js truthiness. But here it is. Anyway will try to work around that