Use `options.value` instead of `ev.currentTarget.value` for value handling within vanilla-renderers by TheZoker · Pull Request #2205

Use `options.value` instead of `ev.currentTarget.value` for value handling within vanilla-renderers by TheZoker · Pull Request #2205 · eclipsesource/jsonforms

Merged

sdirix merged 5 commits intoeclipsesource:masterfrom

TheZoker:fix-radio-option-type

Nov 13, 2023

Merged

Use `options.value` instead of `ev.currentTarget.value` for value handling within vanilla-renderers#2205
sdirix merged 5 commits intoeclipsesource:masterfrom
TheZoker:fix-radio-option-type

Conversation

Copy link

Contributor

TheZoker commented
Oct 31, 2023

This PR fixes a couple of issues:

Fix the folder reference for the example apps in the README file
Fix the index file naming within the example folder of vanilla-renderer (bug introduced in Make material renderers aware of input variant in theme #2182)
Use options.value instead of ev.currentTarget.value for value handling to preserve the type of the value

Fixes #2045

TheZoker added 2 commits

October 31, 2023 01:23

General fixes for the README & the vanilla example

9f6173a

Use option.value instead of ev.currentTarget.value to get options…

907d6ab

… value

Copy link

netlify bot commented
Oct 31, 2023
•

edited

Loading

✅ Deploy Preview for jsonforms-examples ready!

Name	Link
🔨 Latest commit	`00968ee`
🔍 Latest deploy log	https://app.netlify.com/sites/jsonforms-examples/deploys/654a0a144df9ef0008341e76
😎 Deploy Preview	https://deploy-preview-2205--jsonforms-examples.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.