Skip to content

refactor: change useForm on AlignSpectra - #4324

Open
Sebastien-Ahkrin wants to merge 4 commits into
mainfrom
4192-refactor-useform-on-alignspectra
Open

refactor: change useForm on AlignSpectra#4324
Sebastien-Ahkrin wants to merge 4 commits into
mainfrom
4192-refactor-useform-on-alignspectra

Conversation

@Sebastien-Ahkrin

Copy link
Copy Markdown
Collaborator

Closes: #4192

@Sebastien-Ahkrin Sebastien-Ahkrin linked an issue Aug 24, 2026 that may be closed by this pull request
@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Aug 24, 2026

Copy link
Copy Markdown

Deploying nmrium with  Cloudflare Pages  Cloudflare Pages

Latest commit: 3717583
Status: ✅  Deploy successful!
Preview URL: https://49cf5dc1.nmrium.pages.dev
Branch Preview URL: https://4192-refactor-useform-on-ali.nmrium.pages.dev

View logs

@tpoisseau tpoisseau left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please explain how to show this form in nmrium.

Comment thread src/component/panels/multipleAnalysisPanel/AlignSpectra.tsx Outdated
Comment thread src/component/panels/multipleAnalysisPanel/AlignSpectra.tsx Outdated
@Sebastien-Ahkrin

Copy link
Copy Markdown
Collaborator Author

Go on localhost, open XTC Multiple spectra, open "Multiple Spectra Analysis Panel" then click on the second icon

@tpoisseau tpoisseau left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Image

There is an issue with scrolling, the content overflow but no scrolling, no scrollbar.
To show all the form fields, I need to fold spectra panel to see the rest of the form.

Image

I though submit button was missing but I saw it in the source code.

@Sebastien-Ahkrin

Copy link
Copy Markdown
Collaborator Author

Ok, i think removing the inner container class looks good on this case. Tell me if its not

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.

Refactor useForm on AlignSpectra

2 participants