Skip to content

feat: use dialog with backdrop to avoid weird interactions behind#85

Merged
tpoisseau merged 2 commits intomainfrom
80-modal-editor-for-custom-atoms
Mar 3, 2026
Merged

feat: use dialog with backdrop to avoid weird interactions behind#85
tpoisseau merged 2 commits intomainfrom
80-modal-editor-for-custom-atoms

Conversation

@tpoisseau
Copy link
Contributor

  • refactor: move AtomLabelEditForm in his own file

Closes: #80

… form

* refactor: move `AtomLabelEditForm` in his own file

Closes: #80
@tpoisseau tpoisseau linked an issue Feb 23, 2026 that may be closed by this pull request
@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Feb 23, 2026

Deploying react-ocl with  Cloudflare Pages  Cloudflare Pages

Latest commit: c65f989
Status: ✅  Deploy successful!
Preview URL: https://4bf6fbab.react-ocl.pages.dev
Branch Preview URL: https://80-modal-editor-for-custom-a.react-ocl.pages.dev

View logs

@tpoisseau
Copy link
Contributor Author

I'll use yalc to check if it has the expected behavior on nmrium

@tpoisseau
Copy link
Contributor Author

CleanShot.2026-02-23.at.10.50.27.mp4

SGTM

@tpoisseau tpoisseau requested a review from lpatiny February 23, 2026 09:56
@lpatiny
Copy link
Contributor

lpatiny commented Feb 25, 2026

Is it normal the dialog leaves the container ?

image

@tpoisseau
Copy link
Contributor Author

I suppose it should not. It may be related to dialog.showModal. I'll check later.

@tpoisseau tpoisseau merged commit a0ed131 into main Mar 3, 2026
11 checks passed
@tpoisseau tpoisseau deleted the 80-modal-editor-for-custom-atoms branch March 3, 2026 07:48
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.

Modal editor for custom atoms ?

2 participants