Closed ChrisKnightLDN closed 3 months ago
What does this change? This switches to using meltui instead of @rgossiaux/svelte-headlessui to implement the modals, and adds a popover component.
Why? @rgossiaux/svelte-headlessui unsupported and causing terminal errors
How? port of modal component to Melt UI
Related issues: Supersedes: https://github.com/Greater-London-Authority/ldn-viz-tools/pull/212
Does this introduce new dependencies? Yes - Melt ui
How is it tested? Storybook
How is it documented? Storybook
Are light and dark themes considered? Yes - though design review required for dark theming of modals
Is it complete?
index.js
Preview: https://dev.ldn-gis.co.uk/storybook-melt-modal/?path=/docs/ui-modal--documentation
jamesscottbrown
commented
4 months ago jamesscottbrown
commented
4 months ago
What does this change? This switches to using meltui instead of @rgossiaux/svelte-headlessui to implement the modals, and adds a popover component.
Why? @rgossiaux/svelte-headlessui unsupported and causing terminal errors
How? port of modal component to Melt UI
Related issues: Supersedes: https://github.com/Greater-London-Authority/ldn-viz-tools/pull/212
Does this introduce new dependencies? Yes - Melt ui
How is it tested? Storybook
How is it documented? Storybook
Are light and dark themes considered? Yes - though design review required for dark theming of modals
Is it complete?
index.js?