- Issue created by @goz
- First commit to issue fork.
- Merge request !169Issue #3535231 by mogtofu33, goz: Add new stories: Display (for modal) → (Open) created by mogtofu33
- 🇫🇷France mogtofu33
Cannot be added on the display level, added on the modal level.
In the meantime fix the modal story by adding a wrapper, so now it works. Plus a group.
- 🇫🇷France goz
Modal with display works as expected.
But, i think this story should be placed in display component, not in modal. This is a way to implement a 'display' component and is not relevant by the modal component itself.
- 🇫🇷France goz
When display is in modal, clicking on radio change the preview radio status, not the one in the modal.
This is caused by the fact we display two form display.
If it's possible by the dsfr library, we should have two distinct event displays OR an action on one should dispatch on all
- 🇫🇷France mogtofu33
In my opinion this is not a problem, obviously it's coming from DSFR with some specific radio js that assume there is only one set of radios in the page.
see display-selector.js
and display.js.So there is nothing we can do worth the effort.
You will have the same kind of problems with other components like consent banner.