Skip to content

feat: reorder spectra by drag and drop and persist the order in the original state - #4399

Merged
hamed-musallam merged 2 commits into
mainfrom
react-table-drag-drop-reorder
Oct 5, 2026
Merged

hamed-musallam merged 2 commits into
mainfrom
react-table-drag-drop-reorder

Conversation

@hamed-musallam

Copy link
Copy Markdown
Member

No description provided.

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Oct 2, 2026 •

Copy link
Copy Markdown

Deploying nmrium with  Cloudflare Pages  Cloudflare Pages

Latest commit: c06e3c0
Status: ✅  Deploy successful!
Preview URL: https://e247bea1.nmrium.pages.dev
Branch Preview URL: https://react-table-drag-drop-reorde.nmrium.pages.dev

View logs

@lpatiny

lpatiny commented Oct 2, 2026

Copy link
Copy Markdown
Member

Looks good !

Could we just make this first column much smaller so that it takes not a lot of room even maybe without separator just after ?

Also could we avoid that when we leave the droppable zones there is a '+' appearing. Seems weird.

2026-10-02 15 46 32

@hamed-musallam

Copy link
Copy Markdown
Member Author

Also could we avoid that when we leave the droppable zones there is a '+' appearing. Seems weird.

the '+' icon, it's Chrome’s native drag-and-drop cursor, and now I prevent showing the icon over the table .

@hamed-musallam

Copy link
Copy Markdown
Member Author

Be aware that Safari and Chrome generate the native drag preview differently.

@lpatiny lpatiny left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Works as expected, thanks.

fix: prevent copy cursor during table row reordering

Allow same table drop targets with a move effect and ignore self drops.

refactor: reduce table row drag handle width

refactor: prevent copy cursor when dragging table rows
@hamed-musallam
hamed-musallam force-pushed the react-table-drag-drop-reorder branch from dbe2efa to c06e3c0 Compare October 5, 2026 09:03
@hamed-musallam
hamed-musallam merged commit 70c276b into main Oct 5, 2026
12 checks passed
@hamed-musallam
hamed-musallam deleted the react-table-drag-drop-reorder branch October 5, 2026 09:16
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.

2 participants