Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add dialog html tag. #244

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Johann-Goncalves-Pereira

Recently, the super useful HTML tag get full support as you can see:

So, I made this as an open-source contribution.

@BenAttenborough
Copy link

Be really cool if Elm ever got updates

@ultrox
Copy link

ultrox commented Jun 20, 2023

Untill this modules updates here is link how to implement it in user land with ports https://www.lindsaykwardell.com/blog/native-dialog-in-elm

@laurentpayot
Copy link

@ultrox I tried to use dialog as described in the Lindsay Wardell blog post. It works well but:

  • It uses one port.
  • You cannot close the modal by clicking outside it.
  • You cannot replay the modal opening in the debugger (opening does not depend on the model, although you could easily fix that).

I ended up using a custom element. There is an example in this Elm discourse post.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants