[Tx-builder] Improvements to drag and drop component

Open
#459 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
3/5
Estimated time
1-2 days
Newbie friendliness
38/100
Issue type
Bug
Clarity
Mostly clear
Activity status
Stale
Tech stack
typescript
Domain
frontend

Research direction

Open the Tx builder app and reproduce the issue with a batch larger than 20 transactions, dragging the bottom transaction to the top. Locate the drag-and-drop component responsible for transaction ordering and verify that the dragged transaction remains in the intended position with larger batches.

Written by the indexing model from the issue text.

Description

Enhancement ✨

Description

Drag and drop component gets a bit buggy when there are more than 20txs on a batch making ordering txs a bit tricky.

Steps to reproduce

-Open Tx builder app and create a batch larger than 20 txs.
-Try to drag and drop a tx from the bottom of the list to the top.

Expected result

Smooth drag and drop experience.

Obtained result

Dragged tx is moved automatically to the bottom of the list.

https://user-images.githubusercontent.com/92332710/173811921-d855600f-e6d2-4f98-85e5-86820d9f356d.mp4

Dominant language
TypeScript
Stars
116
Forks
84
PR merge metrics
No merged PRs in 30d

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from safe-global/safe-react-apps

All issues in safe-global/safe-react-apps

Similar issues

More TypeScript issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.